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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0024287
TypeCategorySeverityReproducibilityDate SubmittedLast Update
design defect[Openbravo ERP] A. Platformmajoralways2013-07-08 19:192017-03-15 20:18
ReporterXpand-ITView Statuspublic 
Assigned Toplatform 
PrioritynormalResolutionopenFixed in Version3.0PR17Q2
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSWindowsDatabasePostgreSQLJava version
OS Version7Database versionAnt version
Product Version3.0MP24SCM revision 
Review Assigned Tocaristu
Web browserGoogle Chrome
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0024287: Event Handler runs 2 times when the change is made by another Event Handler

DescriptionWhen an handler changes a child instance with an handler, this second handler runs 2 times.
Steps To ReproduceCreate an Handler for an entity that changes a child entity.
Create an Handler for that child entity.
Change the parent entity.
The second handler runs 2 times.
TagsNo tags attached.
Attached Filesdiff file icon testIssue24287.diff [^] (5,353 bytes) 2013-07-31 14:03 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
duplicate of defect 0035060 closedalostale EntityPersistanceEventObservers are executed twice on commitAndClose 
related to defect 0022011 closedmtaal Can't change Invoice Lines in a Business Handler when the Invoice is changed 
related to design defect 0027903 newTriage Platform Base Not possible to assign a characteristic subset having a tab under the [Characteristic Configuration] tab 
related to design defect 0029477 acknowledgedTriage Platform Base It is necessary to mark the link to parent checkbox to add a child instance from an event handler 

-  Notes
(0059930)
Xpand-IT (reporter)
2013-07-08 19:20

This issue may be related to https://issues.openbravo.com/view.php?id=22011 [^]
(0060423)
AugustoMauch (manager)
2013-07-31 14:08

To test it:
- Apply the attached patch. It will create two event handlers, one for the Country entity and another one for its Region child entity. The Country event handler will modify the description of one of its region by appending an underscore, if it has any. The Region event handler will has log the regions name
- As System Administrator, open the Country and Region window.
- Select a country with regions (i.e. United Kingdom) and change any of its fields (i.e. IBAN length) and save. Notice that the Region event handler is executed twice (the same line is added twice to the log).
(0093938)
hgbot (developer)
2017-01-30 16:24

Repository: erp/devel/pi
Changeset: 1c88fb99f5433041ad708aac2b65afa05d2aec5f
Author: Carlos Aristu <carlos.aristu <at> openbravo.com>
Date: Mon Jan 30 16:19:24 2017 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/1c88fb99f5433041ad708aac2b65afa05d2aec5f [^]

related to bug 24287: observer run once when changes come from another observer
The fix for issue 0035060 is solving also issue 0024287. With this changeset, we are adding two new test cases to cover the scenario of an observed being executed because of changes done with another observer. In those two cases, we are checking that each observer is being executed just once.

---
M modules/org.openbravo.client.application/src-test/org/openbravo/client/application/test/event/DatasourceEventObserver.java
M modules/org.openbravo.client.application/src-test/org/openbravo/client/application/test/event/ObserverBaseTest.java
M modules/org.openbravo.client.application/src-test/org/openbravo/client/application/test/event/OrderLineTestObserver.java
A modules/org.openbravo.client.application/src-test/org/openbravo/client/application/test/event/OrderTestObserver.java
---
(0095139)
hudsonbot (developer)
2017-03-15 20: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/54e102bef53e [^]
Maturity status: Test

- Issue History
Date Modified Username Field Change
2013-07-08 19:19 Xpand-IT New Issue
2013-07-08 19:19 Xpand-IT Assigned To => AugustoMauch
2013-07-08 19:19 Xpand-IT Web browser => Google Chrome
2013-07-08 19:19 Xpand-IT Modules => Core
2013-07-08 19:19 Xpand-IT Triggers an Emergency Pack => No
2013-07-08 19:20 Xpand-IT Note Added: 0059930
2013-07-31 14:03 AugustoMauch File Added: testIssue24287.diff
2013-07-31 14:08 AugustoMauch Note Added: 0060423
2013-08-01 09:45 shankarb Assigned To AugustoMauch => shankarb
2014-09-03 08:14 shankarb Assigned To shankarb => AugustoMauch
2015-01-27 10:25 alostale Relationship added related to 0022011
2015-01-27 10:26 alostale Relationship added related to 0027903
2015-03-17 14:38 alostale Assigned To AugustoMauch => platform
2015-11-10 14:33 alostale Web browser Google Chrome => Google Chrome
2015-11-10 14:33 alostale Status new => acknowledged
2015-11-10 14:33 alostale Type defect => design defect
2015-11-10 14:34 alostale Relationship added related to 0029477
2017-01-26 14:21 alostale Relationship added related to 0035060
2017-01-30 16:24 hgbot Checkin
2017-01-30 16:24 hgbot Note Added: 0093938
2017-01-30 16:25 caristu Status acknowledged => scheduled
2017-01-30 16:27 caristu Review Assigned To => caristu
2017-01-30 16:27 caristu Status scheduled => closed
2017-01-30 16:27 caristu Web browser Google Chrome => Google Chrome
2017-01-30 16:27 caristu Target Version => 3.0PR17Q2
2017-01-30 16:27 caristu Web browser Google Chrome => Google Chrome
2017-01-30 16:27 caristu Fixed in Version => 3.0PR17Q2
2017-01-30 16:27 caristu Target Version 3.0PR17Q2 =>
2017-01-30 16:28 caristu Relationship deleted related to 0035060
2017-01-30 16:28 caristu Relationship added duplicate of 0035060
2017-03-15 20:18 hudsonbot Checkin
2017-03-15 20:18 hudsonbot Note Added: 0095139


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker