Here's the revised patch. May I respectfully suggest that further discussion
of ancillary issues move to other threads? (I actually started another
thread, and got no responses, while this one continued one more round.)

Jameson
diff --git a/NEWS b/NEWS
index 3910546..06c5a53 100644
--- a/NEWS
+++ b/NEWS
@@ -1,3 +1,7 @@
+84
+
+* #6639 Journal can't resume activity bundles using another activity (homunq)
+
 80
 
 * #5017 Correctly expose some strings to gettext. (tomeu)
diff --git a/activity/activity.info b/activity/activity.info
old mode 100755
new mode 100644
diff --git a/journaltoolbox.py b/journaltoolbox.py
index 08b6157..c4d5bee 100644
--- a/journaltoolbox.py
+++ b/journaltoolbox.py
@@ -415,12 +415,13 @@ class EntryToolbar(gtk.Toolbar):
         for menu_item in palette.menu.get_children():
             palette.menu.remove(menu_item)
             menu_item.destroy()
-
-        if not self._jobject.is_activity_bundle():
-            for activity in self._jobject.get_activities():
-                menu_item = MenuItem(activity.name)
-                menu_item.set_image(Icon(file=activity.icon, icon_size=gtk.ICON_SIZE_MENU))
-                menu_item.connect('activate', self._resume_menu_item_activate_cb,
-                                  activity.bundle_id)
-                palette.menu.append(menu_item)
-                menu_item.show()
+            
+        activities = self._jobject.get_activities()
+        for handler in activities:
+            menu_item = MenuItem(handler.name)
+            menu_item.set_image(Icon(file=handler.icon, 
+                                     icon_size=gtk.ICON_SIZE_MENU))
+            menu_item.connect('activate', self._resume_menu_item_activate_cb,
+                              handler.bundle_id)
+            palette.menu.append(menu_item)
+            menu_item.show()
_______________________________________________
Sugar mailing list
[email protected]
http://lists.laptop.org/listinfo/sugar

Reply via email to