﻿id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
12034	Possible issue with build system; string literal corrupted?	enco		"Hello,

I already filed an issue for this on github: https://github.com/ckeditor/ckeditor-releases/issues/19
For your convenience, I'll re-post the thread here:
----------
> It seems that the \u200B string in core/selection.js gets corrupted during the build / minifying process:
> https://github.com/ckeditor/ckeditor-dev/blob/master/core/selection.js#L123
> {{{
> var fillingChar = element.getDocument().createText( '\u200B' );
> }}}
> Compare this to the current release version where this line becomes
> {{{
> i=l.getDocument().createText(""​"")
> }}}
> I stumbled upon this when debugging a strange issue on Google Chrome with question marks being inserted into the editor field every time I changed the text formatting (without active selection) or pressed Shift + Enter to insert a linebreak. In my downloaded version of ckeditor.js, this line looked like this:
> {{{
> i=o.getDocument().createText(""?"");
> }}}
> Sure enough, replacing the question mark by ""\u200B"" did the trick for me.
> 
> There seem to be a few bug reports mentioning this issue [1], but apparently no one was able to identify the problem conclusively. So I hope this helps.
> 
> [1] e.g. http://ckeditor.com/forums/Support/Question-Mark-after-pressing-button-toolbar
----------

This is what Reinmar replied on github:
----------
> What build/minifying tools do you use? CKBuilder does not have this problem, because then millions of users would report it to us. So this is something specific for small number of cases. Are you sure you didn't broke the encoding? The a zero width space is replaced with question mark it's usually because of wrong files encoding. It may be cause by wrong server settings, your builder/minifier saving them incorrectly, etc.
> 
> BTW. This is not the right place to report bugs, unless they are related directly to this repository. Bugs related to CKEditor and its builder should be reported on http://dev.ckeditor.com.
----------

So here's my reply:

Sorry if I wasn't clear enough and sorry if I posted this in the wrong place.
I'm not building CKeditor myself, I'm just comparing the source tree with the release code on github and with the official download on ckeditor.com.
Please compare https://github.com/ckeditor/ckeditor-dev/blob/master/core/selection.js#L123 to the corresponding code in https://github.com/ckeditor/ckeditor-releases/blob/master/ckeditor.js and you will probably see the issue. Unless I'm completely mistaken, the string literal '\u200B' has simply been dropped.

No idea where the question mark in my particular installation came from, but I don't think it is an encoding problem. This is a string literal in the JavaScript code that is never interpreted until the Javascript code is run, so it is not susceptible to encoding issues during transfer of the source code.
My guess is that this string literal was somehow interpreted during the build process instead of being taken literally, but I may be wrong.

Thanks for your time."	Bug	closed	Normal		General		invalid		
