Allow the possibility to specify which tags to be stripped when Strip HTML checkbox is ticked.

And here we are, 8 months later :)

re-roll for 2.1.x, based on a patch from a #15.

re-roll for 2.1.0, based on a patch from a #15.

Thanks for the updated patch @Stockticker. It's working for us!

I'm not sure whether this new patch "uses the same implementation as the views functionality mentioned in the parent meta issue?", so not marking RTBC.

MR was updated to reflect the proper branch. Needs updating or re-rolled patch applied. Might have to close and open a new MR.

I rebased onto the 2.1.x branch.

I had to change:

           // https://stackoverflow.com/questions/12824899/strip-tags-replace-tags-by-space-rather-than-deleting-them
           $output = preg_replace('/<(\w+)(?![^>]*\/>)[^>]*>/', ' <\1>', $output);
-          $output = strip_tags($output);
           $output = str_replace('  ', ' ', $output);
+          $output = strip_tags($output, $this->getSetting('trim_options_allowed_html'));
           $output = trim($output);
 
-          $output = Xss::filter($output, $this->getSetting('trim_options_allowed_html'));

Seems to be working so far.

Very helpful functionality that was also requested in ✨ Add preserve tags option to formatter Closed: duplicate

rossb89 → changed the visibility of the branch 2901579-selective-html-stripping-2.1.x to hidden.

rossb89 → changed the visibility of the branch 2901579-selective-html-stripping-2.1.x to active.

I've redone the last MR (68) as the rebase wasn't done correctly and it appeared to contain duplicated code that was already in 2.1.x...

I've done a new clean MR (92) with the patch re-rolled which is working nicely 👍

rossb89 → changed the visibility of the branch 2901579-selective-html-stripping to hidden.

rossb89 → changed the visibility of the branch 2901579-selective-html-stripping-2.1.x to hidden.

Did a few more tweaks to the work here in MR-92 to get tests passing, which they are now!

Thanks to all those who have contributed to this issue, however in recent testing after first applying patch from the older MR-68 and then later from MR-92 I've found that allowed tags attributes are removed. So if you allow <a> tags the href="[URL]" (and any other attributes) are removed, breaking the links.

The last working patch was https://www.drupal.org/files/issues/2023-07-05/smart_trim--2901579--allo... → (still listed here) from July 2023 against version 2.0.1. It looks like it was actually an unrelated change release (2.1.1) of this module.

Code above patch changes in src/Plugin/Field/FieldFormatter/SmartTrimFormatter.php from MR-92

           $output = preg_replace('/<(\w+)(?![^>]*\/>)[^>]*>/', ' <\1>', $output);

Code above patch changes in src/Plugin/Field/FieldFormatter/SmartTrimFormatter.php from patch #25

           $output = str_replace('<', ' <', $output);

Reverting this change after applying the latest patch, or reverting back to version 2.1.0 and using the older patch both restore links in allowed <a> tags.

This patch will specify which elements will be deleted and force them only by not using the default strip after them.

This patch removes the default 'img' from href=" https://www.drupal.org/project/smart_trim/issues/2901579#comment-15822933 ✨ Allow the possibility to specify which tags to be stripped when Strip HTML checkbox is ticked. Needs work ">#41

Was looking for a way of doing this via twig, but didn't see any other way.

I think the configuration for listing the allowed tags should be built in a more user friendly way and then converted to a format that is passed to preg_replace(), rather than passing that value directly in - the risk for mucking up the syntax is a little high otherwise.

Here's a new patch that takes MR-92 (which we were using previously), and then fixes the regex issues called out in #2901579-39: Allow the possibility to specify which tags to be stripped when Strip HTML checkbox is ticked. → which came from 🐛 [regression] Space inserted before full stop, comma, etc Fixed which was merged in the meantime.

I'm not personally fond of the approach used in #2901579-41: Allow the possibility to specify which tags to be stripped when Strip HTML checkbox is ticked. → and #2901579-42: Allow the possibility to specify which tags to be stripped when Strip HTML checkbox is ticked. → which use some new custom regexes, rather than relying on Xss::filter().

@dsnopek I tested applying your patch (which only applies cleanly to the active development branch for the 2.2 versions - 2.x-dev) and the functionality works as expected.

I also prefer this approach vs. the other approaches using custom regexes.