Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0009728Openbravo ERPA. Platformpublic2009-06-30 20:402009-07-10 14:32
shuehner 
shuehner 
normalminorhave not tried
closedfixed 
5
2.40 
2.40MP7 
Core
No
0009728: All generated pages load the MessagesJS data after loading the page. This extra request should be removed to improve performance
All generated windows make a minimum of two requests:
1.) the html for the window itself
2.) a request to MessagesJS (in 2.50 DynamicJS) to read/preload some error messages.

This second request delays the page rendering until its result is recieved. As the error messages only change on language switch this extra request shoul be avoided to improve performance.
Either:
Use firebug or a similar tool and the net panel to see the network request
Or:
Read apache/tomcat access log on the server to see the requests.

Goto to i.e. product edit view
See the an extra request to MessagesJS
No tags attached.
blocks feature request 0009727 closed shuehner All generated pages load the MessagesJS data after loading the page. This extra request should be removed to improve performance 
Issue History
2009-06-30 20:41shuehnerTypefeature request => backport
2009-06-30 20:41shuehnerfix_in_branch => 2.40
2009-06-30 21:28hgbotCheckin
2009-06-30 21:28hgbotNote Added: 0017700
2009-06-30 21:28hgbotStatusscheduled => resolved
2009-06-30 21:28hgbotResolutionopen => fixed
2009-06-30 21:28hgbotFixed in SCM revision => http://code.openbravo.com/erp/stable/2.40/rev/3406f3cb518cc00b24e9eda050de4d5bc8423ed1 [^]
2009-06-30 21:29shuehnerNote Added: 0017701
2009-07-10 14:32sureshbabuStatusresolved => closed
2009-07-10 14:32sureshbabuNote Added: 0018108
2009-07-10 14:32sureshbabuFixed in Version => 2.40MP7

Notes
(0017700)
hgbot   
2009-06-30 21:28   
Repository: erp/stable/2.40
Changeset: 3406f3cb518cc00b24e9eda050de4d5bc8423ed1
Author: Stefan Hühner <stefan.huehner <at> openbravo.com>
Date: Tue Jun 30 21:27:58 2009 +0200
URL: http://code.openbravo.com/erp/stable/2.40/rev/3406f3cb518cc00b24e9eda050de4d5bc8423ed1 [^]

Fixed 9728: Remove unneeded MessagesJS requests from generated windows and search popup
part1:
- Remove request from edit view and grid view of generated window
- Remove request from search popup
- Instead take needed data from the menu frame
- For this add a request to MEssagesJS to VerticalMenu.html (the menuframe content)

---
M src-wad/src/org/openbravo/wad/Template_Edition.html
M src-wad/src/org/openbravo/wad/Template_Relation.html
M src/org/openbravo/erpCommon/businessUtility/Buscador.html
M src/org/openbravo/erpCommon/security/Login_F1.html
M src/org/openbravo/erpCommon/utility/VerticalMenu.html
M web/js/messages.js
M web/js/utils.js
---
(0017701)
shuehner   
2009-06-30 21:29   
Needed testcases for verifying changes:
1.) search popup displays error message properly
goto product edit view
the search popup is automatically opened
remove all % character in there by default
press ok
A red error box should be shown as no fields are filled

2.) error messages work in edit view of generated window
goto i.e. product edit view (or any other generated window)
press ok in the opened search popup to display a single record
press delete in the toolbar
a popup should be opened displaying the confirmation message.

3.a) error message popup work in grid view of generated window
goto i.e. product grid view (or any other generatd window)
select (highlight) on record.
press delete in the toolbar
a popup should be opened displaying the confirmation message.
3.b) error message popup work in grid view of generated window
goto i.e. product grid view (or any other generatd window)
select (highlight) severals records.
click on the 'switch to edit view' icon in the LeftTabsBar.
a error message that more than one record is selected should be shown.

test another (unchanged) place to check for regression in error message display code
4.) automatically generated parameter popup for a report
goto financial management
goto Accounting
goto analysis tools
goto Create Tax Report
to not select any record in the dropdown
Press ok
a red box with an error message: 'This field is needed and cannot be blank' should be shown
(0018108)
sureshbabu   
2009-07-10 14:32   
Tested working fine