schuetz3/schuetz3-erpnext
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: Permission issue in Total Stock Summary report (#19103)

Browse Source 
Data in "Total Stock Summary" report were not getting filtered
based on applied user permissions because some link fields had
wrong options
This commit is contained in:
Suraj Shetty
2019-09-18 20:01:54 +05:30
committed by Anurag Mishra
parent 78776c42fe
commit a9a7e84873
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
erpnext/stock/report/total_stock_summary/total_stock_summary.py
View File

@@ -15,8 +15,8 @@ def execute(filters=None):
def get_columns(): def get_columns():
columns = [ columns = [
_("Company") + ":Link/Item:250", _("Company") + ":Link/Company:250",
_("Warehouse") + ":Link/Item:150", _("Warehouse") + ":Link/Warehouse:150",
_("Item") + ":Link/Item:150", _("Item") + ":Link/Item:150",
_("Description") + "::300", _("Description") + "::300",
_("Current Qty") + ":Float:100", _("Current Qty") + ":Float:100",
@@ -31,7 +31,7 @@ def get_total_stock(filters):
if filters.get("group_by") == "Warehouse": if filters.get("group_by") == "Warehouse":
if filters.get("company"): if filters.get("company"):
conditions += " AND warehouse.company = '%s'" % frappe.db.escape(filters.get("company"), percent=False) conditions += " AND warehouse.company = '%s'" % frappe.db.escape(filters.get("company"), percent=False)
conditions += " GROUP BY ledger.warehouse, item.item_code" conditions += " GROUP BY ledger.warehouse, item.item_code"
columns += "'' as company, ledger.warehouse" columns += "'' as company, ledger.warehouse"
else: else: