Skip to main content
aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristian W. Damus2014-02-13 14:01:21 +0000
committerChristian W. Damus2014-02-13 14:02:20 +0000
commitb0b8ee5723af195d9cf7e24d759fe864a2475aff (patch)
tree8375dc5879d98caf4ddf2c87afe270ec9a8c4be6
parent653b0ea45e942f9115e9f61d6b6b709181f791ba (diff)
downloadorg.eclipse.papyrus-b0b8ee5723af195d9cf7e24d759fe864a2475aff.tar.gz
org.eclipse.papyrus-b0b8ee5723af195d9cf7e24d759fe864a2475aff.tar.xz
org.eclipse.papyrus-b0b8ee5723af195d9cf7e24d759fe864a2475aff.zip
357250: [General] Application reacts different ways when trying to modify a read-only resource
https://bugs.eclipse.org/bugs/show_bug.cgi?id=357250 Fix the extension point namespace from which we load operation approvers.
-rw-r--r--plugins/infra/gmfdiag/org.eclipse.papyrus.infra.gmfdiag.commands/src/org/eclipse/papyrus/commands/CheckedOperationHistory.java10
1 files changed, 5 insertions, 5 deletions
diff --git a/plugins/infra/gmfdiag/org.eclipse.papyrus.infra.gmfdiag.commands/src/org/eclipse/papyrus/commands/CheckedOperationHistory.java b/plugins/infra/gmfdiag/org.eclipse.papyrus.infra.gmfdiag.commands/src/org/eclipse/papyrus/commands/CheckedOperationHistory.java
index 4fd07f68e65..5973347c169 100644
--- a/plugins/infra/gmfdiag/org.eclipse.papyrus.infra.gmfdiag.commands/src/org/eclipse/papyrus/commands/CheckedOperationHistory.java
+++ b/plugins/infra/gmfdiag/org.eclipse.papyrus.infra.gmfdiag.commands/src/org/eclipse/papyrus/commands/CheckedOperationHistory.java
@@ -1,5 +1,5 @@
/*****************************************************************************
- * Copyright (c) 2011 Atos.
+ * Copyright (c) 2011, 2014 Atos, CEA, and others.
*
*
* All rights reserved. This program and the accompanying materials
@@ -9,6 +9,7 @@
*
* Contributors:
* Mathieu Velten (Atos) - Initial API and implementation
+ * Christian W. Damus (CEA) - bug 357250
*
*****************************************************************************/
package org.eclipse.papyrus.commands;
@@ -72,11 +73,11 @@ public class CheckedOperationHistory implements IOperationHistory {
}
static {
- IConfigurationElement[] configElements = Platform.getExtensionRegistry().getConfigurationElementsFor("org.eclipse.papyrus.commands", "operationApprover");
+ IConfigurationElement[] configElements = Platform.getExtensionRegistry().getConfigurationElementsFor(Activator.PLUGIN_ID, "operationApprover"); //$NON-NLS-1$
List<ApproverPriorityPair> approverPriorityPairs = new LinkedList<ApproverPriorityPair>();
for(IConfigurationElement elem : configElements) {
- if("operationApprover".equals(elem.getName())) {
+ if("operationApprover".equals(elem.getName())) { //$NON-NLS-1$
try {
ApproverPriorityPair approverPriorityPair = new ApproverPriorityPair();
approverPriorityPair.approver = (IOperationApprover2)elem.createExecutableExtension("class");
@@ -84,8 +85,7 @@ public class CheckedOperationHistory implements IOperationHistory {
approverPriorityPairs.add(approverPriorityPair);
} catch (Exception e) {
- // TODO Auto-generated catch block
- e.printStackTrace();
+ Activator.log.error("Uncaught exception in instantiation of operation approver.", e); //$NON-NLS-1$
}
}
}

Back to the top