From 9470cc192cce6e100a0fed0fa2087ffa08bea25f Mon Sep 17 00:00:00 2001 From: Jannat Patel Date: Wed, 10 Dec 2025 14:25:10 +0530 Subject: [PATCH] fix: batch validations --- lms/lms/doctype/lms_batch_enrollment/lms_batch_enrollment.py | 2 +- lms/lms/utils.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/lms/lms/doctype/lms_batch_enrollment/lms_batch_enrollment.py b/lms/lms/doctype/lms_batch_enrollment/lms_batch_enrollment.py index 839e7a43..63cd4d2a 100644 --- a/lms/lms/doctype/lms_batch_enrollment/lms_batch_enrollment.py +++ b/lms/lms/doctype/lms_batch_enrollment/lms_batch_enrollment.py @@ -25,7 +25,7 @@ class LMSBatchEnrollment(Document): return roles = frappe.get_roles(self.owner) - if not ("Moderator" in roles or "Batch Evaluator" in roles): + if "Moderator" not in roles and "Batch Evaluator" not in roles: frappe.throw(_("You must be a Moderator or Batch Evaluator to enroll users in a batch.")) def validate_duplicate_members(self): diff --git a/lms/lms/utils.py b/lms/lms/utils.py index 44b4cb23..fd4babf9 100644 --- a/lms/lms/utils.py +++ b/lms/lms/utils.py @@ -2068,7 +2068,7 @@ def enroll_in_batch(batch, payment_name=None): frappe.throw(_("The specified batch does not exist.")) batch_doc = frappe.db.get_value( - "LMS Batch", batch, ["name", "seat_count", "allow_self_enrollment"], as_dict=True + "LMS Batch", batch, ["name", "seat_count", "allow_self_enrollment", "paid_batch"], as_dict=True ) payment_doc = get_payment_details(payment_name) validate_enrollment_eligibility(batch_doc, payment_doc)