Opened 15 years ago

Closed 15 years ago

#4640 closed Bug (fixed)

Small optimizations in the fileBrowser plugin

Reported by: Alfonso Martínez de Lizarrondo Owned by: Alfonso Martínez de Lizarrondo
Priority: Normal Milestone: CKEditor 3.2
Component: General Version: 3.0.1
Keywords: Review+ Cc:

Description

At the bottom, the elements that are reused can be declared as local variables so it's a little clearer and it will compress a little further (sorry, I still don't have installed here any SVN to provide a diff)

	CKEDITOR.plugins.add( 'filebrowser',
	{
		init : function( editor, pluginPath )
		{
			editor._.filebrowserFn = CKEDITOR.tools.addFunction( setUrl, editor );

			CKEDITOR.on( 'dialogDefinition', function( evt )
			{
				var definition = evt.data.definition, 
					element;
				// Associate filebrowser to elements with 'filebrowser' attribute.
				for ( var i in definition.contents )
				{
					element = definition.contents[ i ] ;
					attachFileBrowser( evt.editor, evt.data.name, definition, element.elements );
					if ( element.hidden && element.filebrowser )
					{
						element.hidden =
							!isConfigured( definition, element[ 'id' ], element.filebrowser );
					}
				}
			} );
		}
	} );

In the isConfigured I think that the last line can be also optimized to avoid repeated function calls: Turn

		return ( definition.getContents( tabId ).get( elementId ).filebrowser && 
				 definition.getContents( tabId ).get( elementId ).filebrowser.url );

Into

		var elementFileBrowser = definition.getContents( tabId ).get( elementId ).filebrowser;
		return ( elementFileBrowser && elementFileBrowser.url );

Attachments (1)

4640.patch (2.0 KB) - added by Alfonso Martínez de Lizarrondo 15 years ago.
Patch. It doesn't change the behavior, just a little rewrite of the code

Download all attachments as: .zip

Change History (5)

comment:1 Changed 15 years ago by Frederico Caldeira Knabben

Milestone: CKEditor 3.2

Changed 15 years ago by Alfonso Martínez de Lizarrondo

Attachment: 4640.patch added

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

comment:2 Changed 15 years ago by Alfonso Martínez de Lizarrondo

Keywords: Review? added; HasPatch removed
Owner: set to Alfonso Martínez de Lizarrondo
Status: newassigned

Provided patch for review.

comment:3 Changed 15 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:4 Changed 15 years ago by Alfonso Martínez de Lizarrondo

Resolution: fixed
Status: assignedclosed

Fixed with [4918]

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy