Factorisation

src/ninjabot.py

@@ -8,6 +8,7 @@ import threading
 import re
 import time
 import quopri
+import secrets
 
 def write_subs(subs):
     with open('subs.py', 'w') as s:
@@ -192,14 +193,15 @@ def idle_loop(host, user, password, ninja):
                 if comment: comment = comment.group('comment')
                 if page_user:
                     ninja.push_update(page_user.group('page'), page_user.group('user'), int(revision.group('rev')), comment)
+                    continue
                 page_new_user = re.search(r'La page « (?P<page>.*) » a été renommée « (?P<new>.*) » par (?P<user>.*) :', body)
                 if page_new_user:
                     ninja.push_update(page_new_user.group('page'), page_user.group('user'), page_new_user.group('new'), comment, type=Update.rename)
-                else:
-                    page_user = re.search(r'vous vous êtes abonné aux notifications de changements pour la page "(?P<page>.*)"\.Une pièce jointe (?P<add>(?:vient d\'y être ajouté)|(?:de cette page vient d\'être supprimée)) par (?P<user>.*)\. Quelques détails sur la pièce jointe :', body)
-                    piece = re.search(r'Nom\s*:\s*(?P<name>[^\n]*)\n', body)
-                    if page_user:
-                        ninja.push_update(page_user.group('page'), page_user.group('user'), piece.group('name'), "ajout d'une pièce jointe" if page_user.group('add').startswith('v') else 'suppression de la pièce jointe', type=Update.attachment)
+                    continue
+                page_user = re.search(r'vous vous êtes abonné aux notifications de changements pour la page "(?P<page>.*)"\.Une pièce jointe (?P<add>(?:vient d\'y être ajouté)|(?:de cette page vient d\'être supprimée)) par (?P<user>.*)\. Quelques détails sur la pièce jointe :', body)
+                piece = re.search(r'Nom\s*:\s*(?P<name>[^\n]*)\n', body)
+                if page_user:
+                    ninja.push_update(page_user.group('page'), page_user.group('user'), piece.group('name'), "ajout d'une pièce jointe" if page_user.group('add').startswith('v') else 'suppression de la pièce jointe', type=Update.attachment)
 
 
 if __name__ == '__main__':