mirror of
https://github.com/frappe/erpnext.git
synced 2026-05-24 07:29:22 +00:00
Merge pull request #42000 from blaggacao/payments/pr-3
perf: flush db at once
This commit is contained in:
@@ -158,11 +158,11 @@ class PaymentRequest(Document):
|
|||||||
# set advance payment status
|
# set advance payment status
|
||||||
ref_doc.set_advance_payment_status()
|
ref_doc.set_advance_payment_status()
|
||||||
|
|
||||||
def on_submit(self):
|
def before_submit(self):
|
||||||
if self.payment_request_type == "Outward":
|
if self.payment_request_type == "Outward":
|
||||||
self.db_set("status", "Initiated")
|
self.status = "Initiated"
|
||||||
elif self.payment_request_type == "Inward":
|
elif self.payment_request_type == "Inward":
|
||||||
self.db_set("status", "Requested")
|
self.status = "Requested"
|
||||||
|
|
||||||
if self.payment_request_type == "Inward":
|
if self.payment_request_type == "Inward":
|
||||||
send_mail = self.payment_gateway_validation() if self.payment_gateway else None
|
send_mail = self.payment_gateway_validation() if self.payment_gateway else None
|
||||||
@@ -234,9 +234,6 @@ class PaymentRequest(Document):
|
|||||||
if self.payment_account and self.payment_channel != "Phone":
|
if self.payment_account and self.payment_channel != "Phone":
|
||||||
self.payment_url = self.get_payment_url()
|
self.payment_url = self.get_payment_url()
|
||||||
|
|
||||||
if self.payment_url:
|
|
||||||
self.db_set("payment_url", self.payment_url)
|
|
||||||
|
|
||||||
def get_payment_url(self):
|
def get_payment_url(self):
|
||||||
if self.reference_doctype != "Fees":
|
if self.reference_doctype != "Fees":
|
||||||
data = frappe.db.get_value(
|
data = frappe.db.get_value(
|
||||||
@@ -361,7 +358,14 @@ class PaymentRequest(Document):
|
|||||||
)
|
)
|
||||||
],
|
],
|
||||||
}
|
}
|
||||||
enqueue(method=frappe.sendmail, queue="short", timeout=300, is_async=True, **email_args)
|
enqueue(
|
||||||
|
method=frappe.sendmail,
|
||||||
|
queue="short",
|
||||||
|
timeout=300,
|
||||||
|
is_async=True,
|
||||||
|
enqueue_after_commit=True,
|
||||||
|
**email_args,
|
||||||
|
)
|
||||||
|
|
||||||
def get_message(self):
|
def get_message(self):
|
||||||
"""return message with payment gateway link"""
|
"""return message with payment gateway link"""
|
||||||
|
|||||||
Reference in New Issue
Block a user