fix: enrollment eligibility conditions
This commit is contained in:
@@ -39,7 +39,7 @@ class LMSEnrollment(Document):
|
|||||||
"reference_doctype": "LMS Course",
|
"reference_doctype": "LMS Course",
|
||||||
"reference_docname": self.course,
|
"reference_docname": self.course,
|
||||||
"member": self.member,
|
"member": self.member,
|
||||||
"payment_receipt": True,
|
"payment_received": True,
|
||||||
},
|
},
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|||||||
@@ -1800,7 +1800,7 @@ def validate_enrollment_eligibility(batch_doc, payment_doc=None):
|
|||||||
if not payment_doc or not payment_doc.payment_received:
|
if not payment_doc or not payment_doc.payment_received:
|
||||||
frappe.throw(_("Payment is required to enroll in this batch."))
|
frappe.throw(_("Payment is required to enroll in this batch."))
|
||||||
|
|
||||||
elif not batch_doc.allow_self_enrollment:
|
elif not batch_doc.allow_self_enrollment and not is_admin():
|
||||||
frappe.throw(_("Enrollment in this batch is restricted. Please contact the Administrator."))
|
frappe.throw(_("Enrollment in this batch is restricted. Please contact the Administrator."))
|
||||||
|
|
||||||
students = frappe.db.count("LMS Batch Enrollment", {"batch": batch_doc.name})
|
students = frappe.db.count("LMS Batch Enrollment", {"batch": batch_doc.name})
|
||||||
@@ -1808,6 +1808,11 @@ def validate_enrollment_eligibility(batch_doc, payment_doc=None):
|
|||||||
frappe.throw(_("There are no seats available in this batch."))
|
frappe.throw(_("There are no seats available in this batch."))
|
||||||
|
|
||||||
|
|
||||||
|
def is_admin():
|
||||||
|
roles = frappe.get_roles(frappe.session.user)
|
||||||
|
return "Course Creator" in roles or "Moderator" in roles or "Batch Evaluator" in roles
|
||||||
|
|
||||||
|
|
||||||
def create_enrollment(batch, payment_doc=None):
|
def create_enrollment(batch, payment_doc=None):
|
||||||
new_student = frappe.new_doc("LMS Batch Enrollment")
|
new_student = frappe.new_doc("LMS Batch Enrollment")
|
||||||
new_student.update(
|
new_student.update(
|
||||||
|
|||||||
Reference in New Issue
Block a user