diff --git a/frappe/public/js/frappe/list/list_view.js b/frappe/public/js/frappe/list/list_view.js index 0104fd44c1..582aaa2ec7 100644 --- a/frappe/public/js/frappe/list/list_view.js +++ b/frappe/public/js/frappe/list/list_view.js @@ -1485,7 +1485,7 @@ frappe.views.ListView = class ListView extends frappe.views.BaseList { const doctype = this.doctype; const items = []; - if (frappe.model.can_import(this.meta)) { + if (frappe.model.can_import(doctype, null, this.meta)) { items.push({ label: __("Import"), action: () => diff --git a/frappe/public/js/frappe/model/model.js b/frappe/public/js/frappe/model/model.js index badd1859e3..041905408a 100644 --- a/frappe/public/js/frappe/model/model.js +++ b/frappe/public/js/frappe/model/model.js @@ -317,14 +317,14 @@ $.extend(frappe.model, { return doc && doc.__last_sync_on && ((new Date() - doc.__last_sync_on)) < 5000; }, - can_import: function(doctype, frm) { - if (!doctype.allow_import) return false; + can_import: function(doctype, frm, meta=null) { + if (meta && !meta.allow_import) return false; // system manager can always import if (frappe.user_roles.includes("System Manager")) return true; if (frm) return frm.perm[0].import===1; - return frappe.boot.user.can_import.indexOf(doctype.name)!==-1; + return frappe.boot.user.can_import.indexOf(doctype)!==-1; }, can_export: function(doctype, frm) {