Howdy, Stranger!

It looks like you're new here. If you want to get involved, click one of these buttons!

Try Vanilla Forums Cloud product

Code review request: Gdn_Format::Url regex

LincLinc Vanilla's Bard (and Director of Development)Detroit Vanilla Staff
edited December 2012 in Development

I refactored the Url method, which is pretty core to the system. I found a lot of problems in it, especially weirdly over-expressive regex and a lot of duplicated code. If anyone who feels good about regex wants to have a look to make sure I didn't commit a grievous error I'd appreciate it. I'm especially interested in testing for my rewritten $UnicodeSupport test.

https://github.com/vanillaforums/Garden/commit/f7f348b419b6a3600d00ac9d66bc279f469db99e

Sign In or Register to comment.