id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
14258	Style Definitions with Variable Values Is Not Replacing ._AC Object Correctly	Phuoc Huynh		"Hello there,

I noticed I bug where CKEDITOR.style is defined in style.js, in the case of variable values, styleDefinition._AC is not being replaced. I encountered this issue when I try to remove a style
with variable values, when checkElementMatch is called, we see a discrepancy between styleDefinition.attributes and styleDefinition._AC. For example, styleDefinition.attributes will have the replaced value, myColor: 'white', but styleDefinition._AC will have the non-replaced placeholder, myColor: '#(color)'.

Relevant code block:


{{{
if ( variablesValues ) {
            styleDefinition = CKEDITOR.tools.clone( styleDefinition );

            replaceVariables( styleDefinition.attributes, variablesValues );
            replaceVariables( styleDefinition.styles, variablesValues );
}
}}}



Proposed code change:


{{{
if ( variablesValues ) {
            styleDefinition = CKEDITOR.tools.clone( styleDefinition );

            replaceVariables( styleDefinition.attributes, variablesValues );
            replaceVariables( styleDefinition.styles, variablesValues );
            if ( styleDefiniton._AC ) {
                    replaceVariables( styleDefinition._AC, variablesValues );
            }
}
}}}

Thank you!"	Bug	closed	Normal		General	4.5.4	expired