Ubuntu Software Center

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

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

Description of the change

This branch fixes a crash in the new a11y code if there is a row that has no data or is not yet preloaded.

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

Nice fix, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'softwarecenter/ui/gtk3/widgets/apptreeview.py'
--- softwarecenter/ui/gtk3/widgets/apptreeview.py 2012-04-13 04:46:07 +0000
+++ softwarecenter/ui/gtk3/widgets/apptreeview.py 2012-04-16 09:42:23 +0000
@@ -537,8 +537,11 @@
537 raw = model[path][AppGenericStore.COL_ROW_DATA]537 raw = model[path][AppGenericStore.COL_ROW_DATA]
538 if self.rowref_is_category(raw):538 if self.rowref_is_category(raw):
539 text = raw.display_name539 text = raw.display_name
540 else:540 elif raw:
541 text = self.db.get_pkgname(raw)541 text = self.db.get_pkgname(raw)
542 else:
543 # this can happen for empty/not-yet-loaded row, LP: #981992
544 text = ""
542 cell.set_property('text', text)545 cell.set_property('text', text)
543546
544 def _app_activated_cb(self, btn, btn_id, app, store, path):547 def _app_activated_cb(self, btn, btn_id, app, store, path):

Subscribers

People subscribed via source and target branches