From da91cfe1e23eed929e88ae5100d0ffc7c06b7988 Mon Sep 17 00:00:00 2001 From: Himanshu Warekar Date: Tue, 28 May 2019 23:38:43 +0530 Subject: [PATCH] fix: set default priority on sla change --- erpnext/support/doctype/issue/issue.js | 1 - erpnext/support/doctype/issue/issue.py | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/erpnext/support/doctype/issue/issue.js b/erpnext/support/doctype/issue/issue.js index af742acc5fe..44b605d91bc 100644 --- a/erpnext/support/doctype/issue/issue.js +++ b/erpnext/support/doctype/issue/issue.js @@ -67,7 +67,6 @@ frappe.ui.form.on("Issue", { message: __('Changing Service Level Agreement.') }); frm.call('change_service_level_agreement_and_priority', { - "priority": frm.doc.priority, "service_level_agreement": frm.doc.service_level_agreement }).then(() => { frappe.msgprint(__("Service Level Agreement changed to {0}.", [frm.doc.service_level_agreement])); diff --git a/erpnext/support/doctype/issue/issue.py b/erpnext/support/doctype/issue/issue.py index b47f7108bc5..734b4433ba7 100644 --- a/erpnext/support/doctype/issue/issue.py +++ b/erpnext/support/doctype/issue/issue.py @@ -163,7 +163,7 @@ class Issue(Document): self.resolution_by_variance = round(time_diff_in_hours(self.resolution_by, now_datetime())) @frappe.whitelist() - def change_service_level_agreement_and_priority(self, priority, service_level_agreement): + def change_service_level_agreement_and_priority(self, priority=None, service_level_agreement=None): self.set_response_and_resolution_time(priority=priority, service_level_agreement=service_level_agreement) self.save(ignore_permissions=True)