Trouble with input filter configuration and processing

open-keywords - November 14, 2008 - 22:54
Project:Signwriter
Version:6.x-2.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Justin W Freeman
Status:closed
Description

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

AttachmentSize
signwriterescapesdev.patch3.37 KB

#1

Justin W Freeman - November 19, 2008 - 06:18
Assigned to:Anonymous» Justin W Freeman
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.

#2

Justin W Freeman - May 25, 2009 - 11:40
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.

#3

Justin W Freeman - July 15, 2009 - 12:12
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.

#4

System Message - July 29, 2009 - 12:20
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.