Possible edge case involving admin overlap

[blankdvth]

The code for changing admins online into a hyperlink may have issues when encountering an overlap. If one admin's name contains another's, it may, in an edge case, result in a double hyper-link or partially hyperlinked name.

Currently, it will replace the first match it finds in the str and iterate through the admins online in order, so a str such as Jayy, Jay, would turn into <a href="etc">Jayy</a>, Jay on the first iteration (it finds Jayy in the list of admins, then hyperlinks it), and <a href="etc"><a href="etc2">Jay</a>y</a>, Jay on the second iteration (in theory, it would find Jay in the list of admins, search left -> right using replace, and find the first Jayy as a result).

Original Conversation

[blankdvth]

A potential solution would be to start replacing at the index in which we are sure the name we intend to replace starts at.

[blankdvth]

Another, much easier solution would be to search and replace , <admin name> as opposed to just the admin name. This should limit replacements to solely entries that have yet to be replaced (if it were already replaced, there would be an anchor tag in front of the name), this may encounter issues if the overlap involves admins not present in the ID list however. (for the first item in the list, just using the admin name should be fine as it's the left most item, all subsequent replacements should use a ,)