UILabel bug GetUrlByCharacterIndex

[bowragon]

Hey,

I am not sure if this is fixed already, but I just found a bug under UILabel's GetUrlByCharacterIndex method.

It lacked another check for ending the url bbcode, the [/url] part. So if you click on any words that come after an already enclosed url, it will always return the url instead of null.

This is my solution to the problem:
int urlEnd = mText.IndexOf("[/url]", linkStart);
    if (characterIndex > linkStart &&
        characterIndex < urlEnd)

I just ran a little test and it solved pretty well, but I'm not sure if this is the best solution.

[ArenMook]

Thanks! I've examined the code on my end and indeed it wasn't capping it properly as you've pointed out. I've fixed that issue, and in the process found another small one that I've also fixed.