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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0035075
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformtrivialhave not tried2017-01-30 09:012017-03-15 20:18
ReporteralostaleView Statuspublic 
Assigned Toalostale 
PrioritynormalResolutionfixedFixed in Version3.0PR17Q2
StatusclosedFix in branchFixed in SCM revision704ea44ad8ad
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tocaristu
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0035075: PersistanceObserver can fail if other observers for OrderLine are present

DescriptionPersistanceObserver test cases fail if there are observers for new order line assuming order is set on save.

The problem is caused because these test cases save partially created order lines that don't link to any order just to check save observer is executed, because there are other observers with this assumption they throw NPE.
Steps To Reproduce1. Install org.openbravo.materialmgmt.freightmgmt
2. Execute PersistanceObserver
  -> It fails with NPE
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0035060 closedalostale EntityPersistanceEventObservers are executed twice on commitAndClose 
related to defect 0035076 closedalostale if test leaves admin mode, following test cases fail 

-  Notes
(0093914)
hgbot (developer)
2017-01-30 09:24

 Repository: erp/devel/pi
Changeset: 704ea44ad8adc758f461faea40ebce3cdbaf354c
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Mon Jan 30 09:04:49 2017 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/704ea44ad8adc758f461faea40ebce3cdbaf354c [^]

fixed bug 35075: PersistanceObserver fails if there're observers for OrderLine

  When there are other observers for order line assuming order line has order set
  on save, they can fail.

  Now when new order lines are created, a random order is assigned to them.

---
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/PersistanceObserver.java
---
(0093921)
caristu (developer)
2017-01-30 10:56

 Code reviewed + tested OK.
(0095135)
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
2017-01-30 09:01 alostale New Issue
2017-01-30 09:01 alostale Assigned To => platform
2017-01-30 09:01 alostale Modules => Core
2017-01-30 09:01 alostale Triggers an Emergency Pack => No
2017-01-30 09:01 alostale Relationship added related to 0035060
2017-01-30 09:02 alostale Assigned To platform => alostale
2017-01-30 09:16 alostale Review Assigned To => caristu
2017-01-30 09:20 alostale Relationship added related to 0035076
2017-01-30 09:24 hgbot Checkin
2017-01-30 09:24 hgbot Note Added: 0093914
2017-01-30 09:24 hgbot Status new => resolved
2017-01-30 09:24 hgbot Resolution open => fixed
2017-01-30 09:24 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/704ea44ad8adc758f461faea40ebce3cdbaf354c [^]
2017-01-30 10:56 caristu Note Added: 0093921
2017-01-30 10:56 caristu Status resolved => closed
2017-01-30 10:56 caristu Fixed in Version => 3.0PR17Q2
2017-03-15 20:18 hudsonbot Checkin
2017-03-15 20:18 hudsonbot Note Added: 0095135

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker