From: Benoît Pin <benoit.pin@gmail.com>
Date: Wed, 7 Sep 2011 20:51:59 +0000 (+0200)
Subject: Bugfix : prendre le innerHTML du body du document n'est pas une bonne idée, dans... 
X-Git-Tag: 3.6.1~2
X-Git-Url: https://scm.cri.mines-paristech.fr/git/ckeditor.git/commitdiff_plain/2bfd716a52b02cffa159523c2911fbbf519772c2?ds=sidebyside;hp=226d255279bd8f5a38a4b55b5dae9250a6cbcbd2

Bugfix : prendre le innerHTML du body du document n'est pas une bonne idée, dans la mesure où ckeditor ajoute des attributs exotiques. Il vaut mieux utiliser dataValue.
---

diff --git a/skins/ckeditor/plugins/plinn_image/plugin.js b/skins/ckeditor/plugins/plinn_image/plugin.js
index 022af4c..ffb96fe 100644
--- a/skins/ckeditor/plugins/plinn_image/plugin.js
+++ b/skins/ckeditor/plugins/plinn_image/plugin.js
@@ -49,11 +49,12 @@ CKEDITOR.plugins.add( 'plinn_image',
 		editor.on('instanceReady', function(){
 			editor.on('getData',
 				function(evt) {
-					var body = evt.editor.document.$.body;
-					var images = body.getElementsByTagName('IMG');
+					var tmpDiv = document.createElement('div');
+					tmpDiv.innerHTML = evt.data.dataValue;
+					var images = tmpDiv.getElementsByTagName('IMG');
 					for (var i = 0 ; i < images.length ; i++)
 						updateImageSizeUrlParameters(images[i]);
-					evt.data.dataValue = evt.editor.document.$.body.innerHTML;
+					evt.data.dataValue = tmpDiv.innerHTML;
 				}
 			);
 		});