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

Bug 311850

Summary: org.eclipse.compare.structureCreators extension point definition specifies wrong interface
Product: [Eclipse Project] Platform Reporter: Carsten Pfeiffer <carsten.pfeiffer>
Component: CompareAssignee: Tomasz Zarna <tomasz.zarna>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: pawel.pogorzelski1
Version: 3.5Flags: Szymon.Brandys: review+
pawel.pogorzelski1: review+
Target Milestone: 3.6 RC2   
Hardware: Other   
OS: Linux   
Whiteboard:
Attachments:
Description Flags
Fix v01
none
mylyn/context/zip
none
Fix v02 none

Description Carsten Pfeiffer CLA 2010-05-06 06:15:57 EDT 
Build Identifier: M20100211-1343

As document in the extension point, the class attribute specifies an element implementing org.eclipse.compare.structuremergeviewer.IStructureCreator, but the attribute itself specifies the interface org.eclipse.compare.structuremergeviewer.IStructureComparator to be implemented.

Reproducible: Always
Comment 1 Tomasz Zarna CLA 2010-05-12 09:31:29 EDT 
Good catch!
Comment 2 Tomasz Zarna CLA 2010-05-17 06:50:29 EDT 
Created attachment 168710 [details]
Fix v01

Fixed schema, now requires org.eclipse.compare.structuremergeviewer.IStructureCreator to be implemented.
Comment 3 Tomasz Zarna CLA 2010-05-17 06:50:31 EDT 
Created attachment 168711 [details]
mylyn/context/zip
Comment 4 Szymon Brandys CLA 2010-05-17 09:20:17 EDT 
I found the same issue in the ext. point overview.
Comment 5 Tomasz Zarna CLA 2010-05-17 09:41:37 EDT 
The overview is fine: the structure creator implementing org.eclipse.compare.structuremergeviewer.IStructureCreator is expected to create
a tree of org.eclipse.compare.structuremergeviewer.IStructureComparators.
Comment 6 Szymon Brandys CLA 2010-05-17 09:43:14 EDT 
Fair enough.
Comment 7 Tomasz Zarna CLA 2010-05-18 05:42:10 EDT 
Created attachment 168883 [details]
Fix v02

Fixed copyrights.
Comment 8 Tomasz Zarna CLA 2010-05-18 05:44:04 EDT 
Released to HEAD, available in builds >=I201005180-0800.