Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 341897 - Papyrus diagram common plug-in breaks all GMF based diagram editors
Summary: Papyrus diagram common plug-in breaks all GMF based diagram editors
Status: RESOLVED FIXED
Alias: None
Product: Papyrus
Classification: Modeling
Component: Core (show other bugs)
Version: 0.7.1   Edit
Hardware: All All
: P3 blocker (vote)
Target Milestone: ---   Edit
Assignee: Remi Schnekenburger CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-05 09:03 EDT by Mariot Chauvin CLA
Modified: 2011-04-07 11:52 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 Mariot Chauvin CLA 2011-04-05 09:03:35 EDT 
The plug-in xml file of org.eclipse.papyrus.diagram.common breaks all GMF based diagram editors by not limit its contribution to Papyrus editor id (see http://help.eclipse.org/helios/index.jsp?topic=/org.eclipse.gmf.doc/tutorials/diagram/paletteConfigurationTutorial.html)

The removal of Note tools should be limited to the Papyrus editor.
Guilty extension is the following :

"<extension
      point="org.eclipse.gmf.runtime.diagram.ui.paletteProviders">
   <paletteProvider
         class="org.eclipse.gmf.runtime.diagram.ui.providers.DefaultPaletteProvider">
      <Priority
            name="Highest">
      </Priority>
      <contribution>
         <predefinedEntry
               id="standardGroup/noteStack/noteTool"
               remove="true">
         </predefinedEntry>
         <predefinedEntry
               id="standardGroup/noteStack/textTool"
               remove="true">
         </predefinedEntry>
         <predefinedEntry
               id="standardGroup/noteStack/noteattachmentTool"
               remove="true">
         </predefinedEntry>
      </contribution>
   </paletteProvider>
</extension>"
Comment 1 Remi Schnekenburger CLA 2011-04-07 09:52:56 EDT 
fixed in branch 0.7.X (r4440) and trunk (r4441).
Comment 2 Mariot Chauvin CLA 2011-04-07 11:52:09 EDT 
(In reply to comment #1)
> fixed in branch 0.7.X (r4440) and trunk (r4441).

Thanks for the quick fix !