fix: converted tabs to spaces
This commit is contained in:
@@ -8,14 +8,14 @@ from frappe.model.document import Document
|
|||||||
from community.www.courses.utils import get_member_with_email
|
from community.www.courses.utils import get_member_with_email
|
||||||
|
|
||||||
class LMSBatch(Document):
|
class LMSBatch(Document):
|
||||||
def validate(self):
|
def validate(self):
|
||||||
if not self.code:
|
if not self.code:
|
||||||
self.generate_code()
|
self.generate_code()
|
||||||
|
|
||||||
def generate_code(self):
|
def generate_code(self):
|
||||||
short_code = frappe.db.get_value("LMS Course", self.course, "short_code")
|
short_code = frappe.db.get_value("LMS Course", self.course, "short_code")
|
||||||
course_batches = frappe.get_all("LMS Batch",{"course":self.course})
|
course_batches = frappe.get_all("LMS Batch",{"course":self.course})
|
||||||
self.code = short_code + str(len(course_batches) + 1)
|
self.code = short_code + str(len(course_batches) + 1)
|
||||||
|
|
||||||
@frappe.whitelist()
|
@frappe.whitelist()
|
||||||
def get_messages(batch):
|
def get_messages(batch):
|
||||||
|
|||||||
@@ -9,35 +9,35 @@ from frappe import _
|
|||||||
|
|
||||||
class LMSBatchMembership(Document):
|
class LMSBatchMembership(Document):
|
||||||
|
|
||||||
def validate(self):
|
def validate(self):
|
||||||
self.validate_membership_in_same_batch()
|
self.validate_membership_in_same_batch()
|
||||||
self.validate_membership_in_different_batch_same_course()
|
self.validate_membership_in_different_batch_same_course()
|
||||||
|
|
||||||
def validate_membership_in_same_batch(self):
|
def validate_membership_in_same_batch(self):
|
||||||
previous_membership = frappe.db.get_value("LMS Batch Membership", {"member": self.member, "batch": self.batch, "name": ["!=", self.name]}, ["member_type","member"], as_dict=1)
|
previous_membership = frappe.db.get_value("LMS Batch Membership", {"member": self.member, "batch": self.batch, "name": ["!=", self.name]}, ["member_type","member"], as_dict=1)
|
||||||
if previous_membership:
|
if previous_membership:
|
||||||
member_name = frappe.db.get_value("Community Member", self.member, "full_name")
|
member_name = frappe.db.get_value("Community Member", self.member, "full_name")
|
||||||
frappe.throw(_("{0} is already a {1} of {2}").format(member_name, previous_membership.member_type, self.batch))
|
frappe.throw(_("{0} is already a {1} of {2}").format(member_name, previous_membership.member_type, self.batch))
|
||||||
|
|
||||||
def validate_membership_in_different_batch_same_course(self):
|
def validate_membership_in_different_batch_same_course(self):
|
||||||
course = frappe.db.get_value("LMS Batch", self.batch, "course")
|
course = frappe.db.get_value("LMS Batch", self.batch, "course")
|
||||||
previous_membership = frappe.get_all("LMS Batch Membership", {"member": self.member}, ["batch", "member_type"])
|
previous_membership = frappe.get_all("LMS Batch Membership", {"member": self.member}, ["batch", "member_type"])
|
||||||
for membership in previous_membership:
|
for membership in previous_membership:
|
||||||
batch_course = frappe.db.get_value("LMS Batch", membership.batch, "course")
|
batch_course = frappe.db.get_value("LMS Batch", membership.batch, "course")
|
||||||
if batch_course == course and (membership.member_type == "Student" or self.member_type == "Student"):
|
if batch_course == course and (membership.member_type == "Student" or self.member_type == "Student"):
|
||||||
member_name = frappe.db.get_value("Community Member", self.member, "full_name")
|
member_name = frappe.db.get_value("Community Member", self.member, "full_name")
|
||||||
frappe.throw(_("{0} is already a {1} of {2} course through {3} batch").format(member_name, membership.member_type, course, membership.batch))
|
frappe.throw(_("{0} is already a {1} of {2} course through {3} batch").format(member_name, membership.member_type, course, membership.batch))
|
||||||
|
|
||||||
@frappe.whitelist()
|
@frappe.whitelist()
|
||||||
def create_membership(batch, course, member=None, member_type="Student", role="Member"):
|
def create_membership(batch, course, member=None, member_type="Student", role="Member"):
|
||||||
if not member:
|
if not member:
|
||||||
member = frappe.db.get_value("Community Member", {"email": frappe.session.user}, "name")
|
member = frappe.db.get_value("Community Member", {"email": frappe.session.user}, "name")
|
||||||
frappe.get_doc({
|
frappe.get_doc({
|
||||||
"doctype": "LMS Batch Membership",
|
"doctype": "LMS Batch Membership",
|
||||||
"batch": batch,
|
"batch": batch,
|
||||||
"role": role,
|
"role": role,
|
||||||
"member_type": member_type,
|
"member_type": member_type,
|
||||||
"member": member
|
"member": member
|
||||||
}).save(ignore_permissions=True)
|
}).save(ignore_permissions=True)
|
||||||
course_slug = frappe.db.get_value("LMS Course", {"title": course}, ["slug"])
|
course_slug = frappe.db.get_value("LMS Course", {"title": course}, ["slug"])
|
||||||
return course_slug
|
return course_slug
|
||||||
|
|||||||
Reference in New Issue
Block a user