Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0032007Openbravo ERPA. Platformpublic2016-01-26 09:372016-03-17 10:55
malsasua 
caristu 
normalmajorsometimes
closedfixed 
20Community Appliance
 
3.0PR16Q2 
alostale
Core
No
0032007: Double request to FIC in CHANGE mode when the form is redrawn by overflow under some circumstances
Double request to FIC in CHANGE mode when the form is redrawn by overflow after changing a combo/selector value which has a callout. In some cases, this can cause that the callouts are not working properly.
Configuration Steps (System Admin Role)
- Go to [Windows, Tabs and Fields]. Look for the [Sales Order] window
- Navigate to the [Lines] tab and select the Project field.
- Set the following display logic for the Project field: @M_Product_ID@='BDE2F1CF46B54EF58D33E20A230DA8D2' (This field is displaying when the selected product is 'Agua sin Gas 1L')

Steps To Reproduce (Group Admin Role)
- Navigate to the [Sales Order] window.
- Create a new record in the header.
- Create a new record in the [Lines] tab, using form view. Notice that the tab content is shown without scrolling. (See attached image, img1.png)
- Select the product 'Bebida Energética': one request is done to the FIC in CHANGE mode. This is correct.
- Select the product 'Agua Sin Gas 1L': the project field is displayed causing the appearance of the scrolling (See attached image, img2.png). Notice that in this case, two requests are done to the FIC in CHANGE mode.
Performance
related to defect 0032030 acknowledged Triage Platform Base Double FIC request after selecting the empty value on a combo with a callout 
causes defect 0037842 closed platform Callout not triggered when clearing a selector 
png img1.png (59,021) 2016-01-27 10:12
https://issues.openbravo.com/file_download.php?file_id=8968&type=bug
png

png img2.png (63,813) 2016-01-27 10:13
https://issues.openbravo.com/file_download.php?file_id=8970&type=bug
png
Issue History
2016-01-26 09:37malsasuaNew Issue
2016-01-26 09:37malsasuaAssigned To => platform
2016-01-26 09:37malsasuaModules => Core
2016-01-26 09:37malsasuaResolution time => 1454713200
2016-01-26 09:37malsasuaTriggers an Emergency Pack => No
2016-01-26 09:49malsasuaNote Added: 0083605
2016-01-26 19:39caristuStatusnew => scheduled
2016-01-26 19:39caristuAssigned Toplatform => caristu
2016-01-27 10:08caristuSummarycallouts are not working with screen resolution 1600x900 => Double request to FIC in CHANGE mode when the form is redrawn by overflow
2016-01-27 10:08caristuDescription Updatedbug_revision_view_page.php?rev_id=10775#r10775
2016-01-27 10:08caristuSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=10777#r10777
2016-01-27 10:11caristuSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=10778#r10778
2016-01-27 10:12caristuFile Added: img1.png
2016-01-27 10:12caristuFile Added: img2.png
2016-01-27 10:13caristuFile Deleted: img2.png
2016-01-27 10:13caristuFile Added: img2.png
2016-01-27 19:08caristuRelationship addedrelated to 0032030
2016-01-27 19:10caristuNote Added: 0083646
2016-01-28 17:09caristuSummaryDouble request to FIC in CHANGE mode when the form is redrawn by overflow => Double request to FIC in CHANGE mode when the form is redrawn by overflow under some cirmcustances
2016-01-28 17:09caristuDescription Updatedbug_revision_view_page.php?rev_id=10803#r10803
2016-01-28 17:09caristuSummaryDouble request to FIC in CHANGE mode when the form is redrawn by overflow under some cirmcustances => Double request to FIC in CHANGE mode when the form is redrawn by overflow under some circumstances
2016-01-28 17:23hgbotCheckin
2016-01-28 17:23hgbotNote Added: 0083689
2016-01-28 17:23hgbotStatusscheduled => resolved
2016-01-28 17:23hgbotResolutionopen => fixed
2016-01-28 17:23hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/baf72de5b678a308c36d5946af3e143d08625eca [^]
2016-01-28 17:24caristuReview Assigned To => alostale
2016-01-28 17:24caristuIssue Monitored: alostale
2016-01-29 09:17hgbotCheckin
2016-01-29 09:17hgbotNote Added: 0083711
2016-01-29 09:18alostaleNote Added: 0083712
2016-01-29 09:18alostaleStatusresolved => closed
2016-01-29 09:18alostaleFixed in Version => 3.0PR16Q2
2016-01-29 10:04alostaleTag Attached: Performance
2016-03-17 10:55hudsonbotCheckin
2016-03-17 10:55hudsonbotNote Added: 0085084
2016-03-17 10:55hudsonbotCheckin
2016-03-17 10:55hudsonbotNote Added: 0085086
2018-02-07 13:07alostaleRelationship addedcauses 0037842

Notes
(0083605)
malsasua   
2016-01-26 09:49   
It is reproducible using the next chrome extension to 1600x900:

http://wiki.openbravo.com/wiki/Retail:Developers_Guide/How-to/How_to_test_screen_resolutions [^]
(0083646)
caristu   
2016-01-27 19:10   
Fix pushed to try
(0083689)
hgbot   
2016-01-28 17:23   
Repository: erp/devel/pi
Changeset: baf72de5b678a308c36d5946af3e143d08625eca
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Thu Jan 28 17:22:21 2016 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/baf72de5b678a308c36d5946af3e143d08625eca [^]

fixes issue 32007: combo/selector with a callout can cause a double FIC request

If a form redrawal was fired after selecting a value of a combo/selector with a callout, a double request to the datasource was being performed due to asynchrony problems. An example of this scenario is when after selecting a value of a combo/selector with a callout, the form needs to be scrolled (overflow) because new fields are displayed in the form (for example because this fields have a display logic).

This kind of scenarios are now detected in the change event of the OBSelectorItem class, when the value to be updated is null but at the same time the pick list already contains a selected value.

---
M modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-item.js
---
(0083711)
hgbot   
2016-01-29 09:17   
Repository: erp/devel/pi
Changeset: 759c92f6e8a28c69180c938f4ea621286b765b16
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Fri Jan 29 09:17:02 2016 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/759c92f6e8a28c69180c938f4ea621286b765b16 [^]

related to bug 32007: use single qoutes to enclose js strings

---
M modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-item.js
---
(0083712)
alostale   
2016-01-29 09:18   
code reviewed + tested
(0085084)
hudsonbot   
2016-03-17 10:55   
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/b22fb0500156 [^]
Maturity status: Test
(0085086)
hudsonbot   
2016-03-17 10:55   
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/b22fb0500156 [^]
Maturity status: Test