Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0040823 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
feature request | [Openbravo ERP] Z. Others | minor | have not tried | 2019-05-09 13:42 | 2019-12-18 12:08 | |||
Reporter | vmromanos | View Status | public | |||||
Assigned To | AtulOpenbravo | |||||||
Priority | low | Resolution | fixed | Fixed in Version | 3.0PR19Q4 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 2e1448b075ab | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | vmromanos | |||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0040823: Delete obsolete db functions that have been deprecated long time ago | |||||||
Description | Here is the list of db functions that could be removed: m_get_offer_price m_get_offer_std_price m_get_offers_price m_get_offers_std_price Verify first whether they are not used anywhere (also in external modules), and remove them if safe. | |||||||
Steps To Reproduce | NA | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
||||||||
|
![]() |
|
(0113601) AtulOpenbravo (viewer) 2019-07-23 19:22 |
m_get_pareto_abc_legacy is related to old costing hence will be a part of old costing cleanup issue https://issues.openbravo.com/view.php?id=41421 [^] Skipped removal of below candidates as the Warehouse rules include the procedure name, which is retrieved by the M_GET_STOCK_PARAM that uses it to launch the rule (aka the procedure). m_warehouse_default_rule m_warehouse_default_rule_param m_get_pareto_abc is also skipped from removal as it is being used in Pareto Product Report. |
(0113623) hgbot (developer) 2019-07-24 17:08 |
Repository: erp/devel/pi Changeset: 2e1448b075abe41f4df301be1cbc24489d0d7267 Author: Atul Gaware <atul.gaware <at> openbravo.com> Date: Tue Jul 16 22:44:29 2019 +0530 URL: http://code.openbravo.com/erp/devel/pi/rev/2e1448b075abe41f4df301be1cbc24489d0d7267 [^] Fixes Issue 0040823: Delete obsolete db functions that have been deprecated long time ago **Removed functions M_GET_OFFER_PRICE, M_GET_OFFER_STD_PRICE, M_GET_OFFERS_PRICE, M_GET_OFFERS_STD_PRICE **Removed references of above removed db functions from other functions/methods. --- M src-db/database/model/functions/C_CREATE_PINVOICE_FROM_OUTS.xml M src-db/database/model/functions/C_GENERATEPOFROMPROJECT.xml M src-db/database/model/functions/C_GENERATESOFROMPROJECT.xml M src-db/database/model/functions/C_GENERATESOFROMPROJECTORDER.xml M src-db/database/model/functions/C_GENERATESOFROMPROJECTPHASE.xml M src-db/database/model/functions/C_PROJECT_GENERATE.xml M src-db/database/model/functions/MRP_PURCHASEORDER.xml M src-db/database/model/functions/M_COPY_PRODUCT_TEMPLATE.xml M src-db/database/model/functions/M_EXPLODEBOMNOTSTOCK.xml M src-db/database/model/functions/M_INOUT_CREATEINVOICE.xml M src-db/database/model/functions/M_INVEXPLODEBOMNOTSTOCK.xml M src-db/database/model/functions/M_REQUISITION_CREATEPO.xml M src/org/openbravo/common/actionhandler/createlinesfromprocess/UpdatePricesAndAmounts.java M src/org/openbravo/erpCommon/ad_actionButton/CopyFromInvoice.java M src/org/openbravo/erpCommon/ad_actionButton/CopyFromInvoice_data.xsql M src/org/openbravo/erpCommon/ad_actionButton/CopyFromOrder.java M src/org/openbravo/erpCommon/ad_actionButton/CopyFromOrder_data.xsql M src/org/openbravo/erpCommon/ad_actionButton/ExpenseSOrder.java M src/org/openbravo/erpCommon/ad_actionButton/ExpenseSOrder_data.xsql M src/org/openbravo/erpCommon/ad_forms/RequisitionToOrder.java M src/org/openbravo/erpCommon/ad_forms/RequisitionToOrder_data.xsql M src/org/openbravo/erpCommon/businessUtility/GetPriceOffer_data.xsql R src-db/database/model/functions/M_GET_OFFERS_PRICE.xml R src-db/database/model/functions/M_GET_OFFERS_STD_PRICE.xml R src-db/database/model/functions/M_GET_OFFER_PRICE.xml R src-db/database/model/functions/M_GET_OFFER_STD_PRICE.xml --- |
(0113646) hgbot (developer) 2019-07-25 11:09 |
Repository: erp/pmods/org.openbravo.document.massinvoicing Changeset: db13da46c3c477e15e7b24dd4f1bc4c8459f8a1e Author: Atul Gaware <atul.gaware <at> openbravo.com> Date: Thu Jul 25 14:27:19 2019 +0530 URL: http://code.openbravo.com/erp/pmods/org.openbravo.document.massinvoicing/rev/db13da46c3c477e15e7b24dd4f1bc4c8459f8a1e [^] Related To Issue 40823:Delete obsolete db functions that have been deprecated long time ago ** During the api change signature of the method GetPriceOfferData.getOffersPriceCurrency is changed hence the updated sent parameter in the classes using it. ** Remove unused variables. --- M src/org/openbravo/document/massinvoicing/InvoicePropertySetter.java M src/org/openbravo/document/massinvoicing/MassInvoicing.java --- |
(0113670) vmromanos (viewer) 2019-07-26 09:41 |
Code review OK |
(0114175) hudsonbot (viewer) 2019-08-22 14:45 |
A changeset related to this issue has been promoted main and to the Central Repository, after passing a series of tests. Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/ad3efd3bd07c [^] Maturity status: Test |
(0116399) hgbot (developer) 2019-12-18 12:08 |
Repository: erp/backports/3.0PR19Q3.3 Changeset: f2bf2a6fb19585bcea6b02a8e123db0ee8b55621 Author: David Miguelez <david.miguelez <at> openbravo.com> Date: Wed Dec 18 11:23:08 2019 +0100 URL: http://code.openbravo.com/erp/backports/3.0PR19Q3.3/rev/f2bf2a6fb19585bcea6b02a8e123db0ee8b55621 [^] Related to Issue 42574. Partial fix of Issue 40823 must be included in order to be compatible with the changes done to solve the proper issue. The changes included are to avoid using functions that are deprecated. --- M src-db/database/model/functions/M_EXPLODEBOMNOTSTOCK.xml M src-db/database/model/functions/M_INVEXPLODEBOMNOTSTOCK.xml --- |
![]() |
|||
Date Modified | Username | Field | Change |
2019-05-09 13:42 | vmromanos | New Issue | |
2019-05-09 13:42 | vmromanos | Assigned To | => Triage Finance |
2019-05-09 13:42 | vmromanos | OBNetwork customer | => No |
2019-05-09 13:42 | vmromanos | Modules | => Core |
2019-05-09 13:42 | vmromanos | Triggers an Emergency Pack | => No |
2019-06-27 19:23 | AtulOpenbravo | Status | new => scheduled |
2019-07-16 14:16 | AtulOpenbravo | Relationship added | depends on 0041334 |
2019-07-22 14:03 | vmromanos | Assigned To | Triage Finance => AtulOpenbravo |
2019-07-23 19:19 | AtulOpenbravo | Description Updated | View Revisions |
2019-07-23 19:22 | AtulOpenbravo | Note Added: 0113601 | |
2019-07-24 17:08 | hgbot | Checkin | |
2019-07-24 17:08 | hgbot | Note Added: 0113623 | |
2019-07-24 17:08 | hgbot | Status | scheduled => resolved |
2019-07-24 17:08 | hgbot | Resolution | open => fixed |
2019-07-24 17:08 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/2e1448b075abe41f4df301be1cbc24489d0d7267 [^] |
2019-07-25 11:09 | hgbot | Checkin | |
2019-07-25 11:09 | hgbot | Note Added: 0113646 | |
2019-07-26 09:41 | vmromanos | Review Assigned To | => vmromanos |
2019-07-26 09:41 | vmromanos | Note Added: 0113670 | |
2019-07-26 09:41 | vmromanos | Status | resolved => closed |
2019-07-26 09:41 | vmromanos | Fixed in Version | => 3.0PR19Q4 |
2019-08-22 14:45 | hudsonbot | Checkin | |
2019-08-22 14:45 | hudsonbot | Note Added: 0114175 | |
2019-12-18 12:08 | hgbot | Checkin | |
2019-12-18 12:08 | hgbot | Note Added: 0116399 |
Copyright © 2000 - 2009 MantisBT Group |