svn commit: r15444 - trunk/src/argouml-app/src/org/argouml/uml/ui/ActionDeleteModelElements.java

Author: bobtarling
Date: 2008-08-01 17:23:23-0700
New Revision: 15444

Modified:
   trunk/src/argouml-app/src/org/argouml/uml/ui/ActionDeleteModelElements.java

Log:
Issue 5270: Don't allow deletion of readonly model element

Modified: trunk/src/argouml-app/src/org/argouml/uml/ui/ActionDeleteModelElements.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/ui/ActionDeleteModelElements.java?view=diff&rev=15444&p1=trunk/src/argouml-app/src/org/argouml/uml/ui/ActionDeleteModelElements.java&p2=trunk/src/argouml-app/src/org/argouml/uml/ui/ActionDeleteModelElements.java&r1=15443&r2=15444
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/ui/ActionDeleteModelElements.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/ui/ActionDeleteModelElements.java	2008-08-01 17:23:23-0700
@@ -267,6 +267,14 @@
      * @return true if the tool should be enabled
      */
     public boolean shouldBeEnabled() {
+        List targets = TargetManager.getInstance().getTargets();
+        for (Object target : targets) {
+            if (Model.getFacade().isAModelElement(target)
+                    && Model.getModelManagementHelper().isReadOnly(target)) {
+                return false;
+            }
+        }
+        
         int size = 0;
         try {
             Editor ce = Globals.curEditor();
@@ -280,6 +288,8 @@
         if (size > 0) {
             return true;
         }
+        // TODO: All of the following can be broken if we have multiple
+        // targets selected
         Object target = TargetManager.getInstance().getTarget();
         if (target instanceof ArgoDiagram) { 
             // we cannot delete the last diagram