Libnotify plugin does not properly render names with '&' in title

Bug #370668 reported by Abhishek Mukherjee
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Exaile
Fix Released
Undecided
Abhishek Mukherjee

Bug Description

Songs with an & in the title are encoded as & before being passed to libnotify. NotifyOSD at the very least will therefore print out the title as "Always & Never", not the expected "Always & Never". It must be investigated of how the different Galago daemons receive this. NotifyOSD apparently will print it verbatim but do others (read: notification-daemon) do the same?

summary: - Libnotify plugin does not properly render names with & in title
+ Libnotify plugin does not properly render names with '&' in title
Revision history for this message
mlissner (mlissner-michaeljaylissner) wrote :

I can confirm this result using v. 0.2.14.

Revision history for this message
Abhishek Mukherjee (linkinpark342) wrote :

@mlissner
What notification system are you using?

A quick notify-send on my computer (now that i have time to think of such things) shows notification-daemon prints everything verbatim as well. I think the solution is just to not escape the summary since those two systems are the only galago things I can think of (i heard awn might have one as well).

Attached patch seems to fix for notification-daemon and (likely) NotifyOSD as well. I've stress tested this patch on a smorgasborg of songs. and by smorgasborg i mean exactly two songs. The patch is against the 0.3.x branch

Changed in exaile:
assignee: nobody → Abhishek Mukherjee (linkinpark342)
Revision history for this message
mlissner (mlissner-michaeljaylissner) wrote :

If I said libnotify 0.4.5 on Ubuntu Jaunty, would that answer your question?

Revision history for this message
Abhishek Mukherjee (linkinpark342) wrote :

You're probably using NotifyOSD. I'll install that AWN thingie and see if the patch fixes it later today.

Revision history for this message
reacocard (reacocard) wrote :

committed r1866

Changed in exaile:
status: New → Fix Committed
reacocard (reacocard)
Changed in exaile:
milestone: none → 0.3.0
reacocard (reacocard)
Changed in exaile:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.