id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
8755	incorrect entities configuration code	Garret Wilson		"The [http://docs.cksource.com/ckeditor_api/symbols/CKEDITOR.config.html#.entities entities documentation] says that config.entities is a boolean value indicating whether to use HTML entities in the output. Unfortunately, the code in plugins/entities/plugin.js also tries to treat this variable as a list and add it to the current list of entities:

{{{
var selectedEntities = '';
if ( config.basicEntities !== false )
  selectedEntities += htmlbase;
if ( config.entities )
{
   selectedEntities += ',' + entities;
}}}

The last line above is incorrect and should be removed.

Furthermore, the code assumes that the basic entities have been added in the first place. Note that the last line above (and every line following that) blindly adds a ',' separator---but if you turn off ""basicEntities"", then the string will be empty initially and you'll be adding a needless ',' character.

In short, configuring CKEditor for normal XML entities is broken. And it's a shame that, even though CKEditor claims XHTML compatibility, its out-of-the-box entity handling isn't XML compliant---the only predefined XML entities are clearly enumerated in the [http://www.w3.org/TR/REC-xml/#sec-predefined-ent specification], and they aren't the same ones defaulted by CKEditor. And because the entity configuration is buggy, it's hard to configure CKEditor to be XML compliant."	Bug	closed	Normal		General		duplicate