Steps:

1. Create a piece of content with a table tag
2. Try to add a class to the table tag using HTML view
3. Go back to wysiwyg view

If the wysiwyg_filter module is enabled, the class gets stripped, even if the filter is not turned on for the input format.

CommentFileSizeAuthor
#1 926364.patch896 bytesroderik
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

roderik’s picture

Status: Reviewed & tested by the community » Active
FileSize
896 bytes

I think I had the same problem. (@lambic: please confirm; thanks.)

"Filtering occurring" means, in my case: filtering occurring inside TinyMCE, on the basis of settings provided by WYSIWYG Filter, even though it is not enabled.
(Since #713922: Provide the valid_elements option to TinyMCE editors dynamically altering the Wysiwyg API settings)

Or, in my practical situation:
- I implement TinyMCE with default WYSIWYG Filter, and add 'img' tag to the filter settings.
- ...but I don't add enough tags, and my client has the bright idea to turn off WYSIWYG Filter in the default filter format, so it is supposedly unfiltered HTML.
- images start inexplicably disappearing from every node which is saved.

(They disappear because the 'valid_elements' setting which is still passed to TinyMCE, has somehow reverted to the default setting. Effect: 'img' tags in the HTML are still visible in the TinyMCE editor area, so nothing seems wrong.... but they get stripped out upon saving. And in TinyMCE's 'source code' view.)

Not a lot of people will encounter this bug... after all, who installs WYSIWYG Filter but does not enable it on some filter format which has a Wysiwyg profile?
But it's a bitch to trace the error. And HTML elements (in my case images) disappearing is not cool. So I think it deserves a stable release.

roderik’s picture

Status: Active » Needs review

.

lambic’s picture

Status: Needs review » Reviewed & tested by the community

Yep, your patch fixes the problem, thanks!

markus_petrux’s picture

Status: Active » Reviewed & tested by the community

Oh, I think I missed this issue. Sorry for that.

I have applied a slightly different version of the patch to CVS: http://drupal.org/cvs?commit=485110

Thanks!

markus_petrux’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.