vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Merge pull request #4110 from mbauskar/minor-fixes

Browse source 
[minor] don't show Bulk Print Option if user don't have Print permission
This commit is contained in:
Rushabh Mehta 2017-09-14 17:18:24 +05:30 • committed by GitHub
commit fca4ced23e
2 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
frappe/public/js/frappe/list/list_renderer.js
View file

@ -47,6 +47,9 @@ frappe.views.ListRenderer = Class.extend({
},
init_settings: function () {
this.settings = frappe.listview_settings[this.doctype] || {};
if(!("selectable" in this.settings)) {
this.settings.selectable = true;
}
this.init_user_settings();
this.order_by = this.user_settings.order_by || this.settings.order_by;

6
frappe/public/js/frappe/list/list_view.js
View file

@ -620,7 +620,9 @@ frappe.views.ListView = frappe.ui.BaseList.extend({
}
this.make_bulk_assignment();
this.make_bulk_printing();
if(frappe.model.can_print(this.doctype)) {
this.make_bulk_printing();
}
// add to desktop
this.page.add_menu_item(__('Add to Desktop'), function () {
@ -777,7 +779,7 @@ frappe.views.ListView = frappe.ui.BaseList.extend({
setup_delete: function () {
var me = this;
if (!(this.can_delete || this.list_renderer.settings.selectable)) {
if (!this.can_delete) {
return;
}
this.$page.on('change', '.list-row-checkbox, .list-select-all', function() {