Bug 410310 - [core] Model remains dirty after save

1 files changed, 9 insertions, 0 deletions

diff --git a/plugins/infra/gmfdiag/org.eclipse.papyrus.infra.gmfdiag.commands/src/org/eclipse/papyrus/commands/NotifyingWorkspaceCommandStack.java b/plugins/infra/gmfdiag/org.eclipse.papyrus.infra.gmfdiag.commands/src/org/eclipse/papyrus/commands/NotifyingWorkspaceCommandStack.java
index 42347076cd8..71779811ed0 100644
--- a/plugins/infra/gmfdiag/org.eclipse.papyrus.infra.gmfdiag.commands/src/org/eclipse/papyrus/commands/NotifyingWorkspaceCommandStack.java
+++ b/plugins/infra/gmfdiag/org.eclipse.papyrus.infra.gmfdiag.commands/src/org/eclipse/papyrus/commands/NotifyingWorkspaceCommandStack.java
@@ -607,16 +607,25 @@ implements IWorkspaceCommandStack {
 		// pass

 	}

 

+

 	@Override

 	public boolean isSaveNeeded() {

+		// Bug 410310 - [core] Model remains dirty after save

+		// seems that super class implementation is good enough

+		// => deactivate implementation below and use the standard from the superclass

+		return super.isSaveNeeded();

 		// We override the execute method and never call the super

 		// implementation

 		// so we have to implement the isSaveNeeded method ourselves.

+		/*

 		IUndoableOperation nextUndoableOperation = history.getUndoOperation(getDefaultUndoContext());

 		if(nextUndoableOperation == null) {

+			// this is the last undoable operation. But the document might have been save

+			// CAVEAT: will trigger 410310, if the model has been saved before

 			return savedContext != null;

 		}

 		return savedContext != null ? !nextUndoableOperation.hasContext(getSavedContext()) : true;

+		*/

 	}

 

 	@Override