nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-05-26 00:14:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #36265 from frappe/mergify/bp/version-14-hotfix/pr-36259

Browse Source 
fix: no default email account causing reposting issue (backport #36259)
This commit is contained in:
rohitwaghchaure
2023-07-24 20:02:30 +05:30
committed by GitHub
parent e867fe77a4 026f9f5a69
commit 4c35942c2d
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
erpnext/stock/doctype/repost_item_valuation/repost_item_valuation.py
View File

@@ -256,7 +256,11 @@ def repost(doc):
message += "<br>" + "Traceback: <br>" + traceback message += "<br>" + "Traceback: <br>" + traceback
frappe.db.set_value(doc.doctype, doc.name, "error_log", message) frappe.db.set_value(doc.doctype, doc.name, "error_log", message)
if not isinstance(e, RecoverableErrors): outgoing_email_account = frappe.get_cached_value(
"Email Account", {"default_outgoing": 1, "enable_outgoing": 1}, "name"
)
if outgoing_email_account and not isinstance(e, RecoverableErrors):
notify_error_to_stock_managers(doc, message) notify_error_to_stock_managers(doc, message)
doc.set_status("Failed") doc.set_status("Failed")
finally: finally: