Simplify (& fix) Message().__str__()

We were still using len(self.get_replies()) for the __str__ summary of a
mail, but 1) len(Messages()) has just gone away 2) the number of replies
can not be retrieved when we got the message via search_messages()
anyway, and 3) it is likely quite expensive to pull all replies for all
messages that we display a summary of.

So we fix this by simplifying str(Message()) to omit the number of replies.

Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
This commit is contained in:
Sebastian Spaeth 2011-06-16 17:33:47 +02:00
parent 995303c21b
commit 12d6f90e77

View file

@ -749,9 +749,7 @@ class Message(object):
msg['from'] = self.get_header('from') msg['from'] = self.get_header('from')
msg['tags'] = str(self.get_tags()) msg['tags'] = str(self.get_tags())
msg['date'] = date.fromtimestamp(self.get_date()) msg['date'] = date.fromtimestamp(self.get_date())
replies = self.get_replies() return "%(from)s (%(date)s) (%(tags)s)" % (msg)
msg['replies'] = len(replies) if replies is not None else 0
return "%(from)s (%(date)s) (%(tags)s) %(replies)d replies" % (msg)
def get_message_parts(self): def get_message_parts(self):