Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0037897Openbravo ERPA. Platformpublic2018-02-06 15:532018-02-15 09:50
Reportervmromanos 
Assigned Toplatform 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target Version3.0PR17Q4.1Fixed in Version3.0PR17Q4.1 
Merge Request Status
Review Assigned Tocaristu
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2016-01-28
Regression introduced in release3.0PR16Q2
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/baf72de5b678a308c36d5946af3e143d08625eca [^]
Triggers an Emergency PackNo
Summary0037897: Callout not triggered when clearing a selector
DescriptionA callout is associated to a nullable column with a OBUISEL_Selector Reference.
When the user selects something in the selector the callout is properly triggered. However when the user clears the selector (i.e. no value is selected) and the focus is changed to another field then the callout is NOT triggered.

I have noticed a special case where the callout is properly triggered: when you open a window and then you clear the selector of an existing record (i.e. a record previously saved in the DB) then the callout is properly triggered.
Steps To ReproduceAssociate a callout with a nullable column with a OBUISEL_Selector Reference.
Compile and restart tomcat in debug mode.
Set a break point at the beginning of the callout.
Create a new record in the correspondent window. Select any value in the selector and change the focus. Verify the callout is properly triggered. OK
Now clears the selector and change the focus. The callout is NOT triggered. WRONG.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0037842 closed platform Callout not triggered when clearing a selector 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2018-02-13 10:36AugustoMauchTypedefect => backport
2018-02-13 10:36AugustoMauchTarget Version => 3.0PR17Q4.1
2018-02-14 09:43hgbotCheckin
2018-02-14 09:43hgbotNote Added: 0102388
2018-02-14 09:43hgbotStatusscheduled => resolved
2018-02-14 09:43hgbotResolutionopen => fixed
2018-02-14 09:43hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR17Q4.1/rev/3329a5d35ffaa7915a9dbc99001cce238602147a [^]
2018-02-15 07:50alostaleReview Assigned To => caristu
2018-02-15 09:50caristuNote Added: 0102430
2018-02-15 09:50caristuStatusresolved => closed
2018-02-15 09:50caristuFixed in Version => 3.0PR17Q4.1

Notes
(0102388)
hgbot   
2018-02-14 09:43   
Repository: erp/backports/3.0PR17Q4.1
Changeset: 3329a5d35ffaa7915a9dbc99001cce238602147a
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Wed Feb 14 09:40:23 2018 +0100
URL: http://code.openbravo.com/erp/backports/3.0PR17Q4.1/rev/3329a5d35ffaa7915a9dbc99001cce238602147a [^]

Fixes issue 37897: Callouts are invoked when clearing a non mandatory selector

This changeset [1] fixed a problem where a callout was being called twice when picking a value from a selector. The first request was done properly, but the
second one was done with a null value. The fix consisted in not notifying a change in selector when it was modified from a non-empty value to an empty value.

But as a result, when a non mandatory selector is emptied, callouts are no longer being invoked. The original issue is very hard to reproduce, I have not been
able to find a new way to reproduce it or fix it. To avoid the regression, now the code that was done to fix [1] will only take place if the selector is mandatory,
to make sure that it is possible to give empty values to non-mandatory selectors.

[1] https://code.openbravo.com/erp/devel/pi/rev/baf72de5b678a308c36d5946af3e143d08625eca [^]

---
M modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-item.js
---
(0102430)
caristu   
2018-02-15 09:50   
Code reviewed + tested OK.

Tested with Product field in the Lines tab of Sales Invoice window.