Merge lp:~xapantu/noise/fix-1073187 into lp:~elementary-apps/noise/trunk

[xapantu]

Fix MPRIS-menu. It now correcty updates its visual appereance. Solution is maybe a bit hacky, we will see.

[lp:~xapantu/noise/fix-1073187]

1658. By xapantu on 2014-08-06

Code style

[Raphael Isemann (teemperor)]

Fixed codestyle and works now. It also seems to be no problem for other players (and i'm not aware of a string that should be send instead of null). => approve

Any (hopefully non-existing) problems with that are hopefully discovered in Beta1.

review: Approve (functionality, code-style)

=== modified file 'plugins/MPRIS/MPRIS.vala'
--- plugins/MPRIS/MPRIS.vala	2014-04-19 22:03:29 +0000
+++ plugins/MPRIS/MPRIS.vala	2014-08-06 21:22:11 +0000
@@ -251,11 +251,11 @@
         changed_properties = null;
         
         try {
-            conn.emit_signal("org.mpris.MediaPlayer2.Noise",
-                             "/org/mpris/MediaPlayer2", 
-                             "org.freedesktop.DBus.Properties", 
-                             "PropertiesChanged", 
-                             new Variant("(sa{sv}as)", 
+            conn.emit_signal (null,
+                              "/org/mpris/MediaPlayer2", 
+                              "org.freedesktop.DBus.Properties", 
+                              "PropertiesChanged", 
+                              new Variant("(sa{sv}as)", 
                                          INTERFACE_NAME, 
                                          builder, 
                                          invalidated_builder)
@@ -653,4 +653,4 @@
         }
     }
 
-}
\ No newline at end of file
+}