What am I doing wrong with this preg_match?

[DISLEX]

I'm working on a plugin and I'm trying to pull out quotes from a post message ($post[pagetext]). For some reason, it works if there's a quote->img->/img->/quote combination, but if there is any text inside the quote WITH an image, it won't pick it up.

here's my regex: $qc = preg_match_all("/(\[quote)(.*)(\[\/quote\])/i", $post[pagetext], $quotematches, PREG_SET_ORDER);

here are some examples of the regex results:

Originally Posted by example post where $qc = 1

hey, how's it going?

In the above, it would detect the quoted message by "user"

Originally Posted by another example where $qc = 1

oh cool I'm not color blind

in the above, it still detects the quote

HOWEVER, in the following:

Originally Posted by example where $qc = 0

oh cool I'm not color blind

it doesn't detect the quote. I've noticed that it doesn't catch a quote if there's text AND an image inside of the quote. How on earth is that not a match?

[hambil]

I think you are forgetting to deal with the new line between the text and the image

[Dismounted]

Have a look at how vBulletin finds quotes in strip_quotes().

[DISLEX]

Originally Posted by hambil

I think you are forgetting to deal with the new line between the text and the image

does (.*) not match newlines?

[Dismounted]

You need to add a PCRE modifier for it to count new lines as well. IIRC, the modifier is m. However, you should still be using vBulletin's method of detecting quotes, as it does not use costly regex functions.

[DISLEX]

Originally Posted by Dismounted

You need to add a PCRE modifier for it to count new lines as well. IIRC, the modifier is m. However, you should still be using vBulletin's method of detecting quotes, as it does not use costly regex functions.

it says that if there are no start-of-line (^) or end-of-line ($) modifiers, then 'm' has no effect:

http://us2.php.net/pcre.pattern.modifiers

[TigerC10]

No, you add the m modifier at the end, next to the i for case insensitive matches. Kind of like the g modifier (makes it "global", which matches more than one on a single line)...