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

Bug 350134

Summary: [Palette] Can't delete local palette definition
Product: [Modeling] Papyrus Reporter: Alexandre Cortier <alexandre.cortier>
Component: DiagramAssignee: Gabriel Pascual <gabriel.pascual>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: cletavernier, gabriel.pascual, klaas.gadeyne, papyrus-bugs, rschnekenburger, yann.tanguy
Version: 1.0.0   
Target Milestone: SR1   
Hardware: All   
OS: All   
Whiteboard:

Description Alexandre Cortier CLA 2011-06-23 08:20:22 EDT
Build Identifier: 20110609-2236

In the customization palette wizard, the "Delete selected local palette definition" action don't work.
An error window opens : "Bad selection (no item or bad item) - Please select a local palette to be deleted" 

I tried to delete the .xml local palette definitions in my workspace ".metadata\.plugins\org.eclipse.papyrus.diagram.common". When I reopen my workspace, my palettes are still visible in the customization wizard : a new (and empty) .xml palette definition file is automatically created (with the same old identifier).


Reproducible: Always

Steps to Reproduce:
1. Open the customization palette wizard
2. Select a local palette definition
3. Click on "Delete selected local palette definition"
Comment 1 Klaas Gadeyne CLA 2014-04-22 09:35:32 EDT
Still the case in the latest nightly builds...
Comment 2 Gabriel Pascual CLA 2014-05-07 01:15:05 EDT
Today, the associated action with delete button is  default extended palette restoration. In state, the delete action method is never called. 

To fix this bug, it's necessary to know what is the normal behavior for method for restoring default extended palette because this is it which open message dialog.
Comment 3 Gabriel Pascual CLA 2014-05-19 10:57:56 EDT
Gerrit review is available at https://git.eclipse.org/r/26867

This patch fixes bad behavior of deleting local palette and add a button for restoring default configuration of extended palette. Both actions have been dissociated.
Comment 4 Camille Letavernier CLA 2014-08-19 11:54:24 EDT
The contribution has been merged to master; I close the taskS