Merge lp:~dobey/rhythmbox-ubuntuone/undup-lib-entries into lp:rhythmbox-ubuntuone

[dobey]

Add ourselves to the libraries list as an escaped URI
Remove the old non-uri and unescaped uri path entries

[Roberto Alsina (ralsina)]

+1

review: Approve

=== modified file 'ubuntuone/ubuntuone.py'
--- ubuntuone/ubuntuone.py	2012-03-20 20:26:40 +0000
+++ ubuntuone/ubuntuone.py	2012-03-23 18:03:23 +0000
@@ -41,8 +41,18 @@
     def _locations_changed(self, *args, **kwargs):
         """Handle the locations setting being changed."""
         libraries = self.rdbconf.get_strv('locations')
-        if U1LIBRARYPATH not in libraries:
-            libraries.append(U1LIBRARYPATH)
+        library_uri = Gio.File.new_for_path(U1LIBRARYPATH).get_uri()
+        if library_uri not in libraries:
+            libraries.append(library_uri)
+            self.rdbconf.set_strv('locations', libraries)
+        # Remove the non-uri path if it exists
+        if U1LIBRARYPATH in libraries:
+            libraries.remove(U1LIBRARYPATH)
+            self.rdbconf.set_strv('locations', libraries)
+        # Remove the unescaped uri path if it exists
+        unescaped_path = u'file://{0}'.format(U1LIBRARYPATH)
+        if unescaped_path in libraries:
+            libraries.remove(unescaped_path)
             self.rdbconf.set_strv('locations', libraries)
 
     def do_activate(self):