Again, I don't know if this is parseable with pure regex, but...

http://drupal.org/node/144132#redirect isn't being caught by Coder.

CommentFileSizeAuthor
#1 coder_223168.patch1.74 KBstella

Comments

stella’s picture

stella commented
Status: Active » Needs review
StatusFileSize
new coder_223168.patch1.74 KB

Try the attached patch.

Cheers,
Stella

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

I tried this and it did not catch the error.

douggreen’s picture

douggreen commented
Status: Needs review » Needs work
stella’s picture

stella commented
Status: Needs work » Postponed (maintainer needs more info)

Hmmm this should work. Could you send me some example code where this check fails?

Cheers,
Stella

nancydru’s picture

nancydru
she/her/hers
Primary language English
Location Boston
commented

It's been so long, I don't remember where it happened and I am now attuned to it an catch it myself. You can close it if you want.

stella’s picture

stella commented
Status: Postponed (maintainer needs more info) » Fixed

Ok, well I've committed the patch to CVS as it correctly catches the example scenarios in the test file. If you find a situation in which it does not work as expected, please feel free to re-open this issue.

Cheers,
Stella

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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