From e3c5e4b4c78ca6347cd26fda4f3936ba2f32ef0b Mon Sep 17 00:00:00 2001 From: Laurent Monin Date: Fri, 16 May 2014 20:38:57 +0200 Subject: [PATCH] Improve __repr__() outputs --- picard/coverartimage.py | 22 ++++++++++++++++++++-- 1 file changed, 20 insertions(+), 2 deletions(-) diff --git a/picard/coverartimage.py b/picard/coverartimage.py index 823aeeab2..436307678 100644 --- a/picard/coverartimage.py +++ b/picard/coverartimage.py @@ -135,7 +135,10 @@ class CoverArtImage: p = [] if self.url is not None: p.append("url=%r" % self.url.toString()) - p.append("types=%r" % self.types) + if self.types: + p.append("types=%r" % self.types) + if self.is_front is not None: + p.append("is_front=%r" % self.is_front) if self.comment: p.append("comment=%r" % self.comment) return "%s(%s)" % (self.__class__.__name__, ", ".join(p)) @@ -144,7 +147,8 @@ class CoverArtImage: p = [u'Image'] if self.url is not None: p.append(u"from %s" % self.url.toString()) - p.append(u"of type %s" % u','.join(self.types)) + if self.types: + p.append(u"of type %s" % u','.join(self.types)) if self.comment: p.append(u"and comment '%s'" % self.comment) return u' '.join(p) @@ -265,3 +269,17 @@ class TagCoverArtImage(CoverArtImage): @property def source(self): return u'Tag %s from %s' % (self.tag if self.tag else '', self.sourcefile) + + def __repr__(self): + p = [] + p.append('%r' % self.sourcefile) + if self.tag is not None: + p.append("tag=%r" % self.tag) + if self.types: + p.append("types=%r" % self.types) + if self.is_front is not None: + p.append("is_front=%r" % self.is_front) + p.append('support_types=%r' % self.support_types) + if self.comment: + p.append("comment=%r" % self.comment) + return "%s(%s)" % (self.__class__.__name__, ", ".join(p))