Merge lp:~roignac/software-center/bug-738771-summary-unicode into lp:software-center

Proposed by Vadim Rutkovsky
Status: Merged
Merge reported by: Michael Vogt
Merged at revision: not available
Proposed branch: lp:~roignac/software-center/bug-738771-summary-unicode
Merge into: lp:software-center
Diff against target: 12 lines (+1/-1)
1 file modified
softwarecenter/ui/gtk/appdetailsview_gtk.py (+1/-1)
To merge this branch: bzr merge lp:~roignac/software-center/bug-738771-summary-unicode
Reviewer Review Type Date Requested Status
Michael Vogt Pending
Review via email: mp+68972@code.launchpad.net

Description of the change

Summary is being decoded as 'latin-1' instead of 'utf-8' to avoid problems with some .debs. Not sure about possible regressions though

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

Thanks for this branch! I fixed the bug now based on your idea but in a slightly different way. I still set the branch to "merged" as the bug is gone. Thanks again for your help on this.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'softwarecenter/ui/gtk/appdetailsview_gtk.py'
2--- softwarecenter/ui/gtk/appdetailsview_gtk.py 2011-07-21 19:30:15 +0000
3+++ softwarecenter/ui/gtk/appdetailsview_gtk.py 2011-07-23 20:36:39 +0000
4@@ -1231,7 +1231,7 @@
5 # make title font size fixed as they should look good compared to the
6 # icon (also fixed).
7 markup = '<span font_desc="bold 20">%s</span>\n<span font_desc="9">%s</span>'
8- markup = markup % (appname, GObject.markup_escape_text(summary))
9+ markup = markup % (appname, GObject.markup_escape_text(summary.decode('latin-1')))
10
11 self.title.set_markup(markup)
12 self.title.a11y.set_name(appname + '. ' + summary)