Bugfix : prendre le innerHTML du body du document n'est pas une bonne idée, dans...

[ckeditor.git] / skins / ckeditor / plugins / plinn_image / plugin.js

diff --git a/skins/ckeditor/plugins/plinn_image/plugin.js b/skins/ckeditor/plugins/plinn_image/plugin.js
index 442a2be..ffb96fe 100644 (file)
--- a/skins/ckeditor/plugins/plinn_image/plugin.js
+++ b/skins/ckeditor/plugins/plinn_image/plugin.js
@@ -23,6 +23,21 @@ function updateImageSizeUrlParameters(img) {
        }
 }
 
        }
 }
 

+function openPlinnImageDialog(path, editor) {
+       var winOptions  = "location=no,menubar=no,toolbar=no,dependent=yes,dialog=yes,minimizable=no,modal=yes,alwaysRaised=yes" +
+               ",resizable=yes"  +
+               ",width=801"  +
+               ",height=600";
+               //",top="  + iTop +
+               //",left=" + iLeft ;
+       
+       var win = open(path + 'dialog/plinn_image.html', 'PlinnImageDialog', winOptions);
+       win.dialogArguments = new Object();
+       win.dialogArguments.editor = editor;
+       win.dialogArguments.pluginPath = path;
+       win.dialogArguments.CKEDITOR = CKEDITOR;
+}
+

 
 CKEDITOR.plugins.add( 'plinn_image',
 {
 
 CKEDITOR.plugins.add( 'plinn_image',
 {


@@ -34,29 +49,27 @@ CKEDITOR.plugins.add( 'plinn_image',
                editor.on('instanceReady', function(){
                        editor.on('getData',
                                function(evt) {
                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]);
                                        for (var i = 0 ; i < images.length ; i++)
                                                updateImageSizeUrlParameters(images[i]);

-                                       evt.data.dataValue = evt.editor.document.$.body.innerHTML;
+                                       evt.data.dataValue = tmpDiv.innerHTML;

                                }
                        );
                });
                                }
                        );
                });

-               
+               var pluginPath = this.path;

                var command = editor.addCommand('plinn_image',
                        {
                var command = editor.addCommand('plinn_image',
                        {

-                               exec : function(editor) {
-                                       alert('Bonjour la France !');
-                               }
+                               exec : function(editor){openPlinnImageDialog(pluginPath, editor);}

                        });
                
                editor.ui.addButton('PlinnImage',
                        {
                                label : editor.lang.common.image,
                        });
                
                editor.ui.addButton('PlinnImage',
                        {
                                label : editor.lang.common.image,

-                               icon : this.path + 'dialog/plinn_image.gif',
+                               icon : pluginPath + 'dialog/plinn_image.gif',

                                command : 'plinn_image'
                        });
                                command : 'plinn_image'
                        });

-               console.log(this.path);

        }
 });
 
        }
 });