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

Bug 357400

Summary: CDOPushTransaction.isDirty() should call delegate.isDirty()
Product: [Modeling] EMF Reporter: Alex Lagarde <alex.lagarde>
Component: cdo.coreAssignee: Eike Stepper <stepper>
Status: CLOSED FIXED QA Contact: Eike Stepper <stepper>
Severity: minor    
Priority: P3    
Version: 4.1   
Target Milestone: ---   
Hardware: PC   
OS: Windows 7   
Whiteboard:

Description Alex Lagarde CLA 2011-09-12 12:32:35 EDT
Hi Everyone ! 

A minor bug that is very easy to work around : when using the new remote locking capabilities of CDO by opening a transaction using its durable locking ID, if I create a push transaction and import changes from a file, the following state occurs : pushTransaction.isDirty() return falses although pushTransaction.getDelegate().isDirty() returns true.

It seems raisonnable to add || delegate.isDirty() in the CDOPushTransaction.isDirty() method.


What do you think ?
Comment 1 Eike Stepper CLA 2011-09-19 02:27:35 EDT
Analyzing in 4.1 first and prting back to 4.0 eventually.
Comment 2 Eike Stepper CLA 2011-09-19 02:37:07 EDT
Fix for 4.0 is in bug 358055.
Comment 3 Eike Stepper CLA 2011-09-19 02:40:58 EDT
Committed revision 9171
Comment 4 Eike Stepper CLA 2012-09-21 07:17:54 EDT
Closing.