Closed (fixed)
Project:
WYSIWYG Filter (D7)
Version:
6.x-1.3
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
19 Aug 2009 at 19:21 UTC
Updated:
3 Sep 2009 at 17:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
mattyoung commentedBefore I do make a patch, do you think if I just make it case in-sensitive would work?
Comment #2
markus_petrux commentedLook at wysiwyg_filter_get_advanced_rules() in wysiwyg_filter.module. I think we need the following, and then check if that's enough or we need to change something else.
Well, the description also needs to be updated to reflect uppercase letters would be also allowed.
Comment #3
mattyoung commentedhere is a patch
This make class and id names case insensitive to allow both upper or lower case letters to be used.
I just change the regex to be case insensitive and change asterisk expansion to allow upper case letter also.
Comment #4
markus_petrux commentedCommitted to CVS an slight variation (patch). I'm adding A-Z to both regular expressions.
Thank you!
Comment #5
markus_petrux commentedComment #6
mattyoung commented>I'm adding A-Z to both regular expressions
I debated about A-Z vs the i switch. A-Z probably perform better than i?
Comment #7
markus_petrux commentedI opted for A-Z because it is easier to read and compare both regexps. There's no much difference in performance.
Comment #8
mattyoung commentedThank you!