Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#1 | 926364.patch | 896 bytes | roderik |
Comments
Comment #1
roderikI 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.
Comment #2
roderik.
Comment #3
lambic CreditAttribution: lambic commentedYep, your patch fixes the problem, thanks!
Comment #4
markus_petrux CreditAttribution: markus_petrux commentedOh, 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!
Comment #5
markus_petrux CreditAttribution: markus_petrux commented