Fixed dirty state

Jacob van Mourik · Jacob van Mourik · commit e72858fe4ed8 · 2017-02-06T21:24:27.000+01:00
diff --git a/src/main/java/com/jvms/i18neditor/editor/Editor.java b/src/main/java/com/jvms/i18neditor/editor/Editor.java
@@ -208,7 +208,9 @@ public boolean saveProject() {
 				}
 			}
 		}
-		setDirty(error);
+		if (dirty) {
+			setDirty(error);			
+		}
 		return !error;
 	}
 	
@@ -486,7 +488,7 @@ public void showVersionDialog(boolean newVersionOnly) {
 	
 	public boolean closeCurrentProject() {
 		boolean result = true;
-		if (isDirty()) {
+		if (dirty) {
 			int confirm = JOptionPane.showConfirmDialog(this, 
 					MessageBundle.get("dialogs.save.text"), 
 					MessageBundle.get("dialogs.save.title"), 
@@ -500,6 +502,9 @@ public boolean closeCurrentProject() {
 		if (result && project != null) {
 			storeProjectState();
 		}
+		if (result && dirty) {
+			setDirty(false);
+		}
 		return result;
 	}
 	

Original file line number	Diff line number	Diff line change
`@@ -208,7 +208,9 @@ public boolean saveProject() {`
`208`	`208`	`}`
`209`	`209`	`}`
`210`	`210`	`}`
`211`		`- setDirty(error);`
	`211`	`+ if (dirty) {`
	`212`	`+ setDirty(error);`
	`213`	`+ }`
`212`	`214`	`return !error;`
`213`	`215`	`}`
`214`	`216`
`@@ -486,7 +488,7 @@ public void showVersionDialog(boolean newVersionOnly) {`
`486`	`488`
`487`	`489`	`public boolean closeCurrentProject() {`
`488`	`490`	`boolean result = true;`
`489`		`- if (isDirty()) {`
	`491`	`+ if (dirty) {`
`490`	`492`	`int confirm = JOptionPane.showConfirmDialog(this,`
`491`	`493`	`MessageBundle.get("dialogs.save.text"),`
`492`	`494`	`MessageBundle.get("dialogs.save.title"),`
`@@ -500,6 +502,9 @@ public boolean closeCurrentProject() {`
`500`	`502`	`if (result && project != null) {`
`501`	`503`	`storeProjectState();`
`502`	`504`	`}`
	`505`	`+ if (result && dirty) {`
	`506`	`+ setDirty(false);`
	`507`	`+ }`
`503`	`508`	`return result;`
`504`	`509`	`}`
`505`	`510`