Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0030488Openbravo ERPA. Platformpublic2015-08-04 09:212015-11-11 12:16
AugustoMauch 
platform 
normalminorhave not tried
closedsuspended 
5
 
 
Core
No
0030488: API Change: Replacement of LOWER with UPPER in queries
In order to solve this feature request [1], all instances of the LOWER function in SQL queries throughout the Openbravo codebase are going to be replaced with upper.

It is going to be done ensuring that the results returned by the queries does not change, but it is possible that some of those queries could take more time to execute for users that have defined function based indexes using the lower function, thus this change can be considered somewhat of an API change.

Note that Openbravo will not start giving support to function based indexes until PR15Q4, so if someone had defined those indexes, they would have to maintain them themselves (there was no way to export/import them)

[1] https://issues.openbravo.com/view.php?id=30476 [^]
-
Approved
related to feature request 0030476 new AugustoMauch Consolidate the user of UPPER/LOWER to achieve case insensitive filtering 
related to feature request 0030561 closed AugustoMauch Use UPPER to achieve case insensitive filtering in CustomQuerySelectorDatasource 
Issue History
2015-08-04 09:21AugustoMauchNew Issue
2015-08-04 09:21AugustoMauchAssigned To => dmitry_mezentsev
2015-08-04 09:21AugustoMauchModules => Core
2015-08-04 09:21AugustoMauchTriggers an Emergency Pack => No
2015-08-04 09:22AugustoMauchRelationship addedrelated to 0030476
2015-08-05 12:07dmitry_mezentsevTag Attached: Approved
2015-08-05 12:07dmitry_mezentsevAssigned Todmitry_mezentsev => AugustoMauch
2015-08-05 12:10dmitry_mezentsevNote Added: 0079198
2015-08-12 14:57AugustoMauchRelationship addedrelated to 0030561
2015-11-10 15:02alostaleStatusnew => acknowledged
2015-11-11 12:13alostaleAssigned ToAugustoMauch => platform
2015-11-11 12:16alostaleStatusacknowledged => scheduled
2015-11-11 12:16alostaleNote Added: 0081696
2015-11-11 12:16alostaleStatusscheduled => closed
2015-11-11 12:16alostaleResolutionopen => suspended

Notes
(0079198)
dmitry_mezentsev   
2015-08-05 12:10   
Augusto,
I think it also makes sense to communicate to staff.engineering not just only this API change but the rule to use UPPER with LOWER being "on special and clear request / demand".
IMHO we should think on a place to reflect the same in the Dev. Guide even.
(0081696)
alostale   
2015-11-11 12:16   
There is no plan for the moment to do this change