eduvia/lms
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: check site conf before sending email

Browse Source
This commit is contained in:
Jannat Patel
2024-04-10 19:38:30 +05:30
parent 13536aac51
commit 20fab8dbd3
5 changed files with 21 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lms/job/doctype/lms_job_application/lms_job_application.py
View File

@@ -14,7 +14,7 @@ class LMSJobApplication(Document):
outgoing_email_account = frappe.get_cached_value(
"Email Account", {"default_outgoing": 1, "enable_outgoing": 1}, "name"
)
if outgoing_email_account:
if outgoing_email_account or frappe.conf.get("mail_login"):
self.send_email_to_employer()
def validate_duplicate(self):

2
lms/lms/doctype/lms_assignment_submission/lms_assignment_submission.py
View File

@@ -18,7 +18,7 @@ class LMSAssignmentSubmission(Document):
outgoing_email_account = frappe.get_cached_value(
"Email Account", {"default_outgoing": 1, "enable_outgoing": 1}, "name"
)
if outgoing_email_account:
if outgoing_email_account or frappe.conf.get("mail_login"):
self.send_mail()
def validate_duplicates(self):

4
lms/lms/doctype/lms_batch/lms_batch.py
View File

@@ -73,7 +73,9 @@ class LMSBatch(Document):
outgoing_email_account = frappe.get_cached_value(
"Email Account", {"default_outgoing": 1, "enable_outgoing": 1}, "name"
)
if not student.confirmation_email_sent and outgoing_email_account:
if not student.confirmation_email_sent and (
outgoing_email_account or frappe.conf.get("mail_login")
):
self.send_mail(student)
student.confirmation_email_sent = 1

2
lms/lms/doctype/lms_certificate/lms_certificate.py
View File

@@ -18,7 +18,7 @@ class LMSCertificate(Document):
outgoing_email_account = frappe.get_cached_value(
"Email Account", {"default_outgoing": 1, "enable_outgoing": 1}, "name"
)
if outgoing_email_account:
if outgoing_email_account or frappe.conf.get("mail_login"):
self.send_mail()
def send_mail(self):