Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0024705Openbravo ERPA. Platformpublic2013-09-05 18:352014-05-22 22:16
egoitz 
dbaz 
urgentmajoralways
closedfixed 
5
 
3.0PR14Q3 
alostale
Core
No
0024705: The windows takes long to be loaed when there are a lot of tabs and a lot of columns on the grid.
The windows takes long to be loaed when there are a lot of tabs and a lot of columns on the grid.

When you click on a menu item to open a window first of all the window is created and then the datasource is called.
When there are many tabs it takes long to creaet the tab, so the request to the datasource is done after some seconds.

It produces a delay on the window load.

ON computer (client side) with a big amount of resources the creating of the window is fater.





-Create several tabs on product window
-Open the window having the developers tool opened.
-See that after click it takes some seconds to do the request to the datasource.

-Without creating tabs, you can see that the request to the datasource is done faster on windows with a unique tab (Preferences) than on other tabs with more tabs (Product, Partner)
-Not to create the window every time that you open it, have it on the cache.
or
-Do the request to the datasource at the same time that the window is created (asynchronous)
Performance
related to defect 00253873.0MP31 closed alostale unneeded flushes in view generation 
related to defect 0026660 closed guillermogil Read Only tabs are editable when tab level is 2 
related to defect 00269903.0PR14Q2.3 closed dbaz Field access tab logic is not working 
related to defect 00270083.0PR14Q4 closed shankarb "Recent Documents" is not working under some circumstances 
related to defect 0028501 closed AugustoMauch Not possible to open a window in read only mode more than once 
related to defect 00288933.0PR15Q2 closed alostale Not possible to nullify fields in grid view using window personalization 
related to defect 0037109 closed caristu Parent form shown in blank when direct opening a record in a child tab having parent tab configured as "Default Edit Mode" 
related to defect 0039269 acknowledged Triage Platform Base Form Personalization > Restore Defaults button does not work properly in some tabs 
causes defect 00271213.0PR14Q4 closed dbaz Windows personalization open always the subtab hidding the header if you save in a special situation 
causes defect 00275033.0PR14Q4 closed AugustoMauch Form view is not working fine when using views 
causes defect 0028454 closed alostale can't nullify fields when 1st edition is in grid 
Issue History
2013-09-05 18:35egoitzNew Issue
2013-09-05 18:35egoitzAssigned To => AugustoMauch
2013-09-05 18:35egoitzModules => Core
2013-09-05 18:35egoitzTriggers an Emergency Pack => No
2013-09-09 16:44jpcalventeIssue Monitored: jpcalvente
2013-12-19 01:22egoitzTag Attached: Performance
2013-12-20 12:39alostaleRelationship addedrelated to 0025387
2014-02-07 15:16hgbotCheckin
2014-02-07 15:16hgbotNote Added: 0064021
2014-02-10 16:48hgbotCheckin
2014-02-10 16:48hgbotNote Added: 0064032
2014-02-12 18:30hudsonbotCheckin
2014-02-12 18:30hudsonbotNote Added: 0064174
2014-02-12 18:30hudsonbotCheckin
2014-02-12 18:30hudsonbotNote Added: 0064180
2014-04-04 15:41johnfandlIssue Monitored: johnfandl
2014-04-21 15:27hgbotCheckin
2014-04-21 15:27hgbotNote Added: 0066518
2014-04-21 15:28dbazAssigned ToAugustoMauch => dbaz
2014-04-21 15:28dbazReview Assigned To => alostale
2014-04-21 15:28dbazSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=5725#r5725
2014-04-21 15:28dbazProposed Solution updated
2014-05-05 10:47hgbotCheckin
2014-05-05 10:47hgbotNote Added: 0066801
2014-05-05 10:47hgbotStatusnew => resolved
2014-05-05 10:47hgbotResolutionopen => fixed
2014-05-05 10:47hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/cd8dfb19562674d797f83a6327f16e230029b005 [^]
2014-05-05 11:18alostaleNote Added: 0066804
2014-05-05 11:18alostaleStatusresolved => closed
2014-05-05 11:18alostaleFixed in Version => PR14Q3
2014-05-06 10:09hgbotCheckin
2014-05-06 10:09hgbotNote Added: 0066830
2014-05-06 10:10alostaleNote Added: 0066831
2014-05-06 10:10alostaleStatusclosed => new
2014-05-06 10:10alostaleResolutionfixed => open
2014-05-06 10:10alostaleFixed in VersionPR14Q3 =>
2014-05-07 18:11hudsonbotCheckin
2014-05-07 18:11hudsonbotNote Added: 0066954
2014-05-07 18:12hudsonbotCheckin
2014-05-07 18:12hudsonbotNote Added: 0066995
2014-05-07 18:12hudsonbotCheckin
2014-05-07 18:12hudsonbotNote Added: 0067000
2014-05-09 10:09hgbotCheckin
2014-05-09 10:09hgbotNote Added: 0067037
2014-05-09 10:09hgbotStatusnew => resolved
2014-05-09 10:09hgbotResolutionopen => fixed
2014-05-09 10:09hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/cd8dfb19562674d797f83a6327f16e230029b005 [^] => http://code.openbravo.com/erp/devel/pi/rev/b6eadf2004263b2719ae7d003a3ae7ba60424be9 [^]
2014-05-12 08:28alostaleNote Added: 0067081
2014-05-12 08:28alostaleStatusresolved => closed
2014-05-12 08:28alostaleFixed in Version => PR14Q3
2014-05-12 11:46hgbotCheckin
2014-05-12 11:46hgbotNote Added: 0067086
2014-05-12 17:35hudsonbotCheckin
2014-05-12 17:35hudsonbotNote Added: 0067095
2014-05-15 17:34hudsonbotCheckin
2014-05-15 17:34hudsonbotNote Added: 0067159
2014-05-22 13:55guillermogilRelationship addedrelated to 0026660
2014-05-22 22:16dbazNote Added: 0067390
2014-07-02 09:36alostaleRelationship addedrelated to 0026990
2014-07-18 10:33shankarbRelationship addedrelated to 0027008
2014-07-28 16:12alostaleRelationship addedcauses 0027121
2014-08-29 10:54AugustoMauchRelationship addedcauses 0027503
2014-12-19 09:25alostaleRelationship addedcauses 0028454
2015-01-13 09:05alostaleRelationship addedrelated to 0028501
2015-02-10 10:41alostaleRelationship addedrelated to 0028893
2017-10-18 19:24caristuRelationship addedrelated to 0037109
2018-09-11 18:36caristuRelationship addedrelated to 0039269

Notes
(0064021)
hgbot   
2014-02-07 15:16   
Repository: erp/devel/pi
Changeset: 09050a5b416213987340319de2dba42c481ca18e
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Fri Feb 07 15:16:17 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/09050a5b416213987340319de2dba42c481ca18e [^]

Related to issue 24705: Lazy evaluation of viewForm
Now most of the elements and the logic of the viewForm is loaded when the form needs to be visualized

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-window.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/personalization/ob-manage-views.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/personalization/ob-personalization.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/personalization/ob-personalize-form.js
---
(0064032)
hgbot   
2014-02-10 16:48   
Repository: erp/devel/pi
Changeset: 200ca8a32147615c0f0ca35c3ee84197721b1f3a
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Mon Feb 10 16:47:58 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/200ca8a32147615c0f0ca35c3ee84197721b1f3a [^]

Related to issue 24705: Lazy evaluation of viewForm - microfix

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js
---
(0064174)
hudsonbot   
2014-02-12 18:30   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/d1a5bb862230 [^]
Maturity status: Test
(0064180)
hudsonbot   
2014-02-12 18:30   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/d1a5bb862230 [^]
Maturity status: Test
(0066518)
hgbot   
2014-04-21 15:27   
Repository: erp/devel/pi
Changeset: cb6171e6cd94ffc8e2004c26159a82f60fc35de3
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Mon Apr 21 15:19:02 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/cb6171e6cd94ffc8e2004c26159a82f60fc35de3 [^]

Fixed issue 26316: Status bar fields are shown again in saved views
Related to issue 24705: 'getPersonalizationDataFromForm' now takes into account existing form fields

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/personalization/ob-manage-views.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/personalization/ob-personalization.js
---
(0066801)
hgbot   
2014-05-05 10:47   
Repository: erp/devel/pi
Changeset: cd8dfb19562674d797f83a6327f16e230029b005
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Mon May 05 10:46:51 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/cd8dfb19562674d797f83a6327f16e230029b005 [^]

fixed issue 24705: improved client side performance when loading windows

lazy loading of grid fields, now grids and their fields are loaded only when
the tabs containing them are opened by user

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-window.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-tab.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/personalization/ob-manage-views.js
---
(0066804)
alostale   
2014-05-05 11:18   
code reviewed + tested
(0066830)
hgbot   
2014-05-06 10:09   
Repository: erp/devel/pi
Changeset: 3a3619678a3c9d34c1bb4d13c2ff79d493d95e74
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue May 06 10:08:33 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/3a3619678a3c9d34c1bb4d13c2ff79d493d95e74 [^]

related to issue 24705: backed out grid lazy load changeset

  it was causing CI to fail

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-window.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-tab.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/personalization/ob-manage-views.js
---
(0066831)
alostale   
2014-05-06 10:10   
Reopening it because last changeset has been to be backed out as it was causing CI to fail
(0066954)
hudsonbot   
2014-05-07 18:11   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/02ca2648bcdc [^]
Maturity status: Test
(0066995)
hudsonbot   
2014-05-07 18:12   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/02ca2648bcdc [^]
Maturity status: Test
(0067000)
hudsonbot   
2014-05-07 18:12   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/02ca2648bcdc [^]
Maturity status: Test
(0067037)
hgbot   
2014-05-09 10:09   
Repository: erp/devel/pi
Changeset: b6eadf2004263b2719ae7d003a3ae7ba60424be9
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Fri May 09 10:08:25 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/b6eadf2004263b2719ae7d003a3ae7ba60424be9 [^]

fixed issue 24705: reaplying changeset after CI modifications

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-window.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-tab.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/personalization/ob-manage-views.js
---
(0067081)
alostale   
2014-05-12 08:28   
closed after successful CI
(0067086)
hgbot   
2014-05-12 11:46   
Repository: erp/devel/pi
Changeset: 9fa863bf7f350fcec062b7899d5fe546536ed1cd
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Mon May 12 11:45:41 2014 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/9fa863bf7f350fcec062b7899d5fe546536ed1cd [^]

Related to issue 24705: Add some comments/help

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js
---
(0067095)
hudsonbot   
2014-05-12 17:35   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/8bdb56ba8f66 [^]
Maturity status: Test
(0067159)
hudsonbot   
2014-05-15 17:34   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/8d6658b982c0 [^]
Maturity status: Test
(0067390)
dbaz   
2014-05-22 22:16   
The lazy loading of form view saves at least a 25% of the total time required to load the tab.

The lazy loading of the grid view saves at least another 25% of the total time required to load the tab.

Depending of the machine where the comparison be done, the percentage of the saving could be even higher.