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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0032308
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformmajorhave not tried2016-02-23 10:552016-03-17 10:56
ReporteralostaleView Statuspublic 
Assigned Toalostale 
PriorityimmediateResolutionfixedFixed in Version3.0PR16Q2
StatusclosedFix in branchFixed in SCM revision64dcdb9cf68b
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tocaristu
Web browser
ModulesCore
Regression levelProduction - QA Approved
Regression date2015-07-03
Regression introduced in release3.0PR15Q4
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/b26a24d2053e04743e90c430dc6f14098348651b [^]
Triggers an Emergency PackNo
Summary

0032308: error on update if entity persistence observer loaded current object in memory

DescriptionIf an entity persistence observer loas current object in memory, an error is thrown when trying to perform an update from UI.
Steps To Reproduce-Add to attached OrderLineObserver.java to sources and compile
-Try to edit any existent order line.
  -> The following error can be seen:
    Saving failed. a different object with the same identifier value was already associated with the session: [OrderLine#7517250EF33F451CAF1CFF9A1CA5FEC9]
TagsNo tags attached.
Attached Files? file icon OrderLineObserver.java [^] (984 bytes) 2016-02-23 11:08

- Relationships [ Relation Graph ] [ Dependency Graph ]
depends on backport 00323093.0PR16Q1 closedalostale error on update if entity persistence observer loaded current object in memory 
depends on backport 00323103.0PR15Q4.4 closedalostale error on update if entity persistence observer loaded current object in memory 
caused by defect 00303083.0PR15Q4 closedcaristu Use get instead of refresh when saving or updating a record 
related to defect 0035060 closedalostale EntityPersistanceEventObservers are executed twice on commitAndClose 

-  Notes
(0084462)
hgbot (developer)
2016-02-24 15:03

 Repository: erp/devel/pi
Changeset: ea854655f9aa2e204bc2d7d51ead6364ea9c8e4f
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Feb 23 16:38:23 2016 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/ea854655f9aa2e204bc2d7d51ead6364ea9c8e4f [^]

related to bug 32308: added test cases covering observers with ds update

---
M src-test/src/org/openbravo/test/AllAntTaskTests.java
A modules/org.openbravo.client.application/src-test/org/openbravo/client/application/test/event/DatasourceEventObserver.java
A modules/org.openbravo.client.application/src-test/org/openbravo/client/application/test/event/OrderLineTestObserver.java
---
(0084463)
hgbot (developer)
2016-02-24 15:03

 Repository: erp/devel/pi
Changeset: 64dcdb9cf68b85655f7f634657eef6b685092cb5
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Feb 23 16:37:30 2016 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/64dcdb9cf68b85655f7f634657eef6b685092cb5 [^]

fixed bug 32308: error on update if an observer loaded current object in memory

  In case an entity observer loaded current object in memory (ie. observing orderLine)
  doing orderLine.getSalesOrder().getOrderLineList() there were two different instances
  in memory representing the same DB row. This caused problems when trying to evict it.
  Causing an exception to be thrown when updating from UI in this case.

  As solution eviction is no longer performed, object is forced to be fetched from DB by
  executing a Criteria.

---
M modules/org.openbravo.service.json/src/org/openbravo/service/json/DefaultJsonDataService.java
M src/org/openbravo/base/structure/BaseOBObject.java
---
(0084470)
caristu (developer)
2016-02-24 17:31

 Code review + testing OK
(0085164)
hudsonbot (developer)
2016-03-17 10:56

 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
(0085165)
hudsonbot (developer)
2016-03-17 10:56

 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

- Issue History
Date Modified Username Field Change
2016-02-23 10:55 alostale New Issue
2016-02-23 10:55 alostale Assigned To => platform
2016-02-23 10:55 alostale Modules => Core
2016-02-23 10:55 alostale Triggers an Emergency Pack => No
2016-02-23 10:57 alostale Regression level => Production - QA Approved
2016-02-23 10:57 alostale Regression date => 2015-07-03
2016-02-23 10:57 alostale Regression introduced in release => 3.0PR15Q4
2016-02-23 10:57 alostale Regression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/b26a24d2053e04743e90c430dc6f14098348651b [^]
2016-02-23 10:57 alostale Assigned To platform => alostale
2016-02-23 10:58 alostale Relationship added caused by 0030308
2016-02-23 10:58 alostale Status new => scheduled
2016-02-23 11:08 alostale File Added: OrderLineObserver.java
2016-02-24 15:02 alostale Review Assigned To => caristu
2016-02-24 15:03 hgbot Checkin
2016-02-24 15:03 hgbot Note Added: 0084462
2016-02-24 15:03 hgbot Checkin
2016-02-24 15:03 hgbot Note Added: 0084463
2016-02-24 15:03 hgbot Status scheduled => resolved
2016-02-24 15:03 hgbot Resolution open => fixed
2016-02-24 15:03 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/64dcdb9cf68b85655f7f634657eef6b685092cb5 [^]
2016-02-24 17:31 caristu Note Added: 0084470
2016-02-24 17:31 caristu Status resolved => closed
2016-02-24 17:31 caristu Fixed in Version => 3.0PR16Q2
2016-03-17 10:56 hudsonbot Checkin
2016-03-17 10:56 hudsonbot Note Added: 0085164
2016-03-17 10:56 hudsonbot Checkin
2016-03-17 10:56 hudsonbot Note Added: 0085165
2017-01-26 14:17 alostale Relationship added related to 0035060

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker