From 83fe747f755c369245e7f5eb2b2d633944d094e4 Mon Sep 17 00:00:00 2001 From: phot0n Date: Thu, 7 Apr 2022 00:03:42 +0530 Subject: [PATCH] chore: remove payment utils and hooks --- frappe/hooks.py | 3 +- frappe/integrations/utils.py | 108 ----------------------------------- 2 files changed, 1 insertion(+), 110 deletions(-) diff --git a/frappe/hooks.py b/frappe/hooks.py index 66820ecd0f..90c64b006f 100644 --- a/frappe/hooks.py +++ b/frappe/hooks.py @@ -199,8 +199,7 @@ scheduler_events = { "frappe.email.queue.flush", "frappe.email.doctype.email_account.email_account.pull", "frappe.email.doctype.email_account.email_account.notify_unreplied", - "frappe.integrations.doctype.razorpay_settings.razorpay_settings.capture_payment", - "frappe.utils.global_search.sync_global_search", + 'frappe.utils.global_search.sync_global_search', "frappe.monitor.flush", ], "hourly": [ diff --git a/frappe/integrations/utils.py b/frappe/integrations/utils.py index f215a73dc6..8d3d23bb85 100644 --- a/frappe/integrations/utils.py +++ b/frappe/integrations/utils.py @@ -2,7 +2,6 @@ # License: MIT. See LICENSE import datetime -import json from urllib.parse import parse_qs import frappe @@ -28,122 +27,15 @@ def make_request(method, url, auth=None, headers=None, data=None): frappe.log_error() raise exc - def make_get_request(url, **kwargs): return make_request("GET", url, **kwargs) - def make_post_request(url, **kwargs): return make_request("POST", url, **kwargs) - def make_put_request(url, **kwargs): return make_request("PUT", url, **kwargs) - -def create_request_log( - data, - integration_type=None, - service_name=None, - name=None, - error=None, - request_headers=None, - output=None, - **kwargs, -): - """ - DEPRECATED: The parameter integration_type will be removed in the next major release. - Use is_remote_request instead. - """ - if integration_type == "Remote": - kwargs["is_remote_request"] = 1 - - elif integration_type == "Subscription Notification": - kwargs["request_description"] = integration_type - - reference_doctype = reference_docname = None - if "reference_doctype" not in kwargs: - if isinstance(data, str): - data = json.loads(data) - - reference_doctype = data.get("reference_doctype") - reference_docname = data.get("reference_docname") - - integration_request = frappe.get_doc( - { - "doctype": "Integration Request", - "integration_request_service": service_name, - "request_headers": get_json(request_headers), - "data": get_json(data), - "output": get_json(output), - "error": get_json(error), - "reference_doctype": reference_doctype, - "reference_docname": reference_docname, - **kwargs, - } - ) - - if name: - integration_request.flags._name = name - - integration_request.insert(ignore_permissions=True) - frappe.db.commit() - - return integration_request - - -def get_json(obj): - return obj if isinstance(obj, str) else frappe.as_json(obj, indent=1) - - -def get_payment_gateway_controller(payment_gateway): - """Return payment gateway controller""" - gateway = frappe.get_doc("Payment Gateway", payment_gateway) - if gateway.gateway_controller is None: - try: - return frappe.get_doc(f"{payment_gateway} Settings") - except Exception: - frappe.throw(_("{0} Settings not found").format(payment_gateway)) - else: - try: - return frappe.get_doc(gateway.gateway_settings, gateway.gateway_controller) - except Exception: - frappe.throw(_("{0} Settings not found").format(payment_gateway)) - - -@frappe.whitelist(allow_guest=True, xss_safe=True) -def get_checkout_url(**kwargs): - try: - if kwargs.get("payment_gateway"): - doc = frappe.get_doc("{} Settings".format(kwargs.get("payment_gateway"))) - return doc.get_payment_url(**kwargs) - else: - raise Exception - except Exception: - frappe.respond_as_web_page( - _("Something went wrong"), - _( - "Looks like something is wrong with this site's payment gateway configuration. No payment has been made." - ), - indicator_color="red", - http_status_code=frappe.ValidationError.http_status_code, - ) - - -def create_payment_gateway(gateway, settings=None, controller=None): - # NOTE: we don't translate Payment Gateway name because it is an internal doctype - if not frappe.db.exists("Payment Gateway", gateway): - payment_gateway = frappe.get_doc( - { - "doctype": "Payment Gateway", - "gateway": gateway, - "gateway_settings": settings, - "gateway_controller": controller, - } - ) - payment_gateway.insert(ignore_permissions=True) - - def json_handler(obj): if isinstance(obj, (datetime.date, datetime.timedelta, datetime.datetime)): return str(obj)