Merge lp:~evfool/software-center/867588 into lp:software-center/5.2

[Robert Roth]

This branch capitalizes the first letter of the package summary, as requested in the specification and in bug #867588.
 I have also added myself in the AUTHORS list.

[Gary Lasker (gary-lasker)]

Very nice, Robert, thank you! Please note that I added a very small unit test for the new method you added in utils.py.

And indeed, your entry is long overdue in the AUTHORS list. :)

Thanks again for your good work, it's very much appreciated!

review: Approve

=== modified file 'AUTHORS'
--- AUTHORS	2011-09-13 10:25:29 +0000
+++ AUTHORS	2012-05-11 14:16:18 +0000
@@ -26,3 +26,4 @@
 Zygmunt Krynicki - Code
 Aaron Peachey - Code
 Konstantin Datz - Icon
+Robert Roth - Code

=== modified file 'softwarecenter/db/application.py'
--- softwarecenter/db/application.py	2012-05-04 01:13:54 +0000
+++ softwarecenter/db/application.py	2012-05-11 14:16:18 +0000
@@ -32,7 +32,7 @@
 from softwarecenter.paths import (APP_INSTALL_CHANNELS_PATH,
                                   SOFTWARE_CENTER_ICON_CACHE_DIR,
                                   )
-from softwarecenter.utils import utf8, split_icon_ext
+from softwarecenter.utils import utf8, split_icon_ext, capitalize_first_word
 from softwarecenter.region import get_region_cached, REGIONTAG
 
 LOG = logging.getLogger(__name__)
@@ -111,7 +111,7 @@
                     appname = "%s (%s)" % (appname, db.get_pkgname(doc))
                 return appname
             else:
-                return db.get_summary(doc)
+                return capitalize_first_word(db.get_summary(doc))
 
     @staticmethod
     def get_display_summary(db, doc):
@@ -121,7 +121,7 @@
         """
         if doc:
             if db.get_appname(doc):
-                return db.get_summary(doc)
+                return capitalize_first_word(db.get_summary(doc))
             else:
                 return db.get_pkgname(doc)
 
@@ -700,7 +700,7 @@
             return ver.summary
         # normal case
         if self._doc:
-            return self._db.get_summary(self._doc)
+            return capitalize_first_word(self._db.get_summary(self._doc))
         elif self._pkg:
             return self._pkg.candidate.summary
 

=== modified file 'softwarecenter/ui/gtk3/models/appstore2.py'
--- softwarecenter/ui/gtk3/models/appstore2.py	2012-05-01 00:18:23 +0000
+++ softwarecenter/ui/gtk3/models/appstore2.py	2012-05-11 14:16:18 +0000
@@ -32,6 +32,7 @@
     ExecutionTime,
     SimpleFileDownloader,
     split_icon_ext,
+    capitalize_first_word,
     utf8,
     )
 from softwarecenter.backend import get_install_backend
@@ -188,9 +189,9 @@
         # but "packages" are displayed with their summary as name
         if app.appname:
             appname = self.get_appname(doc)
-            summary = self.db.get_summary(doc)
+            summary = capitalize_first_word(self.db.get_summary(doc))
         else:
-            appname = self.db.get_summary(doc)
+            appname = capitalize_first_word(self.db.get_summary(doc))
             summary = self.get_pkgname(doc)
 
         return "%s\n<small>%s</small>" % (

=== modified file 'softwarecenter/utils.py'
--- softwarecenter/utils.py	2012-05-11 09:13:51 +0000
+++ softwarecenter/utils.py	2012-05-11 14:16:18 +0000
@@ -120,6 +120,12 @@
     return wrapper
 
 
+def capitalize_first_word(string):
+    """ this takes a package synopsis and uppercases the first  word's first letter"""
+    if string[0].isalpha() and not string[0].isupper():
+        return string[0].capitalize() + string[1:]
+    return string
+    
 def normalize_package_description(desc):
     """ this takes a package description and normalizes it
         so that all uneeded \n are stripped away and all