#2185: FCKPackager incorrectly adds semicolons after functions passed as an
argument to another function.
------------------------------------+---------------------------------------
Reporter: martinkou | Owner:
Type: Bug | Status: new
Priority: Normal | Milestone: FCKeditor 2.6.1
Component: Project : FCKpackager | Version: SVN
Resolution: | Keywords: Confirmed
------------------------------------+---------------------------------------
Old 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 ) ;
> }
> }
> }}}
New 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 after the anonymous function, 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 ) ;
}
}
}}}
--
Ticket URL: <https://dev.fckeditor.net/ticket/2185#comment:3>
FCKeditor <http://www.fckeditor.net>
The text editor for Internet
-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference
Don't miss this year's exciting event. There's still time to save $100.
Use priority code J8TL2D2.
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
FCKeditor-Trac mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/fckeditor-trac
