Opened 17 years ago

Last modified 17 years ago

#2185 closed Bug

FCKPackager incorrectly adds semicolons after functions passed as an argument to another function. — at Initial Version

Reported by: Martin Kou Owned by:
Priority: Normal Milestone: FCKeditor 2.6.1
Component: Project : CKPackager Version: SVN (FCKeditor) - Retired
Keywords: Confirmed Cc:

Description

Write the following JavaScript code and let FCKPackager compress it:

void(
function(b)
{}
);

FCKPackager would give you something like this:

void(function(b){}''';''');

Notice the semicolon in bold, it is incorrect.

This bug is causing the current FCKeditor code in the SVN unusable after running build_release.bat, because of the following code in fckdocumentprocessor.js:

if ( FCKBrowserInfo.IsSafari )
{
        FCKDocumentProcessor.AppendNew().ProcessDocument = function( doc )
        {
                var spans = doc.getElementsByClassName ?
                        doc.getElementsByClassName( 'Apple-style-span' ) :
                        Array.prototype.filter.call(
                                        doc.getElementsByTagName( 'span' ),
                                        function( item ){ return item.className == 'Apple-style-span' ; }
                                        ) ;
                for ( var i = spans.length - 1 ; i >= 0 ; i-- )
                        FCKDomTools.RemoveNode( spans[i], true ) ;
        }
}

Change History (0)

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