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

Bug 336314

Summary: Partial commits choke on CDOMoveFeatureDeltas
Product: [Modeling] EMF Reporter: Caspar D. <caspar_d>
Component: cdo.coreAssignee: Caspar D. <caspar_d>
Status: CLOSED FIXED QA Contact: Eike Stepper <stepper>
Severity: normal    
Priority: P3 CC: saulius.tvarijonas
Version: 4.0Flags: stepper: review+
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Testcase (as patch)
none
Patch (including testcase)
none
Patch v2 none

Description Caspar D. CLA 2011-02-04 01:57:56 EST 
The logic in CommitIntegrityCheck.checkFeatureDelta doesn't
expect to encounter a CDOMoveFeatureDelta. When it does,
it throws a runtime ex.
Comment 1 Caspar D. CLA 2011-02-04 02:01:19 EST 
Created attachment 188292 [details]
Testcase (as patch)
Comment 2 Caspar D. CLA 2011-02-04 02:12:26 EST 
Created attachment 188293 [details]
Patch (including testcase)
Comment 3 Caspar D. CLA 2011-02-04 02:14:13 EST 
Forgot to include the test in the suite.. Will add that change
on approval.
Comment 4 Eike Stepper CLA 2011-02-04 03:13:57 EST 
Created attachment 188294 [details]
Patch v2

Changed the RuntimeException into an IllegalArgumentException and added to AllConfigs. Please commit.
Comment 5 Caspar D. CLA 2011-02-04 05:06:43 EST 
Committed to trunk, rev. 7019
Comment 6 Eike Stepper CLA 2011-06-23 03:41:26 EDT 
Available in R20110608-1407