Merge pull request #1297 from pateljannat/issues-72

fix: check for duplicates before creating batch enrollment in patch
This commit is contained in:
Jannat Patel
2025-02-10 18:03:31 +05:30
committed by GitHub
2 changed files with 13 additions and 12 deletions

View File

@@ -98,4 +98,4 @@ lms.patches.v2_0.update_desk_access_for_lms_roles
lms.patches.v2_0.update_quiz_submission_data lms.patches.v2_0.update_quiz_submission_data
lms.patches.v2_0.convert_quiz_duration_to_minutes lms.patches.v2_0.convert_quiz_duration_to_minutes
lms.patches.v2_0.allow_guest_access #05-02-2025 lms.patches.v2_0.allow_guest_access #05-02-2025
lms.patches.v2_0.migrate_batch_student_data lms.patches.v2_0.migrate_batch_student_data #10-02-2025

View File

@@ -16,6 +16,9 @@ def execute():
) )
for student in students: for student in students:
if not frappe.db.exists(
"LMS Batch Enrollment", {"member": student.student, "batch": student.parent}
):
doc = frappe.new_doc("LMS Batch Enrollment") doc = frappe.new_doc("LMS Batch Enrollment")
doc.member = student.student doc.member = student.student
doc.member_name = student.student_name doc.member_name = student.student_name
@@ -25,5 +28,3 @@ def execute():
doc.batch = student.parent doc.batch = student.parent
doc.confirmation_email_sent = student.confirmation_email_sent doc.confirmation_email_sent = student.confirmation_email_sent
doc.save() doc.save()
frappe.delete_doc("DocType", "Batch Student")