Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0025558Openbravo ERPA. Platformpublic2014-01-24 15:502014-03-21 10:23
egoitz 
AugustoMauch 
highminoralways
closedfixed 
5
 
3.0PR14Q23.0PR14Q3 
alostale
Core
No
0025558: The callout of the combos on manual forms does not work if you have opened previously other form
The callout of the combos on the form does not work if you have opened previously another form, for example, the enterprise module managemente window
-Open the Product movements report
-Find a product with stock having also attribute set.
-See that after select the attribute combo is filled.
-close the form
-Open the enterprise module management window.
-Open again the Product movements report
-Find a product with stock having also attribute set.
*The callout does not work so the combo of attributes does not who values
No tags attached.
related to defect 00270173.0PR14Q4 closed AugustoMauch The callout of the combos on manual forms does not work if you have opened previously other form 
Issue History
2014-01-24 15:50egoitzNew Issue
2014-01-24 15:50egoitzAssigned To => jonalegriaesarte
2014-02-03 13:23egoitzProjectModules => Openbravo ERP
2014-02-03 13:27egoitzReview Assigned To => AugustoMauch
2014-02-03 13:27egoitzModules => Core
2014-02-03 13:27egoitzResolution time => 1392418800
2014-02-03 13:27egoitzTriggers an Emergency Pack => No
2014-02-03 13:27egoitzCategoryTax Report Launcher => A. Platform
2014-02-03 13:27egoitzSummaryThe callout of the combos on the form does not work if you have opened previously the enterprise module managemente window => The callout of the combos on manual forms does not work if you have opened previously other form
2014-02-03 13:27egoitzDescription Updatedbug_revision_view_page.php?rev_id=5409#r5409
2014-02-03 13:27egoitzSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=5411#r5411
2014-02-03 20:54egoitzNote Added: 0063876
2014-02-10 10:37egoitzAssigned Tojonalegriaesarte => AugustoMauch
2014-02-10 10:37egoitzTarget Version => 3.0MP32
2014-03-10 15:24alostaleAssigned ToAugustoMauch => szapata
2014-03-12 10:21jonalegriaesarteResolution time1392418800 => 1394838000
2014-03-15 14:39szapataNote Added: 0065190
2014-03-17 08:28alostaleAssigned Toszapata => AugustoMauch
2014-03-18 17:19AugustoMauchStatusnew => scheduled
2014-03-18 17:19AugustoMauchfix_in_branch => pi
2014-03-18 17:26AugustoMauchIssue Monitored: alostale
2014-03-18 21:05hgbotCheckin
2014-03-18 21:05hgbotNote Added: 0065277
2014-03-18 21:05hgbotStatusscheduled => resolved
2014-03-18 21:05hgbotResolutionopen => fixed
2014-03-18 21:05hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/05acae86769bfc1e346a48c561e918b91e31efb5 [^]
2014-03-18 21:17AugustoMauchReview Assigned ToAugustoMauch => alostale
2014-03-20 17:18hudsonbotCheckin
2014-03-20 17:18hudsonbotNote Added: 0065311
2014-03-21 10:23alostaleNote Added: 0065344
2014-03-21 10:23alostaleStatusresolved => closed
2014-03-21 10:23alostaleFixed in Version => PR14Q3
2014-07-04 13:35caristuRelationship addedrelated to 0027017

Notes
(0063876)
egoitz   
2014-02-03 20:54   
This happens also with other forms that have callout.
For example with the Tax Report Launcher.
(0065190)
szapata   
2014-03-15 14:39   
Here is the situation:
1 You open the first form and the HTML of the entire window contains a frame called 'appFrame'
2 You open the second form and the HTML now contains two forms called 'appFrame' the src of the first one points to the first form and the second one to the second form.
3 You close the first form, so now the HTML only have one frame called 'appFrame' which was the second one, but now is the only one in the HTML.
4 You open again the form with the callout, and now the HTML has again two frames called 'appFrame', but the src property of the first appFrame points to the form you opened in step 2, so when you execute the callout, the response of the callout is injected in this src and the expected fields to be filled by the response does not exist in that frame.

Callouts response shouldn't be managed by the name of the frame only but should be managed by the entire Frame object or the combination of src+frame name.
(0065277)
hgbot   
2014-03-18 21:05   
Repository: erp/devel/pi
Changeset: 05acae86769bfc1e346a48c561e918b91e31efb5
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Tue Mar 18 17:23:09 2014 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/05acae86769bfc1e346a48c561e918b91e31efb5 [^]

Fixes issue 25558: Fixes callout of manual forms

The code that applies the response of the manual forms callout was taking for granted that there one only one frame called appFrame. This was true in Openbravo 2.50, but in Openbravo 3.0 many manual windows can be opened at the same time, so there may be several frames called appFrame in one given moment.

This has been fixed by storing the active manual frame (the one whose tab is focused), and using that frame when applying the response of the callout.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/classic/ob-classic-window.js
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/main/ob-tab.js
M src/org/openbravo/erpCommon/security/Login.html
M src/org/openbravo/erpCommon/security/Login_F1.html
M web/js/callOut.js
M web/js/utils.js
---
(0065311)
hudsonbot   
2014-03-20 17:18   
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/614f58fa156d [^]
Maturity status: Test
(0065344)
alostale   
2014-03-21 10:23   
Code reviewed + tested