From 70275f465c6410061ecbed37f2eed791decaa446 Mon Sep 17 00:00:00 2001
From: Vincent Le Gallic <legallic@crans.org>
Date: Sat, 12 Mar 2016 13:46:19 +0100
Subject: [PATCH] On n'affiche pas "None" pour un contexte d'une quote
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Ça vient du fait que les contextes sont proprifés au restore.
Une quote qui n'est enregistrée qu'en JSON ou en python a None
comme place.
Ce commite implémente des properties place_proper et quoter_proper
pour faire tout ça proprement.
---
 quotes.py | 30 ++++++++++++++++++++++++------
 1 file changed, 24 insertions(+), 6 deletions(-)

diff --git a/quotes.py b/quotes.py
index f68b343..e8e4061 100644
--- a/quotes.py
+++ b/quotes.py
@@ -57,6 +57,22 @@ class Quote(object):
              "place" : self.place, "quoter" : self.quoter}
         return d
     
+    def __get_proper_place(self):
+        """
+        property function pour récupérer ``self.place``
+        mais en virant None ou des chaînes ne contenant que des whitespace.
+        """
+        return self.place if self.place and self.place.strip() != u"" else u""
+    proper_place = property(__get_proper_place)
+    
+    def __get_proper_quoter(self):
+        """
+        property function pour récupérer ``self.place``
+        mais en virant None ou des chaînes ne contenant que des whitespace.
+        """
+        return self.quoter if self.quoter and self.quoter.strip() != u"" else u""
+    proper_quoter = property(__get_proper_quoter)
+    
     def __unicode__(self):
         """ Retourne la quote affichable """
         return config.quote_template % self.__dict__
@@ -70,16 +86,18 @@ class Quote(object):
         """
         s = config.quote_template % self.__dict__
         if show_context:
-            s = u"%s | %s" % (s, self.place)
+            s = u"%s | %s" % (s, self.proper_place)
         return s.encode("utf-8")
-
+    
     def full_str(self):
         """ Retourne une chaîne représentant la totalité des infos de la quote,
             tout en étant parsable et human-readable. """
-        place = self.place if self.place and self.place.strip() != u"" else u""
-        quoter = self.quoter if self.quoter and self.quoter.strip() != u"" else u""
-        return (u"%s %s | %s | %s" % (self.timestamp.strftime("%F_%T"), config.quote_template % self.__dict__
-                  , place, quoter)).encode("utf-8")
+        s = u"%s %s | %s | %s" % (
+            self.timestamp.strftime("%F_%T"),
+            config.quote_template % self.__dict__,
+            self.proper_place,
+            self.proper_quoter)
+        return s.encode("utf-8")
     
     def __eq__(self, otherquote):
         """ Vérifie si cette phrase n'a pas déjà été dite par la même personne.
-- 
2.39.2