Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0046244
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSminoralways2021-04-10 08:102021-04-12 09:57
ReporterjetxarriView Statuspublic 
Assigned ToRetail 
PrioritynormalResolutionopenFixed in Version
StatusnewFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionRR21Q1.1SCM revision 
Review Assigned To
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0046244: Remove legacy code that it is not working and it is making the login slower

DescriptionFile "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 ReproduceDo the login and verify that the above code is executed.
Proposed SolutionAttached patch
TagsNOR
Attached Filesdiff file icon patch.diff [^] (2,797 bytes) 2021-04-10 08:10 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
related to feature request 0046182 newRetail Refactor/improve of Promo Engine and full / incremental refresh to remove the ProductCharacteristicsValue model 

-  Notes
There are no notes attached to this issue.

- 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
Powered by Mantis Bugtracker