When the pattern happens to include an email address (ie: test@gmail.com) we get:

Warning: preg_match(): Unknown modifier 'g' in _protected_submissions_validate()...

We need to quote the patterns that contain the delimiter used in this module. I'll submit a patch for it shortly.

CommentFileSizeAuthor
#7 protected_submissions-preg_match_warning-3118715-7.patch623 bytesbogdan.dinu
#2 preg_match_warning-3118715-2.patch667 bytesfjgarlin

Comments

fjgarlin created an issue. See original summary.

fjgarlin’s picture

fjgarlin commented
StatusFileSize
new preg_match_warning-3118715-2.patch667 bytes

Patch attached.

fjgarlin’s picture

fjgarlin commented
Status: Active » Needs review

  • fjgarlin authored 9c253b0 on 8.x-1.x 
    Issue #3118715 by fjgarlin: preg_match warning when the pattern is an...
altagrade’s picture

altagrade commented
Status: Needs review » Fixed

Fixed on dev. Thanks for the patch!

Status: Fixed » Closed (fixed)

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

bogdan.dinu’s picture

bogdan.dinu commented
Version: 8.x-1.x-dev » 8.x-1.9
StatusFileSize
new protected_submissions-preg_match_warning-3118715-7.patch623 bytes

I think there is a regression regarding this bug. Version 1.9 is throwing the same warning.
After looking at the code, I see the pattern is not quoted anymore.
I created a new patch.

AltaGrade credited brad.bulger.

altagrade’s picture

altagrade commented
Version: 8.x-1.9 » 8.x-1.x-dev

Please test the dev version and re-open this issue if it doesn't work. Note that all patches should be filed against the development version.