[PATCH 3/5] Re: Don't pass a list to ui.copyingmessage()

Sebastian Spaeth Sebastian at SSpaeth.de
Mon Aug 15 08:57:51 BST 2011


On Sun, 14 Aug 2011 17:34:45 +0200, Nicolas Sebrecht <nicolas.s-dev at laposte.net> wrote:
> This patch introduce a bug for TTYUI.  I wonder if other UI are touched.
> Sorry, to not dig into it further for now.

Right you are, that was an error on my side, thanks for catching. Here
comes (as a reply) the updated version that works. Interpatch diff is:

-    def copyingmessage(s, uid, src, destfolder):
-        if s.verbose >= 0:
-            s._msg("Copy message %d %s[%s] -> %s[%s]" % \
-                       (uid, s.getnicename(src), src.getname(), 
-                        destfolder.getnicename(), destfolder))
+    def copyingmessage(self, uid, src, destfolder):
+        """Output a log line stating which message we copy"""
+        if self.verbose >= 0:
+            self._msg("Copy message %d %s[%s] -> %s[%s]" % \
+                       (uid, self.getnicename(src), src, 
+                        self.getnicename(destfolder), destfolder))

Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://alioth-lists.debian.net/pipermail/offlineimap-project/attachments/20110815/b4feb8f2/attachment-0001.sig>


More information about the OfflineIMAP-project mailing list