Merge lp:~gary-lasker/software-center/title-summary-fixes into lp:software-center
Status: | Merged |
---|---|
Merged at revision: | 1172 |
Proposed branch: | lp:~gary-lasker/software-center/title-summary-fixes |
Merge into: | lp:software-center |
Diff against target: |
263 lines (+80/-45) 7 files modified
debian/changelog (+14/-3) softwarecenter/db/application.py (+47/-24) softwarecenter/db/database.py (+5/-8) softwarecenter/view/appview.py (+2/-7) softwarecenter/view/availablepane.py (+5/-1) softwarecenter/view/installedpane.py (+5/-1) softwarecenter/view/softwarepane.py (+2/-1) |
To merge this branch: | bzr merge lp:~gary-lasker/software-center/title-summary-fixes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
software-store-developers | Pending | ||
Review via email: mp+35242@code.launchpad.net |
Description of the change
This branch is to fix bug 636004.
Previously, there were two places that implemented the "special case" code that substituted summary for title and pkgname for summary (as required by the spec). The first was in applist.py for the applist view (where, as mpt noted, the substitution was working fine while the details view was broken). The second was in the db.AppDetails object, and this was used by the app details view.
Since we need to maintain a lightweight object for use with applist, I consolidated the code into a pair of calls (get_display_name and get_display_
Finally, I wired the navigation button to use the display name rather than the simple pkgname as it was using up until now, so that bug is fixed as well. Now, the applist, the appdetails and the navigation bar all share the same code.
I'm sure there are further opportunities for optimization (for instance, once we have appdetails generated for the navigation bar, we could pass it for use in appdetails), but I stopped here as this seemed the minimum surgery to do the trick.
Thanks!
Bah, I'm seeing a freeze when trying to view the app details for the "A Better CD Encoder" package (see bug 636843). This doesn't seem related to my branch as I also see it happening in 2.1.18.1.
Anyway, since this is the test package for this bug, I wanted to report it here as well. You can work around the problem (if you see it) by first doing a search to narrow the length of the list before clicking "More Info" (just type "abcd" into the search bar).