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

Merge pull request #1626 from nabinhait/print_permission

Browse source 
[fix] Check permissions on report print/pdf
This commit is contained in:
Nabin Hait 2016-03-15 12:43:14 +05:30
commit 2cd0abd64a
1 changed files with 10 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
frappe/public/js/frappe/views/reports/query_report.js
View file

@ -139,6 +139,11 @@ frappe.views.QueryReport = Class.extend({
}
},
print_report: function() {
if(!frappe.model.can_print(this.report_doc.ref_doctype)) {
msgprint(__("You are not allowed to print this report"));
return false;
}
if(this.html_format) {
var content = frappe.render(this.html_format,
{data: this.dataView.getItems(), filters:this.get_values(), report:this});
@ -149,6 +154,11 @@ frappe.views.QueryReport = Class.extend({
}
},
pdf_report: function() {
if(!frappe.model.can_print(this.report_doc.ref_doctype)) {
msgprint(__("You are not allowed to make PDF for this report"));
return false;
}
if(this.html_format) {
var content = frappe.render(this.html_format,
{data: this.dataView.getItems(), filters:this.get_values(), report:this});