mirror of
https://github.com/frappe/erpnext.git
synced 2026-05-29 18:04:46 +00:00
fix: make response handling more descriptive
This commit is contained in:
@@ -186,7 +186,7 @@ class PaymentRequest(Document):
|
|||||||
party_amount = self.grand_total
|
party_amount = self.grand_total
|
||||||
|
|
||||||
payment_entry = get_payment_entry(self.reference_doctype, self.reference_name, party_amount=party_amount,
|
payment_entry = get_payment_entry(self.reference_doctype, self.reference_name, party_amount=party_amount,
|
||||||
bank_account=self.payment_account, bank_amount=bank_amount, mode_of_payment=self.mode_of_payment)
|
bank_account=self.payment_account, bank_amount=bank_amount)
|
||||||
|
|
||||||
payment_entry.update({
|
payment_entry.update({
|
||||||
"reference_no": self.name,
|
"reference_no": self.name,
|
||||||
|
|||||||
@@ -109,13 +109,13 @@ def verify_transaction(**kwargs):
|
|||||||
item_response = transaction_response["CallbackMetadata"]["Item"]
|
item_response = transaction_response["CallbackMetadata"]["Item"]
|
||||||
mpesa_receipt = fetch_param_value(item_response, "MpesaReceiptNumber", "Name")
|
mpesa_receipt = fetch_param_value(item_response, "MpesaReceiptNumber", "Name")
|
||||||
frappe.db.set_value("POS Invoice", doc.reference_name, "mpesa_receipt_number", mpesa_receipt)
|
frappe.db.set_value("POS Invoice", doc.reference_name, "mpesa_receipt_number", mpesa_receipt)
|
||||||
request.process_response('output', transaction_response)
|
request.handle_success(transaction_response)
|
||||||
except Exception:
|
except Exception:
|
||||||
request.process_response('error', transaction_response)
|
request.handle_failure(transaction_response)
|
||||||
frappe.log_error(frappe.get_traceback())
|
frappe.log_error(frappe.get_traceback())
|
||||||
|
|
||||||
else:
|
else:
|
||||||
request.process_response('error', transaction_response)
|
request.handle_failure(transaction_response)
|
||||||
|
|
||||||
frappe.publish_realtime('process_phone_payment', doctype="POS Invoice",
|
frappe.publish_realtime('process_phone_payment', doctype="POS Invoice",
|
||||||
docname=transaction_data.payment_reference, user=request.owner, message=transaction_response)
|
docname=transaction_data.payment_reference, user=request.owner, message=transaction_response)
|
||||||
@@ -162,12 +162,12 @@ def process_balance_info(**kwargs):
|
|||||||
ref_doc = frappe.get_doc(transaction_data.reference_doctype, transaction_data.reference_docname)
|
ref_doc = frappe.get_doc(transaction_data.reference_doctype, transaction_data.reference_docname)
|
||||||
ref_doc.db_set("account_balance", balance_info)
|
ref_doc.db_set("account_balance", balance_info)
|
||||||
|
|
||||||
request.process_response('output', account_balance_response)
|
request.handle_success(account_balance_response)
|
||||||
except:
|
except:
|
||||||
request.process_response('error', account_balance_response)
|
request.handle_failure(account_balance_response)
|
||||||
frappe.log_error(title=_("Mpesa Account Balance Processing Error"), message=account_balance_response)
|
frappe.log_error(title=_("Mpesa Account Balance Processing Error"), message=account_balance_response)
|
||||||
else:
|
else:
|
||||||
request.process_response('error', account_balance_response)
|
request.handle_failure(account_balance_response)
|
||||||
|
|
||||||
def convert_to_json(balance_info):
|
def convert_to_json(balance_info):
|
||||||
"""
|
"""
|
||||||
|
|||||||
Reference in New Issue
Block a user