Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 363287 - CDODeltaNotification.getOldValue() returns a CDOIDObjectLongImpl on remotely detached CDOObject
Summary: CDODeltaNotification.getOldValue() returns a CDOIDObjectLongImpl on remotely ...
Status: CLOSED FIXED
Alias: None
Product: EMF
Classification: Modeling
Component: cdo.core (show other bugs)
Version: 4.1   Edit
Hardware: PC Linux
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Eike Stepper CLA
QA Contact: Eike Stepper CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-09 06:24 EST by Esteban DUGUEPEROUX CLA
Modified: 2012-09-21 07:15 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Esteban DUGUEPEROUX CLA 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 CLA 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 CLA 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 CLA 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 CLA 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 CLA 2012-09-21 07:15:51 EDT 
Closing.