Re: Autolink of Ticket References in Comments
Chuck Hagenbuch <[email protected]>
| Newsgroups | gmane.comp.horde.whups |
|---|---|
| Message-ID | <[email protected]> |
Quoting Mike Baptiste <[email protected]>: > I made it a user preference vs a global preference. I was on the fence > about this one (I almost put it in conf.php) and opted to let the user > choose. Another option might be to have the conf file contain the > search regex and the user enable/disable the autolinking itself. I think that would be much better. Would also alleviate any concerns you express below: > Also, you'll note that I put the code AFTER the comment is stripped of > HTML, which means someone MIGHT be able to set their search phrase in a > way that lets them return HTML in ticket views - I guess I could put in > a strip at the end. I wanted to put the added code higher up before > htmlSpaces and enableCapitalLinks, but the Horde::link method retusn a > lower case <a -chuck -- Charles Hagenbuch, <[email protected]> "People ask me all the time what it will be like living without otters." - Google, thanks to Harpers -- Whups mailing list Frequently Asked Questions: http://horde.org/faq/ To unsubscribe, mail: [email protected]