Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0029304Retail ModulesWeb POSpublic2015-03-16 19:332015-03-17 14:30
Reportermigueldejuana 
Assigned Tomigueldejuana 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutioninvalid 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0029304: Navigating to another window, with slow computers takes to much time
DescriptionNavigating to another window, with slow computers takes to much time.
Every time we navigate, we destroy all containerWindow components and then we create (add to) in containerWindow new window components.

When destroying, we iterate all components tu set model off and we iterate all properties of the component to set off and delete them. This take too much time en computers with slow single threat.

Some relevant information of destroying components:
http://enyojs.com/docs/latest/#/kind/enyo.Component:destroy [^]
Steps To ReproduceWe can get rid of deleting of properties of each component because it doesn't affect to the functionality. We are not sure if Garbage collector delete all data but after testing seems that yes.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2015-03-16 19:33migueldejuanaNew Issue
2015-03-16 19:33migueldejuanaAssigned To => migueldejuana
2015-03-16 19:33migueldejuanaTriggers an Emergency Pack => No
2015-03-16 19:42hgbotCheckin
2015-03-16 19:42hgbotNote Added: 0075662
2015-03-16 19:42hgbotStatusnew => resolved
2015-03-16 19:42hgbotResolutionopen => fixed
2015-03-16 19:42hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/d4bc17d3cbf7c9d25baa1da1249ede02d0a9ecc3 [^]
2015-03-17 14:29migueldejuanaStatusresolved => new
2015-03-17 14:29migueldejuanaResolutionfixed => open
2015-03-17 14:30migueldejuanaReview Assigned To => migueldejuana
2015-03-17 14:30migueldejuanaNote Added: 0075689
2015-03-17 14:30migueldejuanaStatusnew => closed
2015-03-17 14:30migueldejuanaResolutionopen => invalid

Notes
(0075662)
hgbot   
2015-03-16 19:42   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: d4bc17d3cbf7c9d25baa1da1249ede02d0a9ecc3
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Mon Mar 16 19:41:18 2015 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/d4bc17d3cbf7c9d25baa1da1249ede02d0a9ecc3 [^]

Fixed issue 0029304: Navigating to another window, with slow computers takes to much time

We delete this line because it takes time iterating all components properties using slow computers. This line, set off and delete each property of each component but it is not needed because destroying component each enough.

---
M web/org.openbravo.mobile.core/source/component/ob-windowview.js
---
(0075689)
migueldejuana   
2015-03-17 14:30   
See commit:

https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/bd61214ee217 [^]