| Summary: | Telepathy chat handler places emoticons in URLs | ||
|---|---|---|---|
| Product: | [Unmaintained] telepathy | Reporter: | Geert Lamers <bugs.kde> |
| Component: | text-ui | Assignee: | Telepathy Bugs <kde-telepathy-bugs> |
| Status: | RESOLVED FIXED | ||
| Severity: | minor | CC: | kde |
| Priority: | NOR | ||
| Version: | git-latest | ||
| Target Milestone: | 0.4.0 | ||
| Platform: | Gentoo Packages | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed In: | ||
| Sentry Crash Report: | |||
| Attachments: | Patch to fix wrongly parsed urls containing emoticons. | ||
Patch looks good. Do you have a KDE git account? If so, please merge it! Otherwise write back, and one of us will merge for you. (sorry for the late reply) Thanks a lot. I don't have a git account, go ahead and merge it. Glad to help :) Git commit d59d56ab545192fef0c0961fdc8d0760f86b99a0 by David Edmundson, on behalf of Geert Lamers. Committed on 02/08/2011 at 01:44. Pushed by davidedmundson into branch 'master'. Fix a bug in which emoticons can appear in URLs in messages, by swapping the order of processing BUG: 278790 Reviewed by: David Edmundson M +6 -3 lib/adium-theme-view.cpp http://commits.kde.org/telepathy-chat-handler/d59d56ab545192fef0c0961fdc8d0760f86b99a0 It seems this issue has reappeared. Git commit ff89d55f6df486ad8ccf6c7a85e43dd2584b733c by David Edmundson. Committed on 07/02/2012 at 19:54. Pushed by davidedmundson into branch 'master'. Fix emoticons appearing in URLs (and breaking everything) M +1 -1 lib/message-processor.cpp http://commits.kde.org/telepathy-text-ui/ff89d55f6df486ad8ccf6c7a85e43dd2584b733c We recently redid all the message processing, and in the process reversed some ordering. That was really quickly spotted, thanks again! |
Created attachment 62306 [details] Patch to fix wrongly parsed urls containing emoticons. Version: git-latest (using KDE 4.7.0) OS: Linux When a chat partner sends a URL, the chat handler replaces emoticons before the message is parsed for links. Any URL containing an emoticon is then no longer recognised as URL and thus not converted to link. In my case, the ':/' part of the protocol is replaced, rendering 90% of all links received unclickable. Reproducible: Always Steps to Reproduce: Send (or receive) a URL containing a character sequence that is supposed to be replaced by an emoticon (for example, a URL prefixed with the protocol, where ':/' may be replaced by an emoticon (depends on emoticon theme)). Actual Results: A funny face appears in the middle of the URL, link cannot be clicked. Expected Results: No funny face, clickable link. I've added a patch that reverses the order (parse for links first, then for emoticons) that seems to fix it.