[Sugar-devel] [PATCH sugar] Journal: rename object_id argument to object_id_or_path

Bert Freudenberg bert at freudenbergs.de
Tue Jun 5 06:47:44 EDT 2012


In particular, this shows up in DBus introspection. The rename should
alert API users not to expect only an object_id.

Signed-off-by: Bert Freudenberg <bert at freudenbergs.de>
---
 src/jarabe/journal/journalactivity.py |   24 ++++++++++++------------
 1 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/src/jarabe/journal/journalactivity.py b/src/jarabe/journal/journalactivity.py
index bb1c7f6..bb45cf2 100644
--- a/src/jarabe/journal/journalactivity.py
+++ b/src/jarabe/journal/journalactivity.py
@@ -66,19 +66,19 @@ class JournalActivityDBusService(dbus.service.Object):
 
     @dbus.service.method(J_DBUS_INTERFACE,
         in_signature='s', out_signature='')
-    def ShowObject(self, object_id):
-        """Pop-up journal and show object with object_id"""
+    def ShowObject(self, object_id_or_path):
+        """Pop-up journal and show object_id_or_path"""
 
-        logging.debug('Trying to show object %s', object_id)
+        logging.debug('Trying to show object %s', object_id_or_path)
 
-        if self._parent.show_object(object_id):
+        if self._parent.show_object(object_id_or_path):
             self._parent.reveal()
 
     def _chooser_response_cb(self, chooser, response_id, chooser_id):
         logging.debug('JournalActivityDBusService._chooser_response_cb')
         if response_id == gtk.RESPONSE_ACCEPT:
-            object_id = chooser.get_selected_object_id()
-            self.ObjectChooserResponse(chooser_id, object_id)
+            object_id_or_path = chooser.get_selected_object_id()
+            self.ObjectChooserResponse(chooser_id, object_id_or_path)
         else:
             self.ObjectChooserCancelled(chooser_id)
         chooser.destroy()
@@ -99,7 +99,7 @@ class JournalActivityDBusService(dbus.service.Object):
         return chooser_id
 
     @dbus.service.signal(J_DBUS_INTERFACE, signature='ss')
-    def ObjectChooserResponse(self, chooser_id, object_id):
+    def ObjectChooserResponse(self, chooser_id, object_id_or_path):
         pass
 
     @dbus.service.signal(J_DBUS_INTERFACE, signature='s')
@@ -224,8 +224,8 @@ class JournalActivity(JournalWindow):
             self.set_canvas(self._main_view)
             self._main_view.show()
 
-    def _show_secondary_view(self, object_id):
-        metadata = model.get(object_id)
+    def _show_secondary_view(self, object_id_or_path):
+        metadata = model.get(object_id_or_path)
         try:
             self._detail_toolbox.entry_toolbar.set_metadata(metadata)
         except Exception:
@@ -242,12 +242,12 @@ class JournalActivity(JournalWindow):
         self.set_canvas(self._secondary_view)
         self._secondary_view.show()
 
-    def show_object(self, object_id):
-        metadata = model.get(object_id)
+    def show_object(self, object_id_or_path):
+        metadata = model.get(object_id_or_path)
         if metadata is None:
             return False
         else:
-            self._show_secondary_view(object_id)
+            self._show_secondary_view(object_id_or_path)
             return True
 
     def __volume_changed_cb(self, volume_toolbar, mount_point):
-- 
1.7.3.2



More information about the Sugar-devel mailing list