aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGravatar jesopo2018-10-16 13:43:45 +0100
committerGravatar jesopo2018-10-16 13:43:45 +0100
commit1eab3178b33c6397a7afb2a1fc684c43a8f039fb (patch)
tree8ecefefbf6936a914c967d26f162c5176a26085e
parentmodules/coins.py._take should have a `server` param (diff)
signature
Remove unneeded complexity in modules/coins.py.reset_coins
-rw-r--r--modules/coins.py10
1 files changed, 3 insertions, 7 deletions
diff --git a/modules/coins.py b/modules/coins.py
index 8eae0817..5fce3380 100644
--- a/modules/coins.py
+++ b/modules/coins.py
@@ -144,13 +144,9 @@ class Module(ModuleManager.BaseModule):
"""
target = event["server"].get_user(event["args_split"][0])
coins = self._get_user_coins(target)
- if coins == DECIMAL_ZERO:
- event["stderr"].write("%s already has %s coins" % (
- target.nickname, str(DECIMAL_ZERO)))
- else:
- self._give_to_pool(event["server"], coins)
- self._reset_user_coins(target)
- event["stdout"].write("Reset coins for %s" % target.nickname)
+ self._take(event["server"], target, coins)
+ self._reset_user_coins(target)
+ event["stdout"].write("Reset coins for %s" % target.nickname)
@utils.hook("received.command.givecoins", min_args=1)
def give_coins(self, event):