Ticket #4640: 4640.patch

File 4640.patch, 2.0 KB (added by Alfonso Martínez de Lizarrondo, 10 years ago)

Patch. It doesn't change the behavior, just a little rewrite of the code

  • _source/plugins/filebrowser/plugin.js

     
    335335                        return false;
    336336                }
    337337
    338                 return ( definition.getContents( tabId ).get( elementId ).filebrowser && definition.getContents( tabId ).get( elementId ).filebrowser.url );
     338                var elementFileBrowser = definition.getContents( tabId ).get( elementId ).filebrowser;
     339                return ( elementFileBrowser && elementFileBrowser.url );
    339340        }
    340341
    341342        function setUrl( fileUrl, data )
     
    366367
    367368                        CKEDITOR.on( 'dialogDefinition', function( evt )
    368369                        {
     370                                var definition = evt.data.definition,
     371                                        element;
    369372                                // Associate filebrowser to elements with 'filebrowser' attribute.
    370                                 for ( var i in evt.data.definition.contents )
     373                                for ( var i in definition.contents )
    371374                                {
    372                                         attachFileBrowser( evt.editor, evt.data.name, evt.data.definition, evt.data.definition.contents[ i ].elements );
    373                                         if ( evt.data.definition.contents[ i ].hidden && evt.data.definition.contents[ i ].filebrowser )
     375                                        element = definition.contents[ i ] ;
     376                                        attachFileBrowser( evt.editor, evt.data.name, definition, element.elements );
     377                                        if ( element.hidden && element.filebrowser )
    374378                                        {
    375                                                 evt.data.definition.contents[ i ].hidden =
    376                                                         !isConfigured( evt.data.definition, evt.data.definition.contents[ i ][ 'id' ], evt.data.definition.contents[ i ].filebrowser );
     379                                                element.hidden = !isConfigured( definition, element[ 'id' ], element.filebrowser );
    377380                                        }
    378381                                }
    379382                        } );
  • CHANGES.html

     
    4444        <p>
    4545                Fixed issues:</p>
    4646        <ul>
    47                 <li></li>
     47                <li><a href="http://dev.fckeditor.net/ticket/4640">#4640</a> : Small optimizations for the fileBrowser plugin.</li>
    4848        </ul>
    4949        <h3>
    5050                CKEditor 3.1</h3>
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy