On n'affiche pas "None" pour un contexte d'une quote

[bots/parrot.git] / quotes.py

diff --git a/quotes.py b/quotes.py
index 39e048dd179873afff176c4db307923e5e6d5b1c..e8e40610b0b67e7e013b445463acd465e6d29a0f 100644 (file)
--- a/quotes.py
+++ b/quotes.py
@@ -16,19 +16,19 @@ quote_matcher_with_timestamp = re.compile(config.quote_regexp_with_timestamp, fl
 spaces_matcher = re.compile(u"\s", flags=re.U)
 
 def equivalence_partition(iterable, relation):
-       """ Partitionne l'itérable en classes d'équivalences. """
-       classes = []
-       for o in iterable:
-               # find the class it is in
-               found = False
-               for c in classes:
-                       if relation( iter(c).next(), o ): # is it equivalent to this class?
-                               c.add( o )
-                               found = True
-                               break
-               if not found: # it is in a new class
-                       classes.append( set( [ o ] ) )
-       return classes
+    """ Partitionne l'itérable en classes d'équivalences. """
+    classes = []
+    for o in iterable:
+        # find the class it is in
+        found = False
+        for c in classes:
+            if relation( iter(c).next(), o ): # is it equivalent to this class?
+                c.add( o )
+                found = True
+                break
+        if not found: # it is in a new class
+            classes.append( set( [ o ] ) )
+    return classes
 
 def get_now():
     """ Renvoie la date actuelle """
@@ -57,18 +57,47 @@ 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__
     def __str__(self):
         return unicode(self).encode("utf-8")
     
+    def display(self, show_context=False):
+        """
+        Retourne une chaîne contenant toujours la quote et l'auteur,
+        et le contexte ssi ``show_context = True``.
+        """
+        s = config.quote_template % self.__dict__
+        if show_context:
+            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 else ""
-        quoter = self.quoter if self.quoter else ""
-        return (u"%s %s | %s | %s" % (self.timestamp.strftime("%F_%T"), unicode(self), 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.
@@ -123,7 +152,7 @@ class QuoteDB(object):
             return author
     
     def get_clash_authors(self):
-        """ Renvoie une liste de liste d'auteurs qui sont enresgitrés avec des casses différentes. """
+        """ Renvoie une liste de liste d'auteurs qui sont enregistrés avec des casses différentes. """
         authors = list(set([q.author for q in self.quotelist]))
         authors = equivalence_partition(authors, lambda x,y: x.lower() == y.lower())
         authors = [list(c) for c in authors if len(c) > 1]
@@ -137,7 +166,6 @@ class QuoteDB(object):
         kwargs["timestamp"] = timestamp
         newquote = Quote(**kwargs)
         if not newquote in self.quotelist:
-            self.search
             self.quotelist.append(newquote)
             return True
         return False