Bug 245111 - [tml][deviceframework] Disable the "Delete" option in popup if the instance is not prepared for deletion
Summary: [tml][deviceframework] Disable the "Delete" option in popup if the instance i...
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Sequoyah (show other bugs)
Version: unspecified   Edit
Hardware: PC All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Fabio Fantato CLA
QA Contact:
URL:
Whiteboard: released_0.1
Keywords: contributed
Depends on:
Blocks: 246082
  Show dependency tree
 
Reported: 2008-08-25 09:05 EDT by Fabio Rigo CLA
Modified: 2016-11-08 14:46 EST (History)
3 users (show)

See Also:


Attachments
Suggestion for controlling if deletion is possible at some state (10.00 KB, patch)
2008-09-03 09:23 EDT, Fabio Rigo CLA
fabio.fantato: iplog+
fabio.fantato: review+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Fabio Rigo CLA 2008-08-25 09:05:32 EDT
It should be possible to disable the delete option in the instance popup menu if the instance is in a status not appropriate for deletion.
Comment 1 Fabio Rigo CLA 2008-09-03 09:23:14 EDT
Created attachment 111578 [details]
Suggestion for controlling if deletion is possible at some state

This patch adds a new attribute at the Status extension point, so that the declarer can forbid the deletion operation to happen when the device is at that state.
Comment 2 Fabio Rigo CLA 2008-09-03 09:27:20 EDT
I have attached a suggestion for solving this bug. In the proposal, a "canDelete" flag was added to the Status declaration, which is kept inside the model. The instance view was updated to use this information for enabling/disabling the "Delete" item in the instance context menu. 

This is a note to clarify the licensing issues:
I wrote the code of all new classes that are contained in the archive. I has
the right to contribute the code to Eclipse. 
Comment 3 Fabio Rigo CLA 2008-09-03 09:41:02 EDT
One additional note: 
I, Fabio Rigo, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. I am authorized by my employer to make this contribution under the EPL.
Comment 4 Fabio Fantato CLA 2008-09-03 13:11:16 EDT
Code revised and patch applied
Comment 5 Fabio Fantato CLA 2008-09-05 13:34:06 EDT
integrated to version 0.1
Comment 6 Fabio Fantato CLA 2008-11-13 08:18:27 EST
CVS branch for 0.1 release: branch_dsdp_tml_official_0_1_0_R