From eba7b53e05b5e49710e4435235b55e432849c2fb Mon Sep 17 00:00:00 2001
From: hanhsu <hanhsu@potix.com>
Date: Thu, 9 Oct 2014 08:32:15 +0100
Subject: [PATCH] Fix Issue #9: unexpected onChange event after setting the
 value from the server

---
 ckez/src/archive/web/js/ckez/CKeditor.js       |  9 +++++++--
 .../ckeztest/src/archive/test2/B70-CKEZ-9.zul  | 18 ++++++++++++++++++
 .../src/archive/test2/config.properties        |  3 ++-
 zkdoc/ckez-release-note                        |  1 +
 4 files changed, 28 insertions(+), 3 deletions(-)
 create mode 100644 test/addon/ckeztest/src/archive/test2/B70-CKEZ-9.zul

diff --git a/ckez/src/archive/web/js/ckez/CKeditor.js b/ckez/src/archive/web/js/ckez/CKeditor.js
index 7f3a9521..7f82505a 100644
--- a/ckez/src/archive/web/js/ckez/CKeditor.js
+++ b/ckez/src/archive/web/js/ckez/CKeditor.js
@@ -21,10 +21,15 @@ ckez.CKeditor = zk.$extends(zul.Widget, {
 	$define: {
 		value: [function(v) {
 			return !v ? '' : v;
-		}, function (v) {
+		}, function (v, fromServer) {
 			var editor = this.getEditor();
-			if (editor)
+			if (editor) {
 				editor.setData(v);
+				// Issue #9: update editor's previousValue if set value from server
+				// to prevent unexpect onChange event
+				if (fromServer)
+					editor._.previousValue = v;
+			}
 		}],
 		autoHeight: null,
 		customConfigurationsPath: _zkf = function () {
diff --git a/test/addon/ckeztest/src/archive/test2/B70-CKEZ-9.zul b/test/addon/ckeztest/src/archive/test2/B70-CKEZ-9.zul
new file mode 100644
index 00000000..633a6525
--- /dev/null
+++ b/test/addon/ckeztest/src/archive/test2/B70-CKEZ-9.zul
@@ -0,0 +1,18 @@
+<zk>
+	<zscript>
+		String myValue = "initial value";
+		public updateMyValue(String newValue) {
+		myValue = newValue;
+		editor.setValue(myValue);
+		//Clients.showNotification("updated myValue to: " + myValue);
+		}
+	</zscript>
+	<vlayout>
+		<button label="1. update editor value to 'abc'" onClick='updateMyValue("&lt;p&gt;abc&lt;/p&gt;");' />
+		<button label="2. update editor value to 'XYZ'" onClick='updateMyValue("&lt;p&gt;XYZ&lt;/p&gt;");' />
+		<button label="3. do something unrelated and show 'myvalue'"
+			onClick='debugLabel.setValue("myValue: " + myValue);' />
+		<label id="debugLabel" />
+		<ckeditor id="editor" value="${myValue}" onChange='myValue = self.getValue(); alert("changed")' />
+	</vlayout>
+</zk>
\ No newline at end of file
diff --git a/test/addon/ckeztest/src/archive/test2/config.properties b/test/addon/ckeztest/src/archive/test2/config.properties
index f4d118ca..caf20a0e 100644
--- a/test/addon/ckeztest/src/archive/test2/config.properties
+++ b/test/addon/ckeztest/src/archive/test2/config.properties
@@ -64,4 +64,5 @@ B65-CKEZ-20.zul=Ckeditor,onChange
 B70-CKEZ-23.zul=Ckeditor,onChange
 B70-CKEZ-22.zul=Ckeditor,FileUpload
 B70-CKEZ-5.zul=Ckeditor,onSize
-B70-CKEZ-7.zul=A,E,Ckeditor,onChanging,FireEvent
\ No newline at end of file
+B70-CKEZ-7.zul=A,E,Ckeditor,onChanging,FireEvent
+B70-CKEZ-9.zul=A,E,Ckeditor,onChange,setValue
\ No newline at end of file
diff --git a/zkdoc/ckez-release-note b/zkdoc/ckez-release-note
index c05bd447..784c4ba1 100644
--- a/zkdoc/ckez-release-note
+++ b/zkdoc/ckez-release-note
@@ -6,6 +6,7 @@ Ckez 4.3.3.1
 * Bugs:
 	Issue #5: textarea of zkeditor not resizing properly
 	Issue #7: onChanging event not fired since ZK CKEditor 4.x
+	Issue #9: unexpected onChange event after setting the value from the server
 
 * Upgrade Notes: