Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0018200Openbravo ERP09. Financial managementpublic2011-08-04 14:192011-08-06 05:33
Reporterdmitry_mezentsev 
Assigned Toiperdomo 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target Version3.0MP2Fixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesAdvanced Payables and Receivables Mngmt
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0018200: Match using imported bank statement lines pop-up when closing it reloads Financial Account window
DescriptionWhen you invoke another pop-up from "Match using imported bank statement lines" pop-up and then if you try to close it (using Cancel or Reconcile buttons) it does not close but loads Financial Account window inside the pop-up and in 2.50 mode (http://awesomescreenshot.com/011hx1d09 [^]).
Steps To Reproducelive.builds.pi (04.08.2011)
1. Financial Account. Create a new bank statement (bank statement tab) and bankstatementline, selecting a business partner and the amount and reference number. Process the bank statment.
2. Press "Match statement", press find, in the pop-up that appeared press Cancel and then press Cancel in the ""Match using imported bank statement lines" pop-up.
3. Observe how it loads Financial Account window inside it.

Actually now in live.builds.pi.postgre some setup is already done. If you go to Financial Account and select US Bank - Account 1, you can directly press "Match statement", pop-up will appear, press Find link in a grid there, then Cancel second pop-up, Cancel "Match using imported bank statement lines" and you´ll see the issue.
Proposed SolutionMikel mentioned that from their side they were not changing this part for quite a bit so they suspect it to come due to some Platform change.
Additional Information
TagsNo tags attached.
Relationships
related to defect 0018044 closed mirurita If the user cancel the "Match using imported bank statement lines" after doing a partial matching, system is not working right. 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-08-04 14:19dmitry_mezentsevNew Issue
2011-08-04 14:19dmitry_mezentsevAssigned To => iperdomo
2011-08-04 14:19dmitry_mezentsevModules => Advanced Payables and Receivables Mngmt
2011-08-04 17:32iperdomoStatusnew => scheduled
2011-08-04 17:32iperdomofix_in_branch => pi
2011-08-04 17:39hgbotCheckin
2011-08-04 17:39hgbotNote Added: 0039873
2011-08-04 17:39hgbotCheckin
2011-08-04 17:39hgbotNote Added: 0039874
2011-08-04 17:39hgbotStatusscheduled => resolved
2011-08-04 17:39hgbotResolutionopen => fixed
2011-08-04 17:39hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/7fc042749dd15f9ee2a248dbee32d3286c9d9260 [^]
2011-08-05 16:59shuehnerNote Added: 0039914
2011-08-05 16:59shuehnerStatusresolved => closed
2011-08-06 05:33hudsonbotCheckin
2011-08-06 05:33hudsonbotNote Added: 0039968
2011-08-06 05:33hudsonbotCheckin
2011-08-06 05:33hudsonbotNote Added: 0039969
2011-08-09 08:43miruritaRelationship addedrelated to 0018044

Notes
(0039873)
hgbot   
2011-08-04 17:39   
Repository: erp/devel/pi
Changeset: 0b16f9819a57dfbebae10fa91cc9453d45137c69
Author: Iván Perdomo <ivan.perdomo <at> openbravo.com>
Date: Thu Aug 04 17:34:41 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/0b16f9819a57dfbebae10fa91cc9453d45137c69 [^]

Related to issue 18200: Avoid useless to the same expensive method getFrame()
- Cache the value of getFrame('LayoutMDI') in a local variable and use it.

---
M src/org/openbravo/base/secureApp/PopUp_Response.html
---
(0039874)
hgbot   
2011-08-04 17:39   
Repository: erp/devel/pi
Changeset: 7fc042749dd15f9ee2a248dbee32d3286c9d9260
Author: Iván Perdomo <ivan.perdomo <at> openbravo.com>
Date: Thu Aug 04 17:38:26 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/7fc042749dd15f9ee2a248dbee32d3286c9d9260 [^]

Fixes issue 18200: Define the target when using submitCommandForm
- The 5th parameter of submitCommandForm is the target where the POST action
  should take place. If you don't use it it will use appFrame wich it seems to
  be unknown after opening a 2nd popup. In short: Stop making a mess opening
  op-ups from antother popup.

---
M modules/org.openbravo.advpaymentmngt/src/org/openbravo/advpaymentmngt/ad_actionbutton/MatchTransaction.html
---
(0039914)
shuehner   
2011-08-05 16:59   
Reviewed/tested on pi (rev id: 7e1aea78244f). Mentioned steps to reproduce now work fine on both chromium12 & FF5. First code change is performance/cleanup to avoid calling getFrame too many times. Second is fix as described in the cset itself.
(0039968)
hudsonbot   
2011-08-06 05: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/5744448753e7 [^]

Maturity status: Test
(0039969)
hudsonbot   
2011-08-06 05: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/5744448753e7 [^]

Maturity status: Test