Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0038371Openbravo ERPA. Platformpublic2018-04-15 11:142022-02-01 08:08
martinsdan 
Triage Platform Base 
normalminorN/A
newopen 
30Openbravo Appliance 14.04
 
 
Core
No
0038371: OperativeQuantity_To_BaseQuantity should be extensible
A hook should be added to this callout to allow it to be extended.

I'm creating a new window based on C_Order table, including lines.
Now, this hook doesn't work on my new window and I can't change it because it is defined at the table level. This is true for Orders, Invoices, and Shipments.

I know it is possible to extend the hook, but with that, I need to duplicate the logic, which is not desirable.
N/A
Attached proposed solution as a patch.
No tags attached.
diff 38371.diff (3,316) 2018-04-15 11:15
https://issues.openbravo.com/file_download.php?file_id=11734&type=bug
Issue History
2018-04-15 11:14martinsdanNew Issue
2018-04-15 11:14martinsdanAssigned To => platform
2018-04-15 11:14martinsdanModules => Core
2018-04-15 11:14martinsdanTriggers an Emergency Pack => No
2018-04-15 11:15martinsdanFile Added: 38371.diff
2018-04-15 11:17martinsdanNote Added: 0103892
2022-02-01 08:08alostaleAssigned Toplatform => Triage Platform Base

Notes
(0103892)
martinsdan   
2018-04-15 11:17   
I can't change the issue anymore.

The proposed solution is to add a hook to return the correct name of the field that needs to be changed by this callout. This way we can reuse the logic and even change the name of the field in new windows.