Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0023344Openbravo ERPA. Platformpublic2013-03-19 17:452013-03-21 17:52
AugustoMauch 
AugustoMauch 
immediatemajorhave not tried
closedfixed 
5
 
3.0MP22 
AugustoMauch
Core
No
0023344: API Change - Remove Smart Sprites functionality
The Smart Sprites functionality improves the performance when lots of images have to be loaded in a window by aggregating them into a sprite, so that instead of using one connection for each image only one connection per sprite is needeed.

This issue [1] raised an inconsistency with the Google Collect library. There are two different versions of it in Openbravo, one in lib/build and another one in modules/org.openbravo.base.weld/lib/runtime/.

The one located is lib/build is very old and is only used by the Smart Sprites functionality. The one located in modules/org.openbravo.base.weld/lib/runtime/ is used by Weld and by some new coworking functionality. Smart Sprites is not compatible with the new version of the library because it depends on some deprecated code, and Weld and the coworking development do not work with the old version.

The only viable choice that we can think to fix this inconsistency is to delete the old library and to get rid of the Smart Sprites functionality.

These are some reasons why we should removing this feature is not risky:
- It has almost no value in Openbravo 3.0. If activated, this feature would only be used in the Login window and in the 2.50 windows. Even in those cases the performance is virtually the same.
- It is disabled by default which means that we were unclear about its value from the beginning, and that all users who have not change this property will not even notice this API change.

To remove this functionality the public method applySpriteTags of the RTLSkin class would have to be removed. This will be an API change, but only because the signature of the method is wrong in the first place, because it should have been a private method, because it only makes sense using it inside that class.

[1] https://issues.openbravo.com/view.php?id=23082 [^]
-
No tags attached.
related to defect 00230823.0MP22 closed AugustoMauch ANT compiling and module's libraries 
Issue History
2013-03-19 17:45AugustoMauchNew Issue
2013-03-19 17:45AugustoMauchAssigned To => iciordia
2013-03-19 17:45AugustoMauchModules => Core
2013-03-19 17:45AugustoMauchTriggers an Emergency Pack => No
2013-03-19 17:46AugustoMauchRelationship addedrelated to 0023082
2013-03-19 17:51AugustoMauchDescription Updatedbug_revision_view_page.php?rev_id=4520#r4520
2013-03-19 18:15iciordiaNote Added: 0057315
2013-03-19 18:15iciordiaAssigned Toiciordia => AugustoMauch
2013-03-21 17:50AugustoMauchStatusnew => scheduled
2013-03-21 17:50AugustoMauchfix_in_branch => pi
2013-03-21 17:50AugustoMauchStatusscheduled => resolved
2013-03-21 17:50AugustoMauchFixed in Version => 3.0MP22
2013-03-21 17:50AugustoMauchFixed in SCM revision => https://code.openbravo.com/erp/devel/pi/rev/47d740ad4b2c3d628f71608b61abb3277a5db740 [^]
2013-03-21 17:50AugustoMauchResolutionopen => fixed
2013-03-21 17:52AugustoMauchReview Assigned To => AugustoMauch
2013-03-21 17:52AugustoMauchNote Added: 0057375
2013-03-21 17:52AugustoMauchStatusresolved => closed

Notes
(0057315)
iciordia   
2013-03-19 18:15   
Approved.

Ismael
(0057375)
AugustoMauch   
2013-03-21 17:52   
try-api automated test has passed: http://builds.openbravo.com/view/try/job/try-api/1344/ [^]