You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ponymail.apache.org by se...@apache.org on 2018/05/24 01:25:50 UTC

[incubator-ponymail] branch master updated: Bug: variable 'mid' used before assignment in archiver.py

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-ponymail.git


The following commit(s) were added to refs/heads/master by this push:
     new d2df6c2  Bug: variable 'mid' used before assignment in archiver.py
d2df6c2 is described below

commit d2df6c289149828e08d7d248ed380cae6028c259
Author: Sebb <se...@apache.org>
AuthorDate: Thu May 24 02:25:47 2018 +0100

    Bug: variable 'mid' used before assignment in archiver.py
    
    This fixes #459
---
 CHANGELOG.md      | 1 +
 tools/archiver.py | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index d43917f..8068406 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,4 +1,5 @@
 ## Changes in 0.11-SNAPSHOT
+- Bug: variable 'mid' used before assignment in archiver.py (#459)
 - Enh: remove duplicated code in tools scripts by using elastic.py module (#456)
 - Enh: separate module to read config file 
 - Bug: push-failures.py expects to find non-existent 'id' key in json file (#454)
diff --git a/tools/archiver.py b/tools/archiver.py
index 5659587..429fd94 100755
--- a/tools/archiver.py
+++ b/tools/archiver.py
@@ -478,7 +478,7 @@ class Archiver(object): # N.B. Also used by import-mbox.py
             )
         
         if logger:
-            logger.info("Pony Mail archived message %s successfully" % mid)
+            logger.info("Pony Mail archived message %s successfully" % ojson['mid'])
         oldrefs = []
         
         # Is this a direct reply to a pony mail email?

-- 
To stop receiving notification emails like this one, please contact
sebb@apache.org.