Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0027612Openbravo ERPA. Platformpublic2014-09-16 09:462014-09-26 17:50
ReporterAugustoMauch 
Assigned Toshankarb 
PriorityimmediateSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0PR14Q4Fixed in Version3.0PR14Q4 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - QA Approved
Regression date2014-05-26
Regression introduced in release3.0PR14Q3
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/402d7bca3f774737bf194cb7cbe41d2454a69326 [^]
Triggers an Emergency PackNo
Summary0027612: It is not possible to delete the contents of the Organization field using the keyboard.
DescriptionIt is not possible to delete the contents of the Organization field using the keyboard. Right after deleting the last character a FIC call in CHANGE mode is done, and the organization field content is reset.
Steps To ReproduceIn any window that has an organization field (i.e. Sales Order):
- Create a new record
- Select all the text of the organization field and press the Delete button. The contents will be erased, and right after that they will be set again.

This happens because just after the last character is deleted a FIC call in CHANGE mode is done. This FIC call should be done when the field loses the focus, not the moment when is emptied.

This fix is making CI fail under some circumstances.

In Q2 this issue was not reproducible.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
caused by feature request 00262673.0PR14Q3 closed shankarb Re-implement combos (table and tableDir) to use datasources 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2014-09-16 09:46AugustoMauchNew Issue
2014-09-16 09:46AugustoMauchAssigned To => AugustoMauch
2014-09-16 09:46AugustoMauchModules => Core
2014-09-16 09:46AugustoMauchTriggers an Emergency Pack => No
2014-09-16 09:54AugustoMauchSeveritymajor => minor
2014-09-16 10:01AugustoMauchNote Added: 0070136
2014-09-16 10:03alostaleRelationship addedcaused by 0026267
2014-09-16 10:04alostaleRegression level => Production - QA Approved
2014-09-16 10:04alostaleRegression introduced in release => 3.0PR14Q3
2014-09-16 10:04alostaleAssigned ToAugustoMauch => shankarb
2014-09-16 10:22alostaleTarget Version => 3.0PR14Q4
2014-09-17 04:31shankarbReview Assigned To => AugustoMauch
2014-09-17 04:32shankarbIssue Monitored: AugustoMauch
2014-09-17 04:34hgbotCheckin
2014-09-17 04:34hgbotNote Added: 0070165
2014-09-17 04:34hgbotStatusnew => resolved
2014-09-17 04:34hgbotResolutionopen => fixed
2014-09-17 04:34hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/cec2db9c3264b5560bc28a6d8b288b95d8993271 [^]
2014-09-17 08:36shankarbNote Added: 0070167
2014-09-17 08:36shankarbStatusresolved => new
2014-09-17 08:36shankarbResolutionfixed => open
2014-09-17 08:37shankarbRegression date => 2014-05-26
2014-09-17 08:37shankarbRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/402d7bca3f774737bf194cb7cbe41d2454a69326 [^]
2014-09-17 08:37shankarbStatusnew => scheduled
2014-09-17 08:37shankarbfix_in_branch => pi
2014-09-17 08:37shankarbStatusscheduled => resolved
2014-09-17 08:37shankarbResolutionopen => fixed
2014-09-17 13:49hudsonbotCheckin
2014-09-17 13:49hudsonbotNote Added: 0070179
2014-09-24 15:34hgbotCheckin
2014-09-24 15:34hgbotNote Added: 0070314
2014-09-25 11:30hudsonbotCheckin
2014-09-25 11:30hudsonbotNote Added: 0070360
2014-09-26 17:50AugustoMauchNote Added: 0070442
2014-09-26 17:50AugustoMauchStatusresolved => closed
2014-09-26 17:50AugustoMauchFixed in Version => 3.0PR14Q4

Notes
(0070136)
AugustoMauch   
2014-09-16 10:01   
In Q2 (this issue was not reproducible then), when the last character of a selector field is erased, the FIC call in CHANGE mode is done, but the value of the selector is not reset.
(0070165)
hgbot   
2014-09-17 04:34   
Repository: erp/devel/pi
Changeset: cec2db9c3264b5560bc28a6d8b288b95d8993271
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Wed Sep 17 02:53:48 2014 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/cec2db9c3264b5560bc28a6d8b288b95d8993271 [^]

Fixes Issue 0027612: Delete contents in organization selector using keyboard works

handles a corner case when the whole text of the selector field is removed, FIC call is
done with comboReload, but reload is not necessary for the changed column as we have just
entered empty value.

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/UIDefinition.java
---
(0070167)
shankarb   
2014-09-17 08:36   
Reopened to update regression fields.
(0070179)
hudsonbot   
2014-09-17 13:49   
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/c8fa8d228056 [^]
Maturity status: Test
(0070314)
hgbot   
2014-09-24 15:34   
Repository: erp/devel/pi
Changeset: 34db3c9da731732b229234482210d3f3228a1e5e
Author: Shankar Balachandran <shankar.balachandran <at> openbravo.com>
Date: Wed Sep 24 19:00:36 2014 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/34db3c9da731732b229234482210d3f3228a1e5e [^]

Related to issue 0027612 : Added test case and validation for ad_org_id

The issue is reproducible only for ad_org_id column. In FIC, computeListOfColumnsSortedByValidationDependencies method,
even if the changed column is ad_org_id,it is added for calculating the org dependency of other fields.
So added the validation in UIDefinition to handle ad_org_id alone.
Added test case to cover this scenario.

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/reference/UIDefinition.java
M src-test/src/org/openbravo/test/datasource/TestComboDatasource.java
---
(0070360)
hudsonbot   
2014-09-25 11: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/63604e8e17b7 [^]
Maturity status: Test
(0070442)
AugustoMauch   
2014-09-26 17:50   
Code reviewed and verified in pi@f5caad158261