Merge lp:~mvo/software-center/lp976525 into lp:software-center

Proposed by Michael Vogt
Status: Merged
Merged at revision: 2973
Proposed branch: lp:~mvo/software-center/lp976525
Merge into: lp:software-center
Diff against target: 16 lines (+4/-2)
1 file modified
softwarecenter/db/application.py (+4/-2)
To merge this branch: bzr merge lp:~mvo/software-center/lp976525
Reviewer Review Type Date Requested Status
Gary Lasker (community) Approve
Review via email: mp+101905@code.launchpad.net

Description of the change

Remove the upgrade button again as it means we can't remove apps that are upgradable. LP: #976525

To post a comment you must log in.
Revision history for this message
Gary Lasker (gary-lasker) wrote :

Small and sweet! This is another really nice one to have fixed for LTS. Many thanks, mvo!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'softwarecenter/db/application.py'
--- softwarecenter/db/application.py 2012-03-20 08:40:10 +0000
+++ softwarecenter/db/application.py 2012-04-13 12:43:20 +0000
@@ -482,8 +482,10 @@
482 for origin in self._pkg.installed.origins]482 for origin in self._pkg.installed.origins]
483 if not self._app.archive_suite in archive_suites:483 if not self._app.archive_suite in archive_suites:
484 return PkgStates.FORCE_VERSION484 return PkgStates.FORCE_VERSION
485 if self._pkg.installed and self._pkg.is_upgradable:485 # Don't handle upgrades yet, see bug LP #976525 we need more UI
486 return PkgStates.UPGRADABLE486 # for this
487 #if self._pkg.installed and self._pkg.is_upgradable:
488 # return PkgStates.UPGRADABLE
487 if self._pkg.is_installed:489 if self._pkg.is_installed:
488 return PkgStates.INSTALLED490 return PkgStates.INSTALLED
489 else:491 else:

Subscribers

People subscribed via source and target branches