Project

General

Profile

Bug #278

Dbus notifications blank if message contains any of: < > &

Added by TerrorBite over 16 years ago. Updated almost 16 years ago.

Status:
Resolved
Priority:
High
Assignee:
Category:
Quassel GUI (Qt)
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Version:
0.13.1
OS:
Any

Description

The notification daemon expects to receive marked up text. Sending it these raw characters < > & causes it to choke on what it sees as invalid markup, and display a notification with a blank body. (XChat also has this issue; they never fixed it.)

A minor annoyance is the notifications always appearing at the location specified in the settings, instead of letting the notification daemon choose where to display it (I have mine set to pop up notifications at top right). It would be nice to have the X and Y position settings optional, and even better, having an option to set the X and Y to wherever the Quassel tray icon happens to be on screen.

The attached patch fixes the blank notifications issue by replacing the offending characters with their HTML entities (< > &) which are correctly decoded and displayed by the notification daemon.
The patch also disables setting the X and Y position hints, disregarding the settings, and letting the daemon decide on position.

Additional information:

The patch is far from optimal. As noted above, there should be options to allow the user to select if the position hints should be sent or not. I don't know anything about the settings system, so I haven't touched it.
Also, Quassel should query the notification daemon about its capabilities, and only send markup if the daemon supports it.

dbus-notifications-html-and-no-hints.patch (1.44 KB) dbus-notifications-html-and-no-hints.patch admin, 08/14/2008 04:55 PM
mainwin.patch (5.59 KB) mainwin.patch admin, 08/26/2008 01:02 PM

Associated revisions

Revision 0c498574 (diff)
Added by Manuel Nickschas about 16 years ago

Complete and pimp DesktopNotificationBackend.

You'll need to reset your notification settings, as things have moved around in the
settings files.
Thanks to TerrorBite for providing the patches in BR #278, this information was
very valuable!

History

#1 Updated by TerrorBite over 16 years ago

Mantis ate my HTML entities, description should read:

"The attached patch fixes the blank notifications issue by replacing the offending characters with their HTML entities (&lt; &gt; &amp;) which are correctly decoded and displayed by the notification daemon."

#2 Updated by xAFFE about 16 years ago

You can use QT::escape for this.

http://doc.trolltech.com/4.4/qt.html#escape

Also you need to check if the daemon is capable of displaying HTML.

#3 Updated by TerrorBite about 16 years ago

New patch uses Qt:escape() and also makes the "Click Me!" button on the notification work.

#4 Updated by Sputnick about 16 years ago

Fixed in <http://git.quassel-irc.org/?p=quassel.git;a=commit;h=0c498574d2f26bae581bc8f9a01e739e1f9c3872>

Thanks TerrorBite, I used some of that information in here, albeit not your patch as I restructured the whole notification thingy :)

Also available in: Atom PDF