From a302393bc2abdf5cdba6b0bab5455df39e321a99 Mon Sep 17 00:00:00 2001 From: Rohit Waghchaure Date: Thu, 6 May 2021 23:23:21 +0530 Subject: [PATCH] fix: show delete button on portal if user has permission to delete document --- frappe/public/js/frappe/web_form/web_form.js | 12 +++++++----- frappe/public/js/frappe/web_form/web_form_list.js | 8 +++++--- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/frappe/public/js/frappe/web_form/web_form.js b/frappe/public/js/frappe/web_form/web_form.js index c490c5c66c..70353de170 100644 --- a/frappe/public/js/frappe/web_form/web_form.js +++ b/frappe/public/js/frappe/web_form/web_form.js @@ -87,11 +87,13 @@ export default class WebForm extends frappe.ui.FieldGroup { } setup_delete_button() { - this.add_button_to_header( - frappe.utils.icon('delete'), - "danger", - () => this.delete() - ); + frappe.has_permission(this.doc_type, "", "delete", () => { + this.add_button_to_header( + frappe.utils.icon('delete'), + "danger", + () => this.delete() + ); + }); } setup_print_button() { diff --git a/frappe/public/js/frappe/web_form/web_form_list.js b/frappe/public/js/frappe/web_form/web_form_list.js index 21eed52e5e..de45b3ac11 100644 --- a/frappe/public/js/frappe/web_form/web_form_list.js +++ b/frappe/public/js/frappe/web_form/web_form_list.js @@ -190,9 +190,11 @@ export default class WebFormList { make_actions() { const actions = document.querySelector(".list-view-actions"); - this.addButton(actions, "delete-rows", "danger", true, "Delete", () => - this.delete_rows() - ); + frappe.has_permission(this.doctype, "", "delete", () => { + this.addButton(actions, "delete-rows", "danger", true, "Delete", () => + this.delete_rows() + ); + }); this.addButton( actions,