Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0024228Openbravo ERPA. Platformpublic2013-06-29 14:592013-07-29 09:42
Reportershuehner 
Assigned ToAugustoMauch 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0MP26 
Merge Request Status
Review Assigned Toshankarb
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0024228: When clicking new in Sales Order Grid request is fired to OrderLine data
DescriptionWhen having the Sales Order grid open with no row selected and clicking in 'New in Form' then a datasource get request is fired for the OrderLine.

That does not make sense as in this situation there cannot be any child-records as with are just creating a new one
Steps To ReproduceOpen Sales Order window
Remove implicit filter so that records are shown in the grid
Without any record selected use the 'New in Form' button from the toolbar.
Observer that there is a request to the OrderLine datasource.
Proposed Solution
Additional Information
TagsPerformance
Relationships
related to defect 0024229 closed AugustoMauch When in 'New in Form' directly leaving that mode with the X without saving causes datasource for children to be loaded 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-06-29 14:59shuehnerNew Issue
2013-06-29 14:59shuehnerAssigned To => AugustoMauch
2013-06-29 14:59shuehnerModules => Core
2013-06-29 14:59shuehnerOBNetwork customer => No
2013-06-29 14:59shuehnerTriggers an Emergency Pack => No
2013-06-29 14:59shuehnerTag Attached: Performance
2013-06-29 15:02shuehnerRelationship addedrelated to 0024229
2013-07-11 07:13shankarbAssigned ToAugustoMauch => shankarb
2013-07-11 07:13shankarbReview Assigned To => mtaal
2013-07-11 07:16shankarbIssue Monitored: mtaal
2013-07-11 07:17hgbotCheckin
2013-07-11 07:17hgbotNote Added: 0059944
2013-07-11 07:17hgbotStatusnew => resolved
2013-07-11 07:17hgbotResolutionopen => fixed
2013-07-11 07:17hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/7cb2a57521034eb2db3f61621dc7fa3136fca2fb [^]
2013-07-16 14:26hudsonbotCheckin
2013-07-16 14:26hudsonbotNote Added: 0060036
2013-07-24 13:23AugustoMauchAssigned Toshankarb => AugustoMauch
2013-07-24 13:23AugustoMauchNote Added: 0060226
2013-07-24 13:23AugustoMauchStatusresolved => new
2013-07-24 13:23AugustoMauchResolutionfixed => open
2013-07-24 16:53AugustoMauchIssue End Monitor: mtaal
2013-07-24 16:53AugustoMauchIssue Monitored: shankarb
2013-07-24 16:53AugustoMauchReview Assigned Tomtaal => shankarb
2013-07-24 16:54hgbotCheckin
2013-07-24 16:54hgbotNote Added: 0060265
2013-07-24 16:54hgbotStatusnew => resolved
2013-07-24 16:54hgbotResolutionopen => fixed
2013-07-24 16:54hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/7cb2a57521034eb2db3f61621dc7fa3136fca2fb [^] => http://code.openbravo.com/erp/devel/pi/rev/295e879c019810985e4eb7290aa94209fe96566b [^]
2013-07-26 20:32hudsonbotCheckin
2013-07-26 20:32hudsonbotNote Added: 0060312
2013-07-29 09:42shankarbNote Added: 0060343
2013-07-29 09:42shankarbStatusresolved => closed
2013-07-29 09:42shankarbFixed in Version => 3.0MP26

Notes
(0059944)
hgbot   
2013-07-11 07:17   
Repository: erp/devel/pi
Changeset: 7cb2a57521034eb2db3f61621dc7fa3136fca2fb
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Thu Jul 11 10:46:28 2013 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/7cb2a57521034eb2db3f61621dc7fa3136fca2fb [^]

Fixes Issue 0024228: When clicking new in Sales Order Grid request is fired to OrderLine data

When new row is opened for editing, if the grid has selected records, then the child tabs has to be refreshed.
If no records are selected then the request to child tabs is prevented.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-standard-view.js
---
(0060036)
hudsonbot   
2013-07-16 14:26   
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/9a5d5983399f [^]

Maturity status: Test
(0060226)
AugustoMauch   
2013-07-24 13:23   
The fix is not proper. If before clicking on the 'New in Form' button the user selects a record in the grid, a request will be done to the Sales Order datasource.
(0060265)
hgbot   
2013-07-24 16:54   
Repository: erp/devel/pi
Changeset: 295e879c019810985e4eb7290aa94209fe96566b
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Wed Jul 24 16:52:37 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/295e879c019810985e4eb7290aa94209fe96566b [^]

Fixes issue 24228, issue 24229: Avoids not needed datasource

Some not needed datasource calls were being made when a record was created in grid or form view. It made no sense to ask the datasource for the data of the child tabs of the new record, we can be sure there are none. To fix this, instead of calling the datasource the data of the child grid is emptied manually, and then the emptyMessage of the grid is updated.

---
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
---
(0060312)
hudsonbot   
2013-07-26 20:32   
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/b9e43ea5aa1b [^]

Maturity status: Test
(0060343)
shankarb   
2013-07-29 09:42   
Code reviewed and tested in pi changeset 2128830a28ff.