Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0022306Openbravo ERPB. User interfacepublic2012-11-14 17:422013-01-28 18:20
Reportercaristu 
Assigned Toshankarb 
PriorityhighSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target Version3.0MP20Fixed in Version3.0MP20 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket17916
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0022306: Event Handler not updates the field that has the focus in grid view
DescriptionThe event handler component is not updating the field that has the focus, if we made the event (add, update) in grid view.
Steps To Reproduce1) Create an Event Handler, for example to update the list price in the Price tab of the Product window. (Use attached class)
2) Go to the [Price] tab in the [Product] Window
3a) Create a new row in form view, and put the focus on the list price field. Save the record. Check that the list price field is updated.
3b) Create a new row in grid view, and put the focus on the unit price field. Save the record. Check that the list price field is updated.
3c)Create a new row in grid view, and put the focus on the list price field. Save the record. Check that the list price field is NOT updated.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files? ProductPriceHandler.java (1,861) 2012-11-14 17:48
https://issues.openbravo.com/file_download.php?file_id=5732&type=bug
diff 22306.diff (719) 2013-01-16 08:31
https://issues.openbravo.com/file_download.php?file_id=5894&type=bug
Issue History
Date ModifiedUsernameFieldChange
2012-11-14 17:42caristuNew Issue
2012-11-14 17:42caristuAssigned To => dbaz
2012-11-14 17:42caristuModules => Core
2012-11-14 17:42caristuOBNetwork customer => Yes
2012-11-14 17:42caristuSupport ticket => 17916
2012-11-14 17:42caristuResolution time => 1359154800
2012-11-14 17:43caristuIssue Monitored: networkb
2012-11-14 17:43dbazAssigned Todbaz => shankarb
2012-11-14 17:48caristuFile Added: ProductPriceHandler.java
2012-11-30 12:28shankarbFile Added: FixIssue22306.diff
2012-11-30 12:43shankarbStatusnew => scheduled
2012-11-30 12:43shankarbfix_in_branch => pi
2012-12-04 06:12shankarbFile Deleted: FixIssue22306.diff
2012-12-07 10:20shankarbReview Assigned To => mtaal
2012-12-07 10:20shankarbTriggers an Emergency Pack => No
2012-12-07 10:20shankarbfix_in_branchpi =>
2012-12-07 10:21shankarbIssue Monitored: mtaal
2012-12-07 10:22hgbotCheckin
2012-12-07 10:22hgbotNote Added: 0054765
2012-12-07 10:22hgbotStatusscheduled => resolved
2012-12-07 10:22hgbotResolutionopen => fixed
2012-12-07 10:22hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/49a412287cd5e10b16e8ab632071aae91a6fd9eb [^]
2012-12-07 14:21hgbotCheckin
2012-12-07 14:21hgbotNote Added: 0054777
2012-12-11 17:54hudsonbotCheckin
2012-12-11 17:54hudsonbotNote Added: 0054858
2012-12-11 17:54hudsonbotCheckin
2012-12-11 17:54hudsonbotNote Added: 0054859
2012-12-18 04:31shankarbNote Added: 0055028
2012-12-18 04:31shankarbStatusresolved => new
2012-12-18 04:31shankarbResolutionfixed => open
2012-12-18 04:32shankarbNote Added: 0055029
2013-01-09 17:01AugustoMauchTarget Version3.0MP19 => 3.0MP20
2013-01-16 08:31shankarbFile Added: 22306.diff
2013-01-17 12:32shankarbReview Assigned Tomtaal => AugustoMauch
2013-01-17 12:32shankarbIssue End Monitor: mtaal
2013-01-17 12:33shankarbIssue Monitored: AugustoMauch
2013-01-17 12:33shankarbStatusnew => scheduled
2013-01-17 12:33shankarbfix_in_branch => pi
2013-01-17 12:34hgbotCheckin
2013-01-17 12:34hgbotNote Added: 0055562
2013-01-17 12:34hgbotStatusscheduled => resolved
2013-01-17 12:34hgbotResolutionopen => fixed
2013-01-17 12:34hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/49a412287cd5e10b16e8ab632071aae91a6fd9eb [^] => http://code.openbravo.com/erp/devel/pi/rev/a9324ffbf3a36e31aade56da744202cfe2a1848e [^]
2013-01-18 20:26hudsonbotCheckin
2013-01-18 20:26hudsonbotNote Added: 0055639
2013-01-21 08:48hgbotCheckin
2013-01-21 08:48hgbotNote Added: 0055641
2013-01-21 09:10hgbotCheckin
2013-01-21 09:10hgbotNote Added: 0055644
2013-01-22 15:33hudsonbotCheckin
2013-01-22 15:33hudsonbotNote Added: 0055725
2013-01-22 15:33hudsonbotCheckin
2013-01-22 15:33hudsonbotNote Added: 0055727
2013-01-28 18:20AugustoMauchNote Added: 0055923
2013-01-28 18:20AugustoMauchStatusresolved => closed
2013-01-28 18:20AugustoMauchFixed in Version => 3.0MP20

Notes
(0054765)
hgbot   
2012-12-07 10:22   
Repository: erp/devel/pi
Changeset: 49a412287cd5e10b16e8ab632071aae91a6fd9eb
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Tue Dec 04 13:30:36 2012 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/49a412287cd5e10b16e8ab632071aae91a6fd9eb [^]

Fixes Issue 0022306: Event Handler does not update field in focus in grid view

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
---
(0054777)
hgbot   
2012-12-07 14:21   
Repository: erp/devel/pi
Changeset: c098fb792dbfbba2e924f6196d3e4b0289c821cd
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Fri Dec 07 14:19:07 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/c098fb792dbfbba2e924f6196d3e4b0289c821cd [^]

Related to issue 22306: Backed out changeset 49a412287cd5

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
---
(0054858)
hudsonbot   
2012-12-11 17:54   
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/3b1b5e2f6c63 [^]

Maturity status: Test
(0054859)
hudsonbot   
2012-12-11 17:54   
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/3b1b5e2f6c63 [^]

Maturity status: Test
(0055028)
shankarb   
2012-12-18 04:31   
Backed out changeset @ http://code.openbravo.com/erp/devel/pi/rev/c098fb792dbfbba2e924f6196d3e4b0289c821cd [^] [^]
(0055029)
shankarb   
2012-12-18 04:32   
The value is actually updated. But not displayed at the client level. If a manual Tab level refresh is done, the updated value is shown.
(0055562)
hgbot   
2013-01-17 12:34   
Repository: erp/devel/pi
Changeset: a9324ffbf3a36e31aade56da744202cfe2a1848e
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Thu Jan 17 17:02:23 2013 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/a9324ffbf3a36e31aade56da744202cfe2a1848e [^]

Fixes Issue 0022306: Event Handler does not update focus field

Event Handler not update the field that has the focus in grid view.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-view-grid.js
---
(0055639)
hudsonbot   
2013-01-18 20: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/4cd4a6cc38e0 [^]

Maturity status: Test
(0055641)
hgbot   
2013-01-21 08:48   
Repository: erp/devel/pi
Changeset: 4db996f55ee3f1cfb7735dfa143efee59d1faa7a
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Mon Jan 21 13:07:54 2013 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/4db996f55ee3f1cfb7735dfa143efee59d1faa7a [^]

Related to Issue 22306 : Code Refactoring

Added comments to variable initialization and modified query.list() to scroll() for performance enhancement.

---
M modules/org.openbravo.userinterface.selector/src/org/openbravo/userinterface/selector/CustomQuerySelectorDatasource.java
---
(0055644)
hgbot   
2013-01-21 09:10   
Repository: erp/devel/pi
Changeset: bc196be64b0340c83a3ad718f17ed5407d1c8051
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Mon Jan 21 13:38:59 2013 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/bc196be64b0340c83a3ad718f17ed5407d1c8051 [^]

Related to Issue 22306 : Backed out changeset 4db996f55ee3

Backed out changset as that is related to issue 22684.

---
M modules/org.openbravo.userinterface.selector/src/org/openbravo/userinterface/selector/CustomQuerySelectorDatasource.java
---
(0055725)
hudsonbot   
2013-01-22 15:33   
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/449aae8be624 [^]

Maturity status: Test
(0055727)
hudsonbot   
2013-01-22 15:33   
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/449aae8be624 [^]

Maturity status: Test
(0055923)
AugustoMauch   
2013-01-28 18:20   
Code reviewed and verified in pi@e035b2855377