htmldataprocessor: fix for convert style attributes to lowercase in IE

[Juergen]

File: _source/plugins/htmldataprocessor/plugin.js (Starting line 240)

The function to convert ALL styles to lowercase causing a problem with capital urls:

background-image: url(http://somedomain/SomeBackground.jpg);

is converted to:

background-image: url(http://somedomain/somebackground.jpg);

On a windows server, this is not the biggest problem, but a linux server won't find that file.

Heres the possible fix:

	if ( CKEDITOR.env.ie )
	{
		// IE outputs style attribute in capital letters. We should convert
		// them back to lower case.
		defaultHtmlFilterRules.attributes.style = function( value, element )
		{
			var res = value.match(/([a-z-]+):/gi);
			if (res) {
				for (var i = 0; i < res.length; ++i) {
					value = value.replace(res[i], res[i].toLowerCase());
				}
				return value;
			}
			else {
				return value.toLowerCase();
			}
		};
	}

Greetings, Juergen

[Tobiasz Cudnik]

I've been able to reproduce it, thank you for the fix. We will check it.

[Charlie]

This appears to be a larger problem with the CSSNormalizer. The lower case function also should not be applied to font names. See: ​http://cksource.com/forums/viewtopic.php?f=11&t=19446

I.E. Instead of outputting: style="font-family: comic sans ms,cursive;" We should get: style="font-family: Comic Sans MS, cursive;"

Since that's what we gave in the fonts plugin and it's possible for fonts to be case-sensitive on certain systems. (In general isn't CSS case-sensitive?)

[Charlie]

ignore previous comment. Proposed patch fixes it (In the scope of this bug)

[Kingsquare]

This problem still exists in 3.4.2. and again: the patch above fixes this problem.

Please apply this patch for the next release!

Thanks

[Sa'ar Zac Elias]

Please don't change the version field.

[Garry Yao]

Could be even simplified?

return value.replace( /(^|;)([^\:]+)/g, function( match )
{
	return match.toLowerCase();
});

[Sa'ar Zac Elias]

Fixed with [6294].