From eb33155db248973335f9f539031ff19645b7ea4d Mon Sep 17 00:00:00 2001 From: Jannat Patel Date: Tue, 24 Feb 2026 18:13:02 +0530 Subject: [PATCH] fix: enqueue progress calculation after validating enrollments --- frontend/src/pages/Lesson.vue | 1 - lms/lms/doctype/course_lesson/course_lesson.py | 9 ++++++--- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/frontend/src/pages/Lesson.vue b/frontend/src/pages/Lesson.vue index a334af82..5bf85fb7 100644 --- a/frontend/src/pages/Lesson.vue +++ b/frontend/src/pages/Lesson.vue @@ -743,7 +743,6 @@ const updateVideoTime = (video) => { const startTimer = () => { if (!lesson.data?.membership) return timerInterval = setInterval(() => { - console.log('Timer:', timer.value) timer.value++ if (timer.value == 30) { clearInterval(timerInterval) diff --git a/lms/lms/doctype/course_lesson/course_lesson.py b/lms/lms/doctype/course_lesson/course_lesson.py index 8c0c821c..8d546e00 100644 --- a/lms/lms/doctype/course_lesson/course_lesson.py +++ b/lms/lms/doctype/course_lesson/course_lesson.py @@ -16,15 +16,15 @@ from ...md import find_macros class CourseLesson(Document): def after_insert(self): - frappe.enqueue(method=self.recalculate_progress, queue="long", is_async=True) + self.validate_progress_recalculation() def after_delete(self): - frappe.enqueue(method=self.recalculate_progress, queue="long", is_async=True) + self.validate_progress_recalculation() def on_update(self): self.validate_quiz_id() - def recalculate_progress(self): + def validate_progress_recalculation(self): if not self.course or not self.chapter: return @@ -36,6 +36,9 @@ class CourseLesson(Document): if not len(enrollments): return + frappe.enqueue(method=self.recalculate_progress, queue="long", is_async=True, enrollments=enrollments) + + def recalculate_progress(self, enrollments): for enrollment in enrollments: recalculate_course_progress(self.course, enrollment.member)