Merge pull request #32324 from s-aga-r/refactor/report/production-planning-report

refactor: rewrite `Production Planning Report` queries in `QB`
This commit is contained in:
Sagar Sharma 2022-09-22 18:12:52 +05:30 committed by GitHub
commit 4efb8b142c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -4,42 +4,10 @@
import frappe import frappe
from frappe import _ from frappe import _
from pypika import Order
from erpnext.stock.doctype.warehouse.warehouse import get_child_warehouses from erpnext.stock.doctype.warehouse.warehouse import get_child_warehouses
# and bom_no is not null and bom_no !=''
mapper = {
"Sales Order": {
"fields": """ item_code as production_item, item_name as production_item_name, stock_uom,
stock_qty as qty_to_manufacture, `tabSales Order Item`.parent as name, bom_no, warehouse,
`tabSales Order Item`.delivery_date, `tabSales Order`.base_grand_total """,
"filters": """`tabSales Order Item`.docstatus = 1 and stock_qty > produced_qty
and `tabSales Order`.per_delivered < 100.0""",
},
"Material Request": {
"fields": """ item_code as production_item, item_name as production_item_name, stock_uom,
stock_qty as qty_to_manufacture, `tabMaterial Request Item`.parent as name, bom_no, warehouse,
`tabMaterial Request Item`.schedule_date """,
"filters": """`tabMaterial Request`.docstatus = 1 and `tabMaterial Request`.per_ordered < 100
and `tabMaterial Request`.material_request_type = 'Manufacture' """,
},
"Work Order": {
"fields": """ production_item, item_name as production_item_name, planned_start_date,
stock_uom, qty as qty_to_manufacture, name, bom_no, fg_warehouse as warehouse """,
"filters": "docstatus = 1 and status not in ('Completed', 'Stopped')",
},
}
order_mapper = {
"Sales Order": {
"Delivery Date": "`tabSales Order Item`.delivery_date asc",
"Total Amount": "`tabSales Order`.base_grand_total desc",
},
"Material Request": {"Required Date": "`tabMaterial Request Item`.schedule_date asc"},
"Work Order": {"Planned Start Date": "planned_start_date asc"},
}
def execute(filters=None): def execute(filters=None):
return ProductionPlanReport(filters).execute_report() return ProductionPlanReport(filters).execute_report()
@ -63,40 +31,78 @@ class ProductionPlanReport(object):
return self.columns, self.data return self.columns, self.data
def get_open_orders(self): def get_open_orders(self):
doctype = ( doctype, order_by = self.filters.based_on, self.filters.order_by
"`tabWork Order`"
if self.filters.based_on == "Work Order"
else "`tab{doc}`, `tab{doc} Item`".format(doc=self.filters.based_on)
)
filters = mapper.get(self.filters.based_on)["filters"] parent = frappe.qb.DocType(doctype)
filters = self.prepare_other_conditions(filters, self.filters.based_on) query = None
order_by = " ORDER BY %s" % (order_mapper[self.filters.based_on][self.filters.order_by])
self.orders = frappe.db.sql( if doctype == "Work Order":
""" SELECT {fields} from {doctype} query = (
WHERE {filters} {order_by}""".format( frappe.qb.from_(parent)
doctype=doctype, .select(
filters=filters, parent.production_item,
order_by=order_by, parent.item_name.as_("production_item_name"),
fields=mapper.get(self.filters.based_on)["fields"], parent.planned_start_date,
), parent.stock_uom,
tuple(self.filters.docnames), parent.qty.as_("qty_to_manufacture"),
as_dict=1, parent.name,
) parent.bom_no,
parent.fg_warehouse.as_("warehouse"),
)
.where(parent.status.notin(["Completed", "Stopped"]))
)
def prepare_other_conditions(self, filters, doctype): if order_by == "Planned Start Date":
if self.filters.docnames: query = query.orderby(parent.planned_start_date, order=Order.asc)
field = "name" if doctype == "Work Order" else "`tab{} Item`.parent".format(doctype)
filters += " and %s in (%s)" % (field, ",".join(["%s"] * len(self.filters.docnames)))
if doctype != "Work Order": if self.filters.docnames:
filters += " and `tab{doc}`.name = `tab{doc} Item`.parent".format(doc=doctype) query = query.where(parent.name.isin(self.filters.docnames))
else:
child = frappe.qb.DocType(f"{doctype} Item")
query = (
frappe.qb.from_(parent)
.from_(child)
.select(
child.bom_no,
child.stock_uom,
child.warehouse,
child.parent.as_("name"),
child.item_code.as_("production_item"),
child.stock_qty.as_("qty_to_manufacture"),
child.item_name.as_("production_item_name"),
)
.where(parent.name == child.parent)
)
if self.filters.docnames:
query = query.where(child.parent.isin(self.filters.docnames))
if doctype == "Sales Order":
query = query.select(
child.delivery_date,
parent.base_grand_total,
).where((child.stock_qty > child.produced_qty) & (parent.per_delivered < 100.0))
if order_by == "Delivery Date":
query = query.orderby(child.delivery_date, order=Order.asc)
elif order_by == "Total Amount":
query = query.orderby(parent.base_grand_total, order=Order.desc)
elif doctype == "Material Request":
query = query.select(child.schedule_date,).where(
(parent.per_ordered < 100) & (parent.material_request_type == "Manufacture")
)
if order_by == "Required Date":
query = query.orderby(child.schedule_date, order=Order.asc)
query = query.where(parent.docstatus == 1)
if self.filters.company: if self.filters.company:
filters += " and `tab%s`.company = %s" % (doctype, frappe.db.escape(self.filters.company)) query = query.where(parent.company == self.filters.company)
return filters self.orders = query.run(as_dict=True)
def get_raw_materials(self): def get_raw_materials(self):
if not self.orders: if not self.orders:
@ -134,29 +140,29 @@ class ProductionPlanReport(object):
bom_nos.append(bom_no) bom_nos.append(bom_no)
bom_doctype = ( bom_item_doctype = (
"BOM Explosion Item" if self.filters.include_subassembly_raw_materials else "BOM Item" "BOM Explosion Item" if self.filters.include_subassembly_raw_materials else "BOM Item"
) )
qty_field = ( bom = frappe.qb.DocType("BOM")
"qty_consumed_per_unit" bom_item = frappe.qb.DocType(bom_item_doctype)
if self.filters.include_subassembly_raw_materials
else "(bom_item.qty / bom.quantity)"
)
raw_materials = frappe.db.sql( if self.filters.include_subassembly_raw_materials:
""" SELECT bom_item.parent, bom_item.item_code, qty_field = bom_item.qty_consumed_per_unit
bom_item.item_name as raw_material_name, {0} as required_qty_per_unit else:
FROM qty_field = bom_item.qty / bom.quantity
`tabBOM` as bom, `tab{1}` as bom_item
WHERE raw_materials = (
bom_item.parent in ({2}) and bom_item.parent = bom.name and bom.docstatus = 1 frappe.qb.from_(bom)
""".format( .from_(bom_item)
qty_field, bom_doctype, ",".join(["%s"] * len(bom_nos)) .select(
), bom_item.parent,
tuple(bom_nos), bom_item.item_code,
as_dict=1, bom_item.item_name.as_("raw_material_name"),
) qty_field.as_("required_qty_per_unit"),
)
.where((bom_item.parent.isin(bom_nos)) & (bom_item.parent == bom.name) & (bom.docstatus == 1))
).run(as_dict=True)
if not raw_materials: if not raw_materials:
return return