[SOLVED] Small bug with 'bbcode' regex

[Garrett]

I have the following:

 

$bb_replace = array ('/(\[[bb]\])(.+)(\[\/[bb]\])/s','/(\[url=)(.+)(\])(.+)(\[\/url\])/s','/(\[code=)(.+)(\])(.+)(\[\/code\])/s');

 

Now it works...but it appears to use the last occurrence of a tag. For example:

 

[ code=test]
something
...
...
[ /code]
[ code=test2]
something
...
...
[ /code]

[ code=test] will use the very last occurrence of [ /code], thus combining everything into the same block.

 

Hopefully my description of the bug is easy to understand what is going on. Anything that can help would be great.

[cags]

Regular expressions are by default greedy and will go as far as they can to match a pattern rather than matching it at the first opportunity. You need to either use the question mark to make each required item lazy or alternatively you could use the U modifier to make matches ungreedy by default. Just bear in mind that nested tags will bring up a whole different set of problems.