Refactor conversion of None to empty introduced in #49 and #40

gitsome/github.py

@@ -35,6 +35,7 @@
 from .table import Table
 from .view_entry import ViewEntry
 from .web_viewer import WebViewer
+from .utils import TextUtils
 
 
 class GitHub(object):
@@ -68,6 +69,7 @@ def __init__(self):
         self.table = Table(self.config)
         self.web_viewer = WebViewer(self.config)
         self.trend_parser = feedparser
+        self.text_utils = TextUtils()
 
     def authenticate(func):
         """Decorator that authenticates credentials.
@@ -222,7 +224,7 @@ def create_issue(self, user_repo, issue_title, issue_desc=''):
                                              issue_title,
                                              issue_desc)
         if type(issue) is not null.NullObject:
-            body = issue.body if issue.body is not None else ''
+            body = self.text_utils.sanitize_if_none(issue.body)
             click.secho('Created issue: ' + issue.title + '\n' + body,
                         fg=self.config.clr_message)
         else:
@@ -245,7 +247,7 @@ def create_repo(self, repo_name, repo_desc='', private=False):
             repo = self.config.api.create_repository(repo_name,
                                                      repo_desc,
                                                      private=private)
-            desc = repo.description if repo.description is not None else ''
+            desc = self.text_utils.sanitize_if_none(repo.description)
             click.secho(('Created repo: ' + repo.full_name + '\n' + desc),
                         fg=self.config.clr_message)
         except UnprocessableEntity as e:

gitsome/utils.py

@@ -184,3 +184,14 @@ def _safe_split(self, text):
             return words
         except:
             return text
+
+    def sanitize_if_none(self, text):
+        """Sanitizes text to ensure it is not None
+
+        :type text:str
+        :param text: String to sanitize
+
+        :rtype: str
+        :return: String which is empty if None.
+        """
+        return text if text is not None else ''