Colors are possibly removed when using pastefromword

[linnex]

Hello everybody,

in plugin pastefromword\filter\default.js (V 3.6.3, Line 1190ff) the styleFilter for colors, or more specific the new-value function does not return anything but "false" for transparent colors (or the replace for gecko). When this filter is applied to styles colors and background-colors are removed - which I believe is not desired. Shouldn't this function "return value;" if the two other tests have "failed".

Tested in IE on an Excel copy&paste: without: background-color is removed with "return value": background-color is retained (as expected - removeStyles in config is false).

Problem still remains in todays downloaded nightly build.

Thanks linnex

[Jakub Ś]

DUP of #6465.

@linnex from your ticket report I understand that have come up with a patch? Can you tell me what have you changed that fixed this filter?

[linnex]

Hi,

I tried to explain it above, maybe I wasn't exact enough though.

I just added one line to the following code snippet in "pastefromword\filter\default.js"

					// Otherwise provide a black-list of styles that we remove.
					[
						[ ( /^mso-/ ) ],
						// Fixing color values.
						[ ( /-color$/ ), null, function( value )
						{
							if ( value == 'transparent' )
								return false;
							if ( CKEDITOR.env.gecko )
								return value.replace( /-moz-use-text-color/g, 'transparent' );
      // 2012-07-25 linnex: the following line is new
      return value;
						} ],

I believe the problem is the blacklist that falsely removes all back-colors from Excel pastes instead of transparent only.

It worked for me.

Cheers linnex