Changeset 2751:a96ec5640056 for plugins/dcCKEditor

Timestamp:

09/09/14 16:30:31 (11 years ago)

Author:

Nicolas <nikrou77@…>

Branch:

default

Message:

Each editor (ckeditor, legacy ones) loads its own context when it is active.
Add plugin_id parameter to transmit that context
Remove call of ckeditorExtraPlugins behavior until a good solution is found.
Closes #1983

Location:

plugins/dcCKEditor

Files:

• _post_config.php (modified) (6 diffs)
• js/ckeditor-plugins/dclink/plugin.js (modified) (2 diffs)
• js/ckeditor-plugins/entrylink/plugin.js (modified) (3 diffs)
• js/ckeditor-plugins/media/plugin.js (modified) (2 diffs)
• js/popup_link.js (modified) (1 diff)
• js/popup_media.js (modified) (2 diffs)
• js/popup_posts.js (modified) (1 diff)

plugins/dcCKEditor/_post_config.php

@@ -31 +31 @@
                }
           }
-          
+
           return url;
      };
 
-     /* Retrieve editor name from URL */
-     $.getEditorName = function getEditorName(url) {
-          return url.replace(/.*editor=([^&]*)(&.*)?/,'$1');
+     /* Retrieve editor from popup */
+    $.active_editor = null;
+     $.getEditorName = function getEditorName() {
+          return $.active_editor;
      }
 })(jQuery);
@@ -49 +50 @@
      CKEDITOR.config.skin = 'dotclear,'+dotclear.dcckeditor_plugin_url+'/js/ckeditor-skins/dotclear/';
 
-<?php if (!empty($dcckeditor_cancollapse_button)):?>    
+<?php if (!empty($dcckeditor_cancollapse_button)):?>
      CKEDITOR.config.toolbarCanCollapse = true;
 <?php endif;?>
-     
-     CKEDITOR.plugins.addExternal('entrylink',dotclear.dcckeditor_plugin_url+'/js/ckeditor-plugins/entrylink/');   
+
+     CKEDITOR.plugins.addExternal('entrylink',dotclear.dcckeditor_plugin_url+'/js/ckeditor-plugins/entrylink/');
      CKEDITOR.plugins.addExternal('dclink',dotclear.dcckeditor_plugin_url+'/js/ckeditor-plugins/dclink/');
      CKEDITOR.plugins.addExternal('media',dotclear.dcckeditor_plugin_url+'/js/ckeditor-plugins/media/');
@@ -75 +76 @@
           extraPlugins: '<?php echo $defautExtraPlugins;?>',
 
-          <?php if (!empty($dcckeditor_format_select)):?>                  
+          <?php if (!empty($dcckeditor_format_select)):?>
           // format tags
           format_tags: 'p;h1;h2;h3;h4;h5;h6;pre;address',
@@ -102 +103 @@
 <?php endif;?>
                          'Bold','Italic','Underline','Strike','Subscript','Superscript','Code','Blockquote',
-                         
+
 <?php if (!empty($dcckeditor_list_buttons)):?>
                          'NumberedList', 'BulletedList',
@@ -135 +136 @@
                 ,'TextColor'
 <?php endif;?>
-                    ] 
+                    ]
                },
                <?php // add extra buttons comming from dotclear plugins
@@ -156 +157 @@
                $('#cke_post_excerpt').addClass('hide');
           }
-          
+
           $('#excerpt-area label').click(function() {
                $('#cke_post_excerpt').toggleClass('hide',$('#post_excerpt').hasClass('hide'));
           });
      });
+
+     // @TODO: find a better way to retrieve active editor
+     for (var id in CKEDITOR.instances) {
+          CKEDITOR.instances[id].on('focus', function(e) {
+               $.active_editor = e.editor.name;
+          });
+     }
 });

plugins/dcCKEditor/js/ckeditor-plugins/dclink/plugin.js

@@ -6 +6 @@
                     exec: function(editor) {
                          if (editor.getSelection().getNative().toString().replace(/\s*/,'')!='') {
-                              $.toolbarPopup('popup_link.php?editor='+editor.name);
+                              $.toolbarPopup('popup_link.php?plugin_id=dcCKEditor');
                          }
                     }
@@ -26 +26 @@
                               editor.getSelection().selectElement(element);
 
-                              var popup_url = 'popup_link.php?editor='+editor.name;
+                              var popup_url = 'popup_link.php?plugin_id=dcCKEditor';
                               if (element.getAttribute('href')) {
                                    popup_url += '&href='+element.getAttribute('href');

plugins/dcCKEditor/js/ckeditor-plugins/entrylink/plugin.js

@@ -6 +6 @@
                     exec: function(editor) {
                          if (editor.getSelection().getNative().toString().replace(/\s*/,'')!='') {
-                              $.toolbarPopup('popup_posts.php?popup=1&editor='+editor.name);
+                              $.toolbarPopup('popup_posts.php?popup=1&plugin_id=dcCKEditor');
                          }
                     }
@@ -18 +18 @@
 
                editor.on('doubleclick',function(e) {
-                    var element = CKEDITOR.plugins.link.getSelectedLink( editor ) || e.data.element;
+                    var element = CKEDITOR.plugins.link.getSelectedLink(editor) || e.data.element;
                     if (!element.isReadOnly()) {
                          if (element.is('a')
@@ -25 +25 @@
                               editor.getSelection().selectElement(element);
 
-                              $.toolbarPopup('popup_posts.php?popup=1&editor='+editor.name);
+                              $.toolbarPopup('popup_posts.php?popup=1&plugin_id=dcCKEditor');
                               return false;
                          }

plugins/dcCKEditor/js/ckeditor-plugins/media/plugin.js

@@ -5 +5 @@
                editor.addCommand('mediaCommand', {
                     exec: function(editor) {
-                         $.toolbarPopup('media.php?popup=1&editor='+editor.name);
+                         $.toolbarPopup('media.php?popup=1&plugin_id=dcCKEditor');
                     }
                });
@@ -19 +19 @@
                     if (!element.isReadOnly()) {
                          if (element.is('img') || (element.is('a') && element.hasClass('media-link'))) {
-                              $.toolbarPopup('media.php?popup=1&editor='+editor.name);
+                              $.toolbarPopup('media.php?popup=1&plugin_id=dcCKEditor');
                               return false;
                          }

plugins/dcCKEditor/js/popup_link.js

@@ -6 +6 @@
      $('#link-insert-ok').click(function() {
           var insert_form = $('#link-insert-form').get(0);
-          if (insert_form == undefined) { 
-               return; 
+          if (insert_form == undefined) {
+               return;
           }
 
-          var editor_name = window.opener.$.getEditorName(document.location.search),
+          var editor_name = window.opener.$.getEditorName(),
           editor = window.opener.CKEDITOR.instances[editor_name],
-          link = '',     
+          link = '',
           selected_text = editor.getSelection().getNative().toString();
 

plugins/dcCKEditor/js/popup_media.js

@@ -3 +3 @@
           window.close();
      });
-     
+
      $('#media-insert-ok').click(function() {
           var insert_form = $('#media-insert-form').get(0);
-          if (insert_form === undefined) { 
-               return; 
+          if (insert_form === undefined) {
+               return;
           }
 
-          var editor_name = window.opener.$.getEditorName(document.location.search),
+          var editor_name = window.opener.$.getEditorName(),
           editor = window.opener.CKEDITOR.instances[editor_name],
           type = insert_form.elements.type.value,
@@ -47 +47 @@
                          element = window.opener.CKEDITOR.dom.element.createFromHtml(img);
                     }
-                         
+
                     editor.insertElement(element);
                }

plugins/dcCKEditor/js/popup_posts.js

@@ -6 +6 @@
      $('#form-entries tr>td.maximal>a').click(function(e) {
           e.preventDefault();
-          var editor_name = window.opener.$.getEditorName(document.location.search),
+          var editor_name = window.opener.$.getEditorName(),
           editor = window.opener.CKEDITOR.instances[editor_name],
           link = '',