From 967ed65e00b790df25c1e6925f211779bd0098d9 Mon Sep 17 00:00:00 2001
From: jesopo
Date: Mon, 24 Sep 2018 13:18:15 +0100
Subject: Actually implement !reloadallmodules in modules.py

---
 modules/modules.py | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

(limited to 'modules')

diff --git a/modules/modules.py b/modules/modules.py
index 154a1765..0d4844e9 100644
--- a/modules/modules.py
+++ b/modules/modules.py
@@ -57,18 +57,23 @@ class Module(object):
     def reload_all(self, event):
         reloaded = []
         failed = []
-        for name in self.bot.modules.modules.keys():
+        for name in list(self.bot.modules.modules.keys()):
             try:
                 self._reload(name)
-            except ModuleManager.ModuleNotFoundException:
-                failed.append
-            if not self._reload(event):
+            except ModuleWarning:
+                continue
+            except:
                 failed.append(name)
-            else:
-                reloaded.append(name)
-
-        event["stdout"].write("Reloaded modules: %s" % \
-                              " ".join(modules_reloaded))
+                continue
+            reloaded.append(name)
+
+        if reloaded and failed:
+            event["stdout"].write("Reloaded %d modules, %d failed" % (
+                len(reloaded), len(failed)))
+        elif failed:
+            event["stdout"].write("Failed to reload all modules")
+        else:
+            event["stdout"].write("Reloaded %d modules" % len(reloaded))
 
     def enable(self, event):
         name = event["args_split"][0].lower()
-- 
cgit v1.3.1-10-gc9f91