MIDNIGHT/enlight-lms
2
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #2012 from raizasafeel/fix/chapter-deletion

Browse Source 
fix(chapter): recalculate indices on deletion to prevent duplicate index errors
This commit is contained in:
Jannat Patel
2026-01-29 16:37:44 +05:30
committed by GitHub
parent 7360780022 59e40d5a83
commit 3170c066dc
1 changed files with 13 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
lms/lms/api.py
View File

@@ -1056,11 +1056,24 @@ def delete_chapter(chapter):
if chapterInfo.is_scorm_package:
delete_scorm_package(chapterInfo.scorm_package_path)
course = frappe.db.get_value("Chapter Reference", {"chapter": chapter}, "parent")
frappe.db.delete("Chapter Reference", {"chapter": chapter})
frappe.db.delete("Lesson Reference", {"parent": chapter})
frappe.db.delete("Course Lesson", {"chapter": chapter})
frappe.db.delete("Course Chapter", chapter)
# reset chapter reference index after deletion
if course:
chapters = frappe.get_all(
"Chapter Reference", filters={"parent": course}, fields=["name"], order_by="idx asc"
)
i = 1
for chapter in chapters:
frappe.db.set_value("Chapter Reference", chapter.name, "idx", i)
i += 1
def delete_scorm_package(scorm_package_path):
scorm_package_path = frappe.get_site_path("public", scorm_package_path[1:])