Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0018200
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 09. Financial managementmajoralways2011-08-04 14:192011-08-06 05:33
Reporterdmitry_mezentsevView Statuspublic 
Assigned Toiperdomo 
PriorityimmediateResolutionfixedFixed in Version
StatusclosedFix in branchpiFixed in SCM revision7fc042749dd1
ProjectionnoneETAnoneTarget Version3.0MP2
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Review Assigned To
Web browser
ModulesAdvanced Payables and Receivables Mngmt
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0018200: 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.
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0018044 closedmirurita If the user cancel the "Match using imported bank statement lines" after doing a partial matching, system is not working right. 

-  Notes
(0039873)
hgbot (developer)
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 (developer)
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 (administrator)
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 (developer)
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 (developer)
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

- Issue History
Date Modified Username Field Change
2011-08-04 14:19 dmitry_mezentsev New Issue
2011-08-04 14:19 dmitry_mezentsev Assigned To => iperdomo
2011-08-04 14:19 dmitry_mezentsev Modules => Advanced Payables and Receivables Mngmt
2011-08-04 17:32 iperdomo Status new => scheduled
2011-08-04 17:32 iperdomo fix_in_branch => pi
2011-08-04 17:39 hgbot Checkin
2011-08-04 17:39 hgbot Note Added: 0039873
2011-08-04 17:39 hgbot Checkin
2011-08-04 17:39 hgbot Note Added: 0039874
2011-08-04 17:39 hgbot Status scheduled => resolved
2011-08-04 17:39 hgbot Resolution open => fixed
2011-08-04 17:39 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/7fc042749dd15f9ee2a248dbee32d3286c9d9260 [^]
2011-08-05 16:59 shuehner Note Added: 0039914
2011-08-05 16:59 shuehner Status resolved => closed
2011-08-06 05:33 hudsonbot Checkin
2011-08-06 05:33 hudsonbot Note Added: 0039968
2011-08-06 05:33 hudsonbot Checkin
2011-08-06 05:33 hudsonbot Note Added: 0039969
2011-08-09 08:43 mirurita Relationship added related to 0018044

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker