Libero QA: PP_Order_Transactions view : never use ORDER BY clause in a view; it's a bad practice
This commit is contained in:
parent
2f9e8fac78
commit
9227c53c4f
|
@ -29,8 +29,4 @@ FROM pp_order o
|
|||
JOIN pp_order_bomline ol ON ol.pp_order_id = o.pp_order_id
|
||||
JOIN pp_cost_collector cc ON cc.pp_order_bomline_id = ol.pp_order_bomline_id
|
||||
LEFT JOIN m_transaction mt ON mt.pp_cost_collector_id = cc.pp_cost_collector_id
|
||||
ORDER BY
|
||||
o.ad_client_id, o.ad_org_id, o.isactive, o.created, o.createdby, o.updatedby, o.updated, o.documentno, ol.m_product_id, mt.m_locator_id, mt.movementdate, o.pp_order_id, o.qtydelivered, o.qtyscrap, ol.qtydelivered, o.qtydelivered * ol.qtybatch / 100, ol.qtyscrap, o.qtyscrap * ol.qtybatch / 100, mt.createdby, mt.updatedby,
|
||||
(SELECT sum(t.movementqty) AS sum FROM m_transaction t WHERE t.pp_cost_collector_id = cc.pp_cost_collector_id), (o.qtydelivered + o.qtyscrap) * ol.qtybatch / 100 + (( SELECT sum(t.movementqty) AS sum FROM m_transaction t WHERE t.pp_cost_collector_id = cc.pp_cost_collector_id)),
|
||||
o.issotrx,
|
||||
o.dateordered;
|
||||
;
|
||||
|
|
|
@ -29,8 +29,4 @@ FROM pp_order o
|
|||
JOIN pp_order_bomline ol ON ol.pp_order_id = o.pp_order_id
|
||||
JOIN pp_cost_collector cc ON cc.pp_order_bomline_id = ol.pp_order_bomline_id
|
||||
LEFT JOIN m_transaction mt ON mt.pp_cost_collector_id = cc.pp_cost_collector_id
|
||||
ORDER BY
|
||||
o.ad_client_id, o.ad_org_id, o.isactive, o.created, o.createdby, o.updatedby, o.updated, o.documentno, ol.m_product_id, mt.m_locator_id, mt.movementdate, o.pp_order_id, o.qtydelivered, o.qtyscrap, ol.qtydelivered, o.qtydelivered * ol.qtybatch / 100, ol.qtyscrap, o.qtyscrap * ol.qtybatch / 100, mt.createdby, mt.updatedby,
|
||||
(SELECT sum(t.movementqty) AS sum FROM m_transaction t WHERE t.pp_cost_collector_id = cc.pp_cost_collector_id), (o.qtydelivered + o.qtyscrap) * ol.qtybatch / 100 + (( SELECT sum(t.movementqty) AS sum FROM m_transaction t WHERE t.pp_cost_collector_id = cc.pp_cost_collector_id)),
|
||||
o.issotrx,
|
||||
o.dateordered;
|
||||
;
|
||||
|
|
|
@ -93,11 +93,7 @@ FROM pp_order o
|
|||
JOIN pp_order_bomline ol ON ol.pp_order_id = o.pp_order_id
|
||||
JOIN pp_cost_collector cc ON cc.pp_order_bomline_id = ol.pp_order_bomline_id
|
||||
LEFT JOIN m_transaction mt ON mt.pp_cost_collector_id = cc.pp_cost_collector_id
|
||||
ORDER BY
|
||||
o.ad_client_id, o.ad_org_id, o.isactive, o.created, o.createdby, o.updatedby, o.updated, o.documentno, ol.m_product_id, mt.m_locator_id, mt.movementdate, o.pp_order_id, o.qtydelivered, o.qtyscrap, ol.qtydelivered, o.qtydelivered * ol.qtybatch / 100, ol.qtyscrap, o.qtyscrap * ol.qtybatch / 100, mt.createdby, mt.updatedby,
|
||||
(SELECT sum(t.movementqty) AS sum FROM m_transaction t WHERE t.pp_cost_collector_id = cc.pp_cost_collector_id), (o.qtydelivered + o.qtyscrap) * ol.qtybatch / 100 + (( SELECT sum(t.movementqty) AS sum FROM m_transaction t WHERE t.pp_cost_collector_id = cc.pp_cost_collector_id)),
|
||||
o.issotrx,
|
||||
o.dateordered;
|
||||
;
|
||||
|
||||
ALTER TABLE m_transaction DROP COLUMN pp_order_id;
|
||||
ALTER TABLE m_transaction DROP COLUMN pp_order_bomline_id;
|
|
@ -96,11 +96,7 @@ FROM pp_order o
|
|||
JOIN pp_order_bomline ol ON ol.pp_order_id = o.pp_order_id
|
||||
JOIN pp_cost_collector cc ON cc.pp_order_bomline_id = ol.pp_order_bomline_id
|
||||
LEFT JOIN m_transaction mt ON mt.pp_cost_collector_id = cc.pp_cost_collector_id
|
||||
ORDER BY
|
||||
o.ad_client_id, o.ad_org_id, o.isactive, o.created, o.createdby, o.updatedby, o.updated, o.documentno, ol.m_product_id, mt.m_locator_id, mt.movementdate, o.pp_order_id, o.qtydelivered, o.qtyscrap, ol.qtydelivered, o.qtydelivered * ol.qtybatch / 100, ol.qtyscrap, o.qtyscrap * ol.qtybatch / 100, mt.createdby, mt.updatedby,
|
||||
(SELECT sum(t.movementqty) AS sum FROM m_transaction t WHERE t.pp_cost_collector_id = cc.pp_cost_collector_id), (o.qtydelivered + o.qtyscrap) * ol.qtybatch / 100 + (( SELECT sum(t.movementqty) AS sum FROM m_transaction t WHERE t.pp_cost_collector_id = cc.pp_cost_collector_id)),
|
||||
o.issotrx,
|
||||
o.dateordered;
|
||||
;
|
||||
|
||||
ALTER TABLE m_transaction DROP COLUMN pp_order_id;
|
||||
ALTER TABLE m_transaction DROP COLUMN pp_order_bomline_id;
|
Loading…
Reference in New Issue