Opened 18 years ago
Last modified 18 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 ) ;
}
}
