1
0
mirror of https://github.com/tomahawk-player/tomahawk.git synced 2025-08-15 02:24:50 +02:00

Having this replace in there actually breaks notification for me.

I'm guessing that this is notification-system dependent; however, we
can't show & to people not using HTML-based notification renderers.

The fix needs to lie with the notification renderer vendor.
This commit is contained in:
Jeff Mitchell
2012-07-05 13:39:12 -04:00
parent ba4532593b
commit 10462ee257

View File

@@ -109,7 +109,7 @@ FdoNotifyPlugin::notifyUser( const QString &messageText )
arguments << quint32( 0 ); //notification_id
arguments << QString(); //app_icon
arguments << QString( "Tomahawk" ); //summary
arguments << QString( messageText ).replace("&", "&amp;"); //body
arguments << QString( messageText ); //body
arguments << QStringList(); //actions
QVariantMap dict;
dict["desktop-entry"] = QString( "tomahawk" );
@@ -150,7 +150,7 @@ FdoNotifyPlugin::nowPlaying( const QVariant &input )
arguments << m_nowPlayingId; //notification_id
arguments << QString(); //app_icon
arguments << QString( "Tomahawk" ); //summary
arguments << messageText.replace("&", "&amp;"); //body
arguments << messageText; //body
arguments << QStringList(); //actions
QVariantMap dict;
dict["desktop-entry"] = QString( "tomahawk" );