style: fixed formatting

frontend/src/pages/CourseForm.vue

@@ -446,7 +446,7 @@ const submitCourse = () => {
 }
 
 const deleteCourse = createResource({
-	url: "lms.lms.api.delete_course",
+	url: 'lms.lms.api.delete_course',
 	makeParams(values) {
 		return {
 			course: props.courseName,
@@ -461,7 +461,9 @@ const deleteCourse = createResource({
 const trashCourse = () => {
 	$dialog({
 		title: __('Delete Course'),
-		message: __('Deleting the course will also delete all its chapters and lessons. Are you sure you want to delete this course?'),
+		message: __(
+			'Deleting the course will also delete all its chapters and lessons. Are you sure you want to delete this course?'
+		),
 		actions: [
 			{
 				label: __('Delete'),

frontend/src/pages/Quizzes.vue

@@ -47,7 +47,10 @@
 			</ListRows>
 		</ListView>
 	</div>
-	<div v-else class="text-center p-5 text-gray-600 mt-52 w-3/4 md:w-1/2 mx-auto space-y-2">
+	<div
+		v-else
+		class="text-center p-5 text-gray-600 mt-52 w-3/4 md:w-1/2 mx-auto space-y-2"
+	>
 		<BookOpen class="size-10 mx-auto stroke-1 text-gray-500" />
 		<div class="text-xl font-medium">
 			{{ __('No quizzes found') }}

lms/install.py

@@ -2,6 +2,7 @@ import frappe
 from frappe.desk.page.setup_wizard.setup_wizard import add_all_roles_to
 from lms.lms.api import give_dicussions_permission
 
+
 def after_install():
 	add_pages_to_nav()
 	create_batch_source()

lms/lms/api.py

@@ -811,25 +811,22 @@ def get_announcements(batch):
 		order_by="communication_date desc",
 	)
 
+
 @frappe.whitelist()
 def delete_course(course):
 
-	chapters = frappe.get_all("Course Chapter", {
+	chapters = frappe.get_all("Course Chapter", {"course": course}, pluck="name")
-		"course": course
-	}, pluck="name")
 
-	chapter_references = frappe.get_all("Chapter Reference", {
+	chapter_references = frappe.get_all(
-		"parent": course
+		"Chapter Reference", {"parent": course}, pluck="name"
-	}, pluck="name")
+	)
 
 	for chapter in chapters:
-		lessons = frappe.get_all("Course Lesson", {
+		lessons = frappe.get_all("Course Lesson", {"chapter": chapter}, pluck="name")
-			"chapter": chapter
-		}, pluck="name")
 
-		lesson_references = frappe.get_all("Lesson Reference", {
+		lesson_references = frappe.get_all(
-			"parent": chapter
+			"Lesson Reference", {"parent": chapter}, pluck="name"
-		}, pluck="name")
+		)
 
 		for lesson in lesson_references:
 			frappe.delete_doc("Lesson Reference", lesson)
@@ -837,15 +834,14 @@ def delete_course(course):
 		for lesson in lessons:
 			frappe.db.delete("LMS Course Progress", {"lesson": lesson})
 
-			topics = frappe.get_all("Discussion Topic", {
+			topics = frappe.get_all(
-				"reference_doctype": "Course Lesson",
+				"Discussion Topic",
-				"reference_docname": lesson
+				{"reference_doctype": "Course Lesson", "reference_docname": lesson},
-			}, pluck="name")
+				pluck="name",
+			)
 
 			for topic in topics:
-				frappe.db.delete("Discussion Reply", {
+				frappe.db.delete("Discussion Reply", {"topic": topic})
-					"topic": topic
-				})
 
 				frappe.db.delete("Discussion Topic", topic)
 
@@ -862,19 +858,19 @@ def delete_course(course):
 
 
 def give_dicussions_permission():
-    doctypes = ["Discussion Topic", "Discussion Reply"]
+	doctypes = ["Discussion Topic", "Discussion Reply"]
-    roles = ["LMS Student", "Course Creator", "Moderator", "Batch Evaluator"]
+	roles = ["LMS Student", "Course Creator", "Moderator", "Batch Evaluator"]
-    for doctype in doctypes:
+	for doctype in doctypes:
-        for role in roles:
+		for role in roles:
-            if not frappe.db.exists("Custom DocPerm", {"parent": doctype, "role": role}):
+			if not frappe.db.exists("Custom DocPerm", {"parent": doctype, "role": role}):
-                frappe.get_doc(
+				frappe.get_doc(
-                    {
+					{
-                        "doctype": "Custom DocPerm",
+						"doctype": "Custom DocPerm",
-                        "parent": doctype,
+						"parent": doctype,
-                        "role": role,
+						"role": role,
-                        "read": 1,
+						"read": 1,
-                        "write": 1,
+						"write": 1,
-                        "create": 1,
+						"create": 1,
-                        "delete": 1,
+						"delete": 1,
-                    }
+					}
-                ).save(ignore_permissions=True)
+				).save(ignore_permissions=True)

lms/lms/doctype/course_chapter/course_chapter.py

@@ -8,9 +8,7 @@ from lms.lms.api import update_course_statistics
 
 
 class CourseChapter(Document):
-	
 	def on_update(self):
-		print("on_update")
 		self.recalculate_course_progress()
 		update_course_statistics()
 
@@ -19,15 +17,9 @@ class CourseChapter(Document):
 		current_lessons = self.lessons
 
 		if previous_lessons != current_lessons:
-			enrolled_members = frappe.get_all("LMS Enrollment", {
+			enrolled_members = frappe.get_all(
-				"course": self.course
+				"LMS Enrollment", {"course": self.course}, ["member", "name"]
-			}, ["member", "name"])
+			)
-			print("enrolled_members", enrolled_members)
 			for enrollment in enrolled_members:
-				print(self.course, enrollment.member)
 				new_progress = get_course_progress(self.course, enrollment.member)
-				print("new_progress", new_progress)
 				frappe.db.set_value("LMS Enrollment", enrollment.name, "progress", new_progress)
-
-
-

lms/patches/v2_0/add_course_statistics.py

@@ -1,5 +1,6 @@
 import frappe
 from lms.lms.api import update_course_statistics
 
+
 def execute():
 	update_course_statistics()

lms/patches/v2_0/give_discussions_permissions.py

@@ -1,5 +1,6 @@
 import frappe
 from lms.lms.api import give_dicussions_permission
 
+
 def execute():
 	give_dicussions_permission()