fix: minor issues

community/lms/doctype/lms_batch_membership/lms_batch_membership.py

@@ -55,7 +55,7 @@ def create_membership(batch, member=None, member_type="Student", role="Member"):
     return "OK"
 
 @frappe.whitelist()
-def update_current_membership(batch, course, member=frappe.session.user):
+def update_current_membership(batch, course, member):
     all_memberships = frappe.get_all("LMS Batch Membership", {"member": member, "course": course})
     for membership in all_memberships:
         frappe.db.set_value("LMS Batch Membership", membership.name, "is_current", 0)

community/lms/doctype/lms_course/lms_course.py

@@ -192,16 +192,16 @@ class LMSCourse(Document):
             return
         return f"/courses/{self.name}/learn/{lesson_number}"
 
-    def get_current_batch(self, member=frappe.session.user):
+    def get_current_batch(self, member):
         current_membership = frappe.get_all("LMS Batch Membership", {"member": member, "course": self.name, "is_current": 1}, pluck="batch")
-        print(current_membership, member, self.name)
+        print(current_membership, member, self.name, frappe.session.user)
         if len(current_membership):
             return current_membership[0]
         print(frappe.db.get_value("LMS Batch Membership", {"member": member, "course": self.name}, "batch"))
         return frappe.db.get_value("LMS Batch Membership", {"member": member, "course": self.name}, "batch")
 
     def get_all_memberships(self, member=frappe.session.user):
-        print(member)
+        print(member, frappe.session.user)
         all_memberships = frappe.get_all("LMS Batch Membership", {"member": member, "course": self.name}, ["batch", "is_current"])
         print(all_memberships)
         for membership in all_memberships:

community/lms/widgets/BatchTabs.html

@@ -65,7 +65,8 @@
         method: "community.lms.doctype.lms_batch_membership.lms_batch_membership.update_current_membership",
         args: {
           batch: batch,
-          course: course
+          course: course,
+          member: frappe.session.user
         },
         callback: (data) => {
           window.location.reload();