what causes Warning: preg_match(): Unknown modifier 'p'

Question

I am having trouble I could not fix this error. I am using this upload script to check the mime type.

Warning: preg_match(): Unknown modifier 'p'
Warning: preg_match(): Unknown modifier 'g'
Warning: preg_match(): Unknown modifier '('

if (preg_match('/^image/p?jpeg$/i', $_FILES['upload']['type']) or 

    preg_match('/^image/gif$/i', $_FILES['upload']['type']) or 

    preg_match('/^image/(x-)?png$/i', $_FILES['upload']['type'])) 

{ 

  // Handle the file... 

} 

else 

{ 

  $error = 'Please submit a JPEG, GIF, or PNG image file.'; 

  include $_SERVER['DOCUMENT_ROOT'] . '/includes/error.html.php'; 

  exit(); 

}

Thank you in advance

you have to escape **/** in your regex if you use */* as start-end delimiter — donald123, Jul 13 '15 at 09:22

Jonny 5 · Accepted Answer · 2015-07-13T12:17:54.340

4

Everything after second / (closing delimiter) is considered flags. Use another delimiter such as ~

~^image/p?jpeg$~i

Or to match the delimiter literal inside the pattern escape it using the backslash:

/^image\/p?jpeg$/i

Most comfortable to pick a delimiter, that you don't need inside the pattern > no worry for escaping. Frequently used delimiters are /,~,#,@ or even bracket style delimiters such as ( pattern ).

side note

You could combine all three preg_match to a single one using alternation:

if(preg_match('~^image/(?:gif|p?jpeg|(?:x-)?png)$~i', $_FILES['upload']['type'])) { ... }

(?: starts a non-capture group. Good for testing: regex101.com

edited Jul 13 '15 at 12:17

answered Jul 13 '15 at 09:21

Jonny 5

12,171
2
25
42

you mean like this if (preg_match('/\^image/p?jpeg$/i', $_FILES['upload']['type']) or preg_match('/\^image/gif$/i', $_FILES['upload']['type']) or preg_match('/\^image/(x-)?png$/i', $_FILES['upload']['type'])) – jemz Jul 13 '15 at 09:28
@jemz Like this: `preg_match('~^image/p?jpeg$~i'`... or `preg_match('/^image\/p?jpeg$/i'`... – Jonny 5 Jul 13 '15 at 09:30

what causes Warning: preg_match(): Unknown modifier 'p'

1 Answers1