Hi

Thanks for this very useful and well documented module.

However, I have faced a few trouble when using the input filter feature:
- the label to enter the pattern is "Shadow Colour" instead of "Input Filter Pattern"
- the sample pattern given for h1 isn't consistent with the one for h* and leads to content being truncated after the first occurrence of h1
- preg_replace and the 'e' modifier may escape quotes, then the image being generated contains backslashes when the text contained
- some patterns could be made case insensitive
- quotes in text are meaningful and shouldn't be ignored

Please find below a comprehensive patch for all these issues (the main one being related to backslashes and to the h1 pattern)

Regards

CommentFileSizeAuthor
signwriterescapesdev.patch3.37 KBopen-keywords
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Agileware’s picture

Assigned: Unassigned » Agileware
Status: Active » Needs review

Thanks for the patch. I've tested against D5 version and looks OK. Trying to apply the same changes to D6 with mixed results, which is expected. So just working thru those.

Agileware’s picture

Version: 5.x-1.x-dev » 6.x-1.x-dev

This has been applied to the 5.x-1.x-dev version.

Changing to the 6 branch as it has not been applied there yet.

Agileware’s picture

Version: 6.x-1.x-dev » 6.x-2.x-dev
Status: Needs review » Fixed

Changing to 6.x-2.x version.

These changes are already done / no longer required in the newest version.

Status: Fixed » Closed (fixed)

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