* The `confirm` email command now properly handles `Re:`-like prefixes, even

   if they contain non-ASCII characters.  (LP: #685261)

Also:

 - When the registrar confirms an email address, the verified_on value gets
   set using the Mailman datetime interface.  This simply improves testability
   for that API.
 - In Runner._one_iteration(), be sure to commit the database transaction
   *before* calling _short_circuit().

1 files changed, 2 insertions, 2 deletions

diff --git a/src/mailman/core/runner.py b/src/mailman/core/runner.py
index 30189600d..e86741c41 100644
--- a/src/mailman/core/runner.py
+++ b/src/mailman/core/runner.py
@@ -169,12 +169,12 @@ class Runner:
             # Other work we want to do each time through the loop.
             dlog.debug('[%s] doing periodic', me)
             self._do_periodic()
+            dlog.debug('[%s] committing transaction', me)
+            config.db.commit()
             dlog.debug('[%s] checking short circuit', me)
             if self._short_circuit():
                 dlog.debug('[%s] short circuiting', me)
                 break
-            dlog.debug('[%s] commiting', me)
-            config.db.commit()
         dlog.debug('[%s] ending oneloop: %s', me, len(files))
         return len(files)