notifyOSD shows formatting: eg &

[Matthew Waters]

Found on both Exaile 0.3.2.0 (installed from source) and revision no 3571.
Both installations are running Ubuntu 10.04 that are completely updated.

The expected behaviour is that when the OSD is displayed, the information such as the title, artist and album do not contain any formatting such as &

The actual behaviour is that when the OSD is diplayed, the information contains an "&" where there should be a "&"

[Matthew Waters (waters-matthew00)]

The problem seems to occur when one calls, cgi.excape() on the text. Without that function, the ampersand (the ampersand is the only one that doesn't work that cgi.escape() affects. see here: http://wiki.python.org/moin/EscapingHtml) is printed correctly. This is also supported by the attached test which displays all the characters that cgi.escape() should change in both character format and HTML format.

[Matthew Waters (waters-matthew00)]

The problem seems to occur when one calls, cgi.excape() on the text. Without that function, the ampersand (the ampersand is the only one that doesn't work that cgi.escape() affects. see here: http://wiki.python.org/moin/EscapingHtml) is printed correctly. This is also supported by the attached test which displays all the characters that cgi.escape() should change in both character format and HTML format.

[Mathias Brodala (mathbr)]

What notification server are you using?

[Matthew Waters (waters-matthew00)]

Notification Server:

Name: notify-osd
Vendor: Canonical Ltd
Version: 1.0
Spec Version: 1.1
Capabilities:
 body
 body-markup
 icon-static
 image/svg+xml
 x-canonical-private-synchronous
 x-canonical-append
 x-canonical-private-icon-only
 x-canonical-truncation
 private-synchronous
 append
 private-icon-only
 truncation

[Steve Dodier-Lazaro (sidi)]

It took me some time due to me having to fix another bug that what introduced when the covers and icons manager were changed. Patch attached and available on mupuf.org as usual.

I hope that removing cgi.escape will not raise any issue. I do not remember why I put it in a first time but I think there was a reason...

[Mathias Brodala (mathbr)]

The plugin does not show this issue with the latest trunk.