Replace <qt> tags in our KNotification event paylods with <html> tags.
authorEike Hein <hein@kde.org>
Mon, 17 May 2010 02:46:48 +0000 (04:46 +0200)
committerEike Hein <hein@kde.org>
Mon, 17 May 2010 02:46:48 +0000 (04:46 +0200)
commitfa49cee5e4804387ca212b55375accb62e5e7332
tree91e083f2e95075b5889a366f34495beb57a5f0f8
parent1bbddaf97fe8f0cd828d9555548dc2b072ba378d
Replace <qt> tags in our KNotification event paylods with <html> tags.

The <qt> was introduced to denote a Qt rich-text document, and used to
be necessary to make the HTML entities we substitute for the pointy
brackets around nicknames work. Using HTML entities was necessary to
avoid the nicknames getting parsed as tags and dropped as unknown. How-
ever, KNotification now forwards popup requests to freedesktop.org-
compliant frontends via D-Bus, and those frontends may not be implemen-
ted using Qt and thus can't handle the <qt> tag (an example of this is
apparently the Awesome window manager). KNotify also makes no attempt
to filter them from the payload. According to the Qt 4 documentation,
<qt> is now a synonym for <html> that is provided for backward compati-
bility. Thus, we now use <html>. This was also adviced by KNotify main-
tainer Olivier Goffart.

BUG:237544
CCMAIL:ogoffart@kde.org
ChangeLog
src/notificationhandler.cpp