Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||||||
ID | ||||||||||||
0046244 | ||||||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||||||
defect | [Retail Modules] Web POS | minor | always | 2021-04-10 08:10 | 2021-04-12 09:57 | |||||||
Reporter | jetxarri | View Status | public | |||||||||
Assigned To | Retail | |||||||||||
Priority | normal | Resolution | open | Fixed in Version | ||||||||
Status | new | Fix in branch | Fixed in SCM revision | |||||||||
Projection | none | ETA | none | Target Version | ||||||||
OS | Any | Database | Any | Java version | ||||||||
OS Version | Database version | Ant version | ||||||||||
Product Version | RR21Q1.1 | SCM revision | ||||||||||
Review Assigned To | ||||||||||||
Regression level | ||||||||||||
Regression date | ||||||||||||
Regression introduced in release | ||||||||||||
Regression introduced by commit | ||||||||||||
Triggers an Emergency Pack | No | |||||||||||
Summary | 0046244: Remove legacy code that it is not working and it is making the login slower | |||||||||||
Description | File "web/org.openbravo.mobile.core/source/retail/component/ob-retail-searchproductcharacteristic.js" We can find the following peace of code if (!OB.MobileApp.model.hasPermission('OBPOS_remote.product', true)) { loadProductCharacteristics(); } else { var criteria = {}; OB.Dal.find( OB.Model.Characteristic, criteria, successCallbackProductCh, errorCallback ); } If we go deeper it is going to load all the product characteristics value let characteritics = await OB.App.MasterdataModels.ProductCharacteristicValue.find( criteriaCharacteristics ); And finally include into array for (let i = 0; i < characteritics.length; i++) { transformedCharacteristics.push( OB.Dal.transform( OB.Model.ProductCharacteristicValue, characteritics[i] ) ); } And finally include in the component attribute me.productsCh.reset(dataProductCh.models); But as it is an array, dataProductCh.models is always undefined and because it is not a backbone collection. This is an old code that maybe had sense in the past but no longer. P.S.: me.productsCh is set in another peace of code with product characteristic and not "product characteristics value" what it has more sense | |||||||||||
Steps To Reproduce | Do the login and verify that the above code is executed. | |||||||||||
Proposed Solution | Attached patch | |||||||||||
Tags | NOR | |||||||||||
Attached Files | patch.diff [^] (2,797 bytes) 2021-04-10 08:10 [Show Content] | |||||||||||
Relationships [ Relation Graph ] [ Dependency Graph ] | ||||||||
|
Issue History | |||
Date Modified | Username | Field | Change |
2021-04-10 08:10 | jetxarri | New Issue | |
2021-04-10 08:10 | jetxarri | Assigned To | => Retail |
2021-04-10 08:10 | jetxarri | File Added: patch.diff | |
2021-04-10 08:10 | jetxarri | Triggers an Emergency Pack | => No |
2021-04-12 09:45 | jetxarri | Tag Attached: NOR | |
2021-04-12 09:57 | rafaroda | Relationship added | related to 0046182 |
Copyright © 2000 - 2009 MantisBT Group |