Merge lp:~bilalakhtar/update-manager/unity-crash-fix into lp:update-manager

[Bilal Akhtar]

Fix the crash in Unity by using the ENABLED property instead of VISIBLE. We may revert this later when bug #759174 is fixed in Unity.

[Bilal Akhtar (bilalakhtar)]

I guess this would need more discussion before implementation.

I've filed merge request https://code.launchpad.net/~bilalakhtar/unity/fix-unity-crash-on-hidden-quicklistitem/+merge/61067 which fixes the Unity crash on setting VISIBLE to false.

Now we have 2 choices:

1) Go with the current arrangement, since that Unity branch should land in very soon.

2) Use the ENABLED property to disable/enable the Install All Updates button, even when the Unity bug gets fixed.

Its an ergonomic difference. Michael?

[Michael Terry (mterry)]

I'm not the Michael you meant, but I'm just going to go ahead and say we can drop this. The Unity bug got fixed in oneiric. Let's just stay the status quo. mvo can whack me if I'm wrong.

Unmerged revisions

2109. By Bilal Akhtar on 2011-05-16

Use MENUITEM_PROP_ENABLED until Unity is fixed to support the Visible property

=== modified file 'UpdateManager/UnitySupport.py'
--- UpdateManager/UnitySupport.py	2011-05-02 16:19:57 +0000
+++ UpdateManager/UnitySupport.py	2011-05-16 05:23:23 +0000
@@ -74,7 +74,7 @@
             self._unity.set_property("urgent", False)
 
     def set_install_menuitem_visible(self, visible):
-        self.install_dbusmenuitem.property_set_bool(Dbusmenu.MENUITEM_PROP_VISIBLE, visible)
+        self.install_dbusmenuitem.property_set_bool(Dbusmenu.MENUITEM_PROP_ENABLED, visible)
 
 
 # check what to export to the clients