Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1258 closed Bug (invalid)

Three garbage characters at beginning of many files.

Reported by: Miles Owned by:
Priority: Normal Milestone:
Component: General Version:
Keywords: Cc:

Description

At the top of a number of files (listed below) in FCKEditor there are three extra nonprinting characters, ASCII 239, 187, and 191. Looks something like this: /*

They can be shown with a little perl script like this:

open $file, "<editor/js/fckeditorcode_gecko.js";
while ($char = getc($file)){
    print ord($char) ." $char\n";
}

These characters confuse and break some editors and the lighttpd web server. I can see no valid reason for these characters to be present, and as the attached screen shot shows, they occasionally break JS parsing.

The full list of files that have this problem, from the September 17th nightly:

./editor/_source/internals/fckplugins.js
./editor/_source/internals/fckselection.js
./editor/_source/internals/fckdebug.js
./editor/_source/internals/fck_ie.js
./editor/_source/internals/fckxhtml.js
./editor/_source/internals/fckselection_gecko.js
./editor/_source/internals/fckcodeformatter.js
./editor/_source/internals/fckdialog.js
./editor/_source/internals/fcktools.js
./editor/_source/internals/fckcommands.js
./editor/_source/internals/fckxhtmlentities.js
./editor/_source/internals/fcktablehandler_ie.js
./editor/_source/internals/fcktoolbaritems.js
./editor/_source/internals/fckxhtml_gecko.js
./editor/_source/internals/fckconfig.js
./editor/_source/internals/fckdialog_gecko.js
./editor/_source/internals/fcktools_gecko.js
./editor/_source/internals/fcklisthandler.js
./editor/_source/internals/fckxhtml_ie.js
./editor/_source/internals/fcktools_ie.js
./editor/_source/internals/fck.js
./editor/_source/internals/fckundo_ie.js
./editor/_source/internals/fcktablehandler.js
./editor/_source/internals/fcklistslib.js
./editor/_source/internals/fck_gecko.js
./editor/_source/internals/fcktablehandler_gecko.js
./editor/_source/internals/fckbrowserinfo.js
./editor/_source/internals/fckdomtools.js
./editor/_source/internals/fckregexlib.js
./editor/_source/internals/fckselection_ie.js
./editor/_source/internals/fckurlparams.js
./editor/_source/internals/fck_contextmenu.js
./editor/_source/internals/fckdialog_ie.js
./editor/_source/internals/fckundo_gecko.js
./editor/_source/internals/fcktoolbarset.js
./editor/_source/internals/fckdocumentprocessor.js
./editor/_source/internals/fcklanguagemanager.js
./editor/_source/commandclasses/fck_othercommands.js
./editor/_source/commandclasses/fcktextcolorcommand.js
./editor/_source/commandclasses/fcknamedcommand.js
./editor/_source/commandclasses/fckpasteplaintextcommand.js
./editor/_source/commandclasses/fckspellcheckcommand_gecko.js
./editor/_source/commandclasses/fcktablecommand.js
./editor/_source/commandclasses/fckpastewordcommand.js
./editor/_source/commandclasses/fckspellcheckcommand_ie.js
./editor/_source/commandclasses/fckfitwindow.js
./editor/_source/commandclasses/fckstylecommand.js
./editor/_source/classes/fcktoolbarspecialcombo.js
./editor/_source/classes/fcktoolbarbreak_ie.js
./editor/_source/classes/fckstyledef_gecko.js
./editor/_source/classes/fckplugin.js
./editor/_source/classes/fckmenublockpanel.js
./editor/_source/classes/fcktoolbarstylecombo.js
./editor/_source/classes/fcktoolbarfontsizecombo.js
./editor/_source/classes/fckevents.js
./editor/_source/classes/fckcontextmenu.js
./editor/_source/classes/fcktoolbarpanelbutton.js
./editor/_source/classes/fcktoolbarbreak_gecko.js
./editor/_source/classes/fckimagepreloader.js
./editor/_source/classes/fcktoolbarfontformatcombo.js
./editor/_source/classes/fckmenuitem.js
./editor/_source/classes/fckenterkey.js
./editor/_source/classes/fckicon.js
./editor/_source/classes/fcktoolbarfontscombo.js
./editor/_source/classes/fcktoolbarbuttonui.js
./editor/_source/classes/fckstyledef_ie.js
./editor/_source/classes/fckdomrange.js
./editor/_source/classes/fcktoolbarbutton.js
./editor/_source/classes/fckpanel.js
./editor/_source/classes/fckspecialcombo.js
./editor/_source/classes/fcktoolbar.js
./editor/_source/classes/fckdomrange_gecko.js
./editor/_source/classes/fckiecleanup.js
./editor/_source/classes/fckelementpath.js
./editor/_source/classes/fckxml_gecko.js
./editor/_source/classes/fckdomrange_ie.js
./editor/_source/classes/fckw3crange.js
./editor/_source/classes/fckstylesloader.js
./editor/_source/classes/fckdocumentfragment_gecko.js
./editor/_source/classes/fckeditingarea.js
./editor/_source/classes/fckxml_ie.js
./editor/_source/classes/fckkeystrokehandler.js
./editor/_source/classes/fckstyledef.js
./editor/_source/classes/fckdocumentfragment_ie.js
./editor/_source/classes/fckmenublock.js
./editor/_source/fckjscoreextensions.js
./editor/_source/fckscriptloader.js
./editor/_source/fckconstants.js
./editor/dialog/fck_flash/fck_flash.js
./editor/dialog/fck_image/fck_image.js
./editor/dialog/fck_link/fck_link.js
./editor/dialog/common/fck_dialog_common.js
./editor/dialog/fck_select/fck_select.js
./editor/dialog/fck_spellerpages/spellerpages/spellChecker.js
./editor/dialog/fck_spellerpages/spellerpages/controlWindow.js
./editor/dialog/fck_spellerpages/spellerpages/wordWindow.js
./editor/lang/fo.js
./editor/lang/bs.js
./editor/lang/cs.js
./editor/lang/en-au.js
./editor/lang/km.js
./editor/lang/es.js
./editor/lang/eu.js
./editor/lang/ko.js
./editor/lang/hu.js
./editor/lang/no.js
./editor/lang/sk.js
./editor/lang/vi.js
./editor/lang/en-uk.js
./editor/lang/ms.js
./editor/lang/uk.js
./editor/lang/zh-cn.js
./editor/lang/ro.js
./editor/lang/pt-br.js
./editor/lang/af.js
./editor/lang/ru.js
./editor/lang/bn.js
./editor/lang/nb.js
./editor/lang/el.js
./editor/lang/gl.js
./editor/lang/ar.js
./editor/lang/en.js
./editor/lang/fr.js
./editor/lang/et.js
./editor/lang/hr.js
./editor/lang/nl.js
./editor/lang/mn.js
./editor/lang/pl.js
./editor/lang/th.js
./editor/lang/it.js
./editor/lang/sl.js
./editor/lang/lt.js
./editor/lang/sr-latn.js
./editor/lang/lv.js
./editor/lang/zh.js
./editor/lang/ca.js
./editor/lang/en-ca.js
./editor/lang/pt.js
./editor/lang/da.js
./editor/lang/sr.js
./editor/lang/tr.js
./editor/lang/fa.js
./editor/lang/bg.js
./editor/lang/de.js
./editor/lang/sv.js
./editor/lang/ja.js
./editor/lang/he.js
./editor/lang/fi.js
./editor/lang/hi.js
./editor/lang/eo.js
./editor/plugins/placeholder/lang/fr.js
./editor/plugins/placeholder/lang/de.js
./editor/plugins/placeholder/lang/pl.js
./editor/plugins/placeholder/lang/it.js
./editor/plugins/placeholder/lang/en.js
./editor/plugins/placeholder/fckplugin.js
./editor/plugins/autogrow/fckplugin.js
./editor/plugins/tablecommands/fckplugin.js
./editor/plugins/simplecommands/fckplugin.js
./editor/filemanager/browser/default/js/fckxml.js
./editor/filemanager/browser/default/js/common.js
./editor/filemanager/browser/default/connectors/asp/util.asp
./editor/filemanager/browser/default/connectors/asp/commands.asp
./editor/filemanager/browser/default/connectors/asp/config.asp
./editor/filemanager/browser/default/connectors/asp/io.asp
./editor/filemanager/browser/default/connectors/asp/class_upload.asp
./editor/filemanager/browser/default/connectors/asp/connector.asp
./editor/filemanager/browser/default/connectors/asp/basexml.asp
./editor/filemanager/upload/asp/upload.asp
./editor/filemanager/upload/asp/config.asp
./editor/filemanager/upload/asp/io.asp
./editor/filemanager/upload/asp/class_upload.asp
./editor/js/fckeditorcode_gecko.js
./editor/js/fckeditorcode_ie.js
./_samples/_plugins/samples/fckplugin.js
./_samples/_plugins/findreplace/lang/fr.js
./_samples/_plugins/findreplace/lang/it.js
./_samples/_plugins/findreplace/lang/en.js
./_samples/_plugins/findreplace/fckplugin.js
./_samples/asp/sample01.asp
./_samples/asp/sample02.asp
./_samples/asp/sample03.asp
./_samples/asp/sample04.asp
./_samples/asp/sampleposteddata.asp
./_samples/html/sample06.config.js
./_samples/html/sampleposteddata.asp
./fckeditor.asp
./fckconfig.js

Attachments (1)

screenshot.png (6.0 KB) - added by Miles 12 years ago.
Screenshot of extra characters breaking js parsing

Download all attachments as: .zip

Change History (3)

Changed 12 years ago by Miles

Attachment: screenshot.png added

Screenshot of extra characters breaking js parsing

comment:1 Changed 12 years ago by Alfonso Martínez de Lizarrondo

Resolution: invalid
Status: newclosed

Those are the Unicode BOM characters.

comment:2 Changed 12 years ago by Frederico Caldeira Knabben

Please check out #310 for more info.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy