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

Proposed by Ritesh Khadgaray
Status: Merged
Approved by: dobey
Approved revision: 3261
Merged at revision: 3318
Proposed branch: lp:~mvo/software-center/lp926763
Merge into: lp:software-center
Diff against target: 40 lines (+13/-3)
1 file modified
softwarecenter/ui/gtk3/views/appdetailsview.py (+13/-3)
To merge this branch: bzr merge lp:~mvo/software-center/lp926763
Reviewer Review Type Date Requested Status
Ritesh Khadgaray (community) Approve
dobey Approve
Review via email: mp+187206@code.launchpad.net

Commit message

Don't require network access for installing local .deb packages.

Description of the change

Cannot install deb files without network connection (lp: #926763)

To post a comment you must log in.
Revision history for this message
dobey (dobey) wrote :

Please use bzr commit --fixes to link the branch, instead.

review: Needs Fixing
lp:~mvo/software-center/lp926763 updated
3261. By Michael Vogt

added missing --fixes lp:926763

Revision history for this message
Ritesh Khadgaray (khadgaray) :
review: Needs Resubmitting
Revision history for this message
Ritesh Khadgaray (khadgaray) wrote :

Please ignore the last comment.

Revision history for this message
dobey (dobey) wrote :

Ritesh, voting resubmit does not "resubmit" the branch. Please change your vote to abstain or approve.

Revision history for this message
dobey (dobey) :
review: Approve
Revision history for this message
Ritesh Khadgaray (khadgaray) :
review: Abstain
Revision history for this message
Ritesh Khadgaray (khadgaray) wrote :

fixed.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'softwarecenter/ui/gtk3/views/appdetailsview.py'
2--- softwarecenter/ui/gtk3/views/appdetailsview.py 2012-12-17 09:53:32 +0000
3+++ softwarecenter/ui/gtk3/views/appdetailsview.py 2013-09-30 11:44:19 +0000
4@@ -35,7 +35,10 @@
5 from softwarecenter.netstatus import (NetState, get_network_watcher,
6 network_state_is_connected)
7 from softwarecenter.db.application import Application
8-from softwarecenter.db import DebFileApplication
9+from softwarecenter.db.debfile import (
10+ DebFileApplication,
11+ AppDetailsDebFile,
12+ )
13 from softwarecenter.backend.reviews import ReviewStats
14 from softwarecenter.enums import (AppActions,
15 PkgStates,
16@@ -452,7 +455,14 @@
17 AppActions.APPLY)):
18 self.set_label(self.app_details.warning)
19
20- sensitive = network_state_is_connected()
21+ connected = network_state_is_connected()
22+ self.set_network_is_connected(connected)
23+
24+ def set_network_is_connected(self, have_network):
25+ sensitive = have_network
26+ # debs can always be installed(?)
27+ if isinstance(self.app_details, AppDetailsDebFile):
28+ sensitive = True
29 self.button.set_sensitive(sensitive)
30
31 def _convert_purchase_date_str_to_datetime(self, purchase_date):
32@@ -911,7 +921,7 @@
33
34 # set addon table and action button states based on sensitivity
35 sensitive = state in NetState.NM_STATE_CONNECTED_LIST
36- self.pkg_statusbar.button.set_sensitive(sensitive)
37+ self.pkg_statusbar.set_network_is_connected(sensitive)
38 self.addon_view.addons_set_sensitive(sensitive)
39 self.addons_statusbar.button_apply.set_sensitive(sensitive)
40 self.addons_statusbar.button_cancel.set_sensitive(sensitive)

Subscribers

People subscribed via source and target branches