Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 363287

Summary:	CDODeltaNotification.getOldValue() returns a CDOIDObjectLongImpl on remotely detached CDOObject
Product:	[Modeling] EMF	Reporter:	Esteban DUGUEPEROUX <esteban.dugueperoux>
Component:	cdo.core	Assignee:	Eike Stepper <stepper>
Status:	CLOSED FIXED	QA Contact:	Eike Stepper <stepper>
Severity:	major
Priority:	P3	CC:	steve.monnier
Version:	4.1
Target Milestone:	---
Hardware:	PC
OS:	Linux
Whiteboard:

Description Esteban DUGUEPEROUX

2011-11-09 06:24:33 EST

Build Identifier: CDO 4.0

In CDODeltaNotification.adapt() method, we see that with a CDOIDObjectLong of a remotely detached CDOObject, the CDOView.getObject(id, true) throws a ObjectNotFoundException, but this exception is ignored silently. In the case we should returns null.

Reproducible: Always

Comment 1 Eike Stepper

2011-11-10 07:14:58 EST

commit 1f620cfdc202d0f8334fff39be7f8279e263d30f
Author: Eike Stepper <stepper@esc-net.de> 2011-11-10 13:14:05
Committer: Eike Stepper <stepper@esc-net.de> 2011-11-10 13:14:05
Parent: 831dd5cfb3a4b02cf0af8a3c8c0d0ce7431cda61 (Revert "Removed src-gen/ folder contents from source control")
Branches: origin/master, master

[363287] CDODeltaNotification.getOldValue() returns a CDOIDObjectLongImpl on remotely detached CDOObject 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=363287

Comment 2 Steve Monnier

2011-11-10 07:44:01 EST

Hello Eike,

Is it possible to have this fix on branch 4.0-maintenance? Should we create another issue for fix version 4.0?

Thanks,
Steve

Comment 3 Eike Stepper

2011-11-10 11:23:52 EST

I already opened bug 363460 and was about to merge the fix back to 4.0 but I struggeled with Git. Stay tuned...

Comment 4 Steve Monnier

2011-11-10 11:27:19 EST

(In reply to comment #3)
> I already opened bug 363460 and was about to merge the fix back to 4.0 but I
> struggeled with Git. Stay tuned...

Nice and good luck :)

Comment 5 Eike Stepper

2012-09-21 07:15:51 EDT

Closing.