Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0021249Openbravo ERPA. Platformpublic2012-07-30 13:252012-08-09 13:07
AugustoMauch 
mtaal 
immediatemajorhave not tried
closedfixed 
20Professional Appliance
 
3.0MP143.0MP14 
Core
Production - QA Approved
https://code.openbravo.com/erp/devel/pi/rev/f060d41ab218 [^]
yes
0021249: Wrong grid layout when restoring original view in grid
If a view is created and set as default, and then the window is closed and opened again, the grid layout will break when restoring the standard view
- Open Unit of Measure window
- Hide the EDI Code column
- Save the view as TEST, and check the Set as Default checkbox
- Close the window, open it again
- Click on the Save View button and select the Standard View
- The grid layout will be broken
No tags attached.
Issue History
2012-07-30 13:25AugustoMauchNew Issue
2012-07-30 13:25AugustoMauchAssigned To => AugustoMauch
2012-07-30 13:25AugustoMauchModules => Core
2012-07-31 12:11AugustoMauchRegression level => Production - QA Approved
2012-07-31 12:11AugustoMauchRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/f060d41ab218 [^]
2012-07-31 13:06AugustoMauchNote Added: 0051030
2012-07-31 13:06AugustoMauchAssigned ToAugustoMauch => dbaz
2012-07-31 13:08AugustoMauchAssigned Todbaz => mtaal
2012-07-31 15:21hgbotCheckin
2012-07-31 15:21hgbotNote Added: 0051032
2012-07-31 15:21hgbotStatusnew => resolved
2012-07-31 15:21hgbotResolutionopen => fixed
2012-07-31 15:21hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/f9d149854a7f0267290d0d3da8e34e31f16e78e1 [^]
2012-07-31 16:04AugustoMauchNote Added: 0051033
2012-07-31 16:04AugustoMauchStatusresolved => closed
2012-07-31 16:04AugustoMauchFixed in Version => 3.0MP14
2012-08-09 13:07hudsonbotCheckin
2012-08-09 13:07hudsonbotNote Added: 0051203

Notes
(0051030)
AugustoMauch   
2012-07-31 13:06   
This [1] is the line where the layout gets broken.

[1] https://code.openbravo.com/erp/devel/pi/file/f7250e458578/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js#l546 [^]
(0051032)
hgbot   
2012-07-31 15:21   
Repository: erp/devel/pi
Changeset: f9d149854a7f0267290d0d3da8e34e31f16e78e1
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue Jul 31 15:20:54 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/f9d149854a7f0267290d0d3da8e34e31f16e78e1 [^]

Fixes issue 21249: Wrong grid layout when restoring original view in grid
Filtereditor had focus, unfocus during set viewstate resulted in an error because
the filtereditor was using an old field definition, resolve by resetting focus
and then recomputing it after viewstate. Recompute focus after viewstate needs to
be done anyway as the setviewstate can have removed the current focus field.

---
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-window.js
---
(0051033)
AugustoMauch   
2012-07-31 16:04   
Code reviewed and verified in pi@f9d149854a7f
(0051203)
hudsonbot   
2012-08-09 13:07   
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/ef9a070edd19 [^]

Maturity status: Test