diff options
| author |  jesopo | 2018-10-16 15:09:58 +0100 |
|---|---|---|
| committer | jesopo | 2018-10-16 15:09:58 +0100 |
| commit | 773d11f6cbaa8da91185547b0ab67f3706d735c2 (patch) | |
| tree | c46eb6709985ef7cdfb5bfe9b035fdbdedc4f91f /modules/permissions.py | |
| parent | Only log exceptions when they're not unsafe (diff) | |
| signature | ||
Change all instances of stdout.write+return to `raise utils.EventError` in
modules
Diffstat (limited to 'modules/permissions.py')
| -rw-r--r-- | modules/permissions.py | 15 |
1 files changed, 5 insertions, 10 deletions
diff --git a/modules/permissions.py b/modules/permissions.py index 6301ced6..64bba006 100644 --- a/modules/permissions.py +++ b/modules/permissions.py @@ -47,14 +47,12 @@ class Module(ModuleManager.BaseModule): identity_mechanism = event["server"].get_setting("identity-mechanism", "internal") if not identity_mechanism == "internal": - event["stderr"].write("The 'identify' command isn't available " + raise utils.EventError("The 'identify' command isn't available " "on this network") - return if not event["user"].channels: - event["stderr"].write("You must share at least one channel " + raise utils.EventError("You must share at least one channel " "with me before you can identify") - return if not event["user"].identified_account_override: if len(event["args_split"]) > 1: @@ -91,9 +89,8 @@ class Module(ModuleManager.BaseModule): identity_mechanism = event["server"].get_setting("identity-mechanism", "internal") if not identity_mechanism == "internal": - event["stderr"].write("The 'identify' command isn't available " + raise utils.EventError("The 'identify' command isn't available " "on this network") - return hash, salt = self._get_hash(event["server"], event["user"].nickname) if not hash and not salt: @@ -203,8 +200,7 @@ class Module(ModuleManager.BaseModule): event["server"], event["args_split"][0]) if target.get_identified_account() == None: - event["stderr"].write("%s isn't registered" % target.nickname) - return + raise utils.EventError("%s isn't registered" % target.nickname) if permission in permissions: event["stderr"].write("%s already has permission '%s'" % ( @@ -226,8 +222,7 @@ class Module(ModuleManager.BaseModule): event["server"], event["args_split"][0]) if target.identified_account == None: - event["stderr"].write("%s isn't registered" % target.nickname) - return + raise utils.EventError("%s isn't registered" % target.nickname) if permission not in permissions: event["stderr"].write("%s doesn't have permission '%s'" % ( |