id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
13886	"Invalid handling of CKEDITOR.style instance with ""styles"" property by CKEDITOR.filter"	Olek Nowodziński	kkrzton	"A followup of https://www.drupal.org/node/2598070.

Revealed by http://cgit.drupalcode.org/drupal/commit/?id=cea9c14, which was meant to fix https://www.drupal.org/node/2589779.

== Steps to reproduce ==

0. For a given filter
 {{{
var filter = new CKEDITOR.filter( 'a {color}' );
}}}
1. Define a style with `styles: {}`. Like 
 {{{
var style = new CKEDITOR.style( { element: 'a', styles: {} } )
}}}
2. Check in in the filter.
 {{{
filter.check( style );
}}}

== Expected result ==

Check passes.

== Actual result ==

Check fails. But if defined a key in `styles` it will pass.

{{{
var style = new CKEDITOR.style( { element: 'a', styles: { color: 'red' } } );
filter.check( style ) -> true.
}}}

Also if removed `styles` key from definition it will pass.

{{{
var style = new CKEDITOR.style( { element: 'a' } );
filter.check( style ) -> true.
}}}

== Ideas ==

Something could be wrong with https://github.com/ckeditor/ckeditor-dev/blob/7856fe93f844be5053fa62cad1ece1ba6fc8635b/core/filter.js#L996-L1017."	Bug	closed	Normal	CKEditor 4.5.9	General	4.5.5	fixed	Drupal	wim.leers@…