From 000ee147117a664a3663ebf040378eb9ff0b727e Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sat, 22 Sep 2018 16:02:39 +0100
Subject: [PATCH 01/16] Make tweets say Twitter instead of tweets, and update
 ducks to add decoys and make minimum/unique a channel setting rather than
 hard setting.

---
 modules/ducks.py  | 105 +++++++++++++++++++++++++++++++++++++++-------
 modules/tweets.py |   1 +
 2 files changed, 91 insertions(+), 15 deletions(-)

diff --git a/modules/ducks.py b/modules/ducks.py
index 86bb2f0c..7fa32910 100644
--- a/modules/ducks.py
+++ b/modules/ducks.py
@@ -23,9 +23,16 @@ class Module(object):
         self.events = events
 
         events.on("received.command.bef").hook(self.befriend,
-            priority=EventManager.PRIORITY_HIGH, help="Befriend a duck!")
+            priority=EventManager.PRIORITY_HIGH,
+            help="Befriend a duck!")
         events.on("received.command.bang").hook(self.shoot,
-            priority=EventManager.PRIORITY_HIGH, help="Shoot a duck! Meanie.")
+            priority=EventManager.PRIORITY_HIGH,
+            help="Shoot a duck! Meanie.")
+        events.on("received.command.decoy").hook(
+            self.duck_decoy,
+            priority=EventManager.PRIORITY_HIGH,
+            help="Lay out a sneaky decoy!")
+
 
         events.on("received.command.friends").hook(self.duck_friends,
             help="See who the friendliest people to ducks are!")
@@ -41,10 +48,20 @@ class Module(object):
             "help": "Should the bot kick if there's no duck?",
              "validate": Utils.bool_or_none})
 
+        exports.add("channelset", {"setting": "ducks-min-unique",
+                                   "help": "Minimum unique users required to "
+                                           "talk before a duck spawns.",
+                                   "validate": Utils.int_or_none})
+
+        exports.add("channelset", {"setting": "ducks-min-messages",
+                                   "help": "Minimum messages between ducks "
+                                           "spawning.",
+                                   "validate": Utils.int_or_none})
+
         events.on("new.channel").hook(self.new_channel)
 
         events.on("received.message.channel").hook(
-            self.channel_message)
+            self.channel_message, EventManager.PRIORITY_LOW)
 
         for server in self.bot.servers.values():
             for channel in server.channels.values():
@@ -53,6 +70,9 @@ class Module(object):
     def new_channel(self, event):
         self.bootstrap(event["channel"])
 
+    def has_command(self, command):
+        return command.lower() in self.events.on("received").on("command").get_children()
+
     def bootstrap(self, channel):
         self.init_game_var(channel)
         # getset
@@ -82,25 +102,39 @@ class Module(object):
 
         channel.games["ducks"] = {'messages': 0, 'duck_spawned': 0,
                                   'unique_users': [],
-                                  'next_duck_time': rand_time}
+                                  'next_duck_time': rand_time,
+                                  'decoy_spawned': 0, 'decoy_requested': 0,
+                                  'next_decoy_time': rand_time}
 
     def start_game(self, channel):
         #   event is immediately the IRCChannel.Channel() event for the current
         #   channel
         self.clear_ducks(channel)
 
+        min_unique = channel.get_setting("ducks-min-unique",
+                                         0)
+        min_messages = channel.get_setting("ducks-min-messages",
+                                           0)
+
+        if min_unique == 0:
+            channel.set_setting("ducks-min-unique", DUCK_MINIMUM_UNIQUE)
+
+        if min_messages == 0:
+            channel.set_setting("ducks-min-messages", DUCK_MINIMUM_MESSAGES)
+
     def generate_next_duck_time(self):
-        rand_time = random.randint(int(time()) + 360, int(time()) + 1200)
+        rand_time = random.randint(int(time()) + 1, int(time()) + 2)
         return rand_time
 
     def is_duck_visible(self, event):
         channel = event["target"]
 
-        visible = bool(channel.games["ducks"]["duck_spawned"])
+        visible = channel.games["ducks"]["duck_spawned"]
         return visible
 
     def should_kick(self, event):
         channel = event["target"]
+
         return channel.get_setting("ducks-kick", False)
 
     def kick_bef(self, event):
@@ -117,29 +151,53 @@ class Module(object):
         channel.send_kick(target,
                           "You tried shooting a non-existent duck. Creepy!")
 
+    def duck_decoy(self, event):
+        channel = event["target"]
+        if self.is_duck_channel(channel) == False:
+            return
+
+        if self.is_duck_visible(event):
+            return
+
+        game = channel.games["ducks"]
+        game["decoy_requested"] = 1
+
+        event.eat()
+
+
     def should_generate_duck(self, event):
         channel = event["channel"]
         game = channel.games["ducks"]
 
-        spawned = game["duck_spawned"]
+        spawned = int(game["decoy_spawned"] or game["duck_spawned"])
+        decoy = bool(game["decoy_requested"])
         unique = len(game["unique_users"])
         messages = game["messages"]
-        next_duck = game["next_duck_time"]
+        next_duck = game["next_decoy_time"] if decoy else game["next_duck_time"]
+
+        min_unique = 1 if decoy else channel.get_setting("ducks-min-unique",
+                                          DUCK_MINIMUM_UNIQUE)
+        min_messages = channel.get_setting("ducks-min-messages", DUCK_MINIMUM_MESSAGES)
+
+        requirement = (unique >= min_unique and messages >= min_messages)
 
         # DUCK_MINIMUM_MESSAGES = 10
         # DUCK_MINIMUM_UNIQUE = 3
 
-        if spawned == 0 and next_duck < time() and unique >= \
-                DUCK_MINIMUM_UNIQUE and messages >= DUCK_MINIMUM_MESSAGES:
-            return True
+        if spawned == 0 and next_duck < time():
+            if requirement:
+                return True
+            else:
+                return False
         else:
             return False
 
     def show_duck(self, event):
         channel = event["channel"]
+        game = channel.games["ducks"]
         duck = ""
 
-        if channel.games["ducks"]["duck_spawned"] == 1:
+        if game["duck_spawned"] == 1 or game["decoy_spawned"] == 1:
             return
 
         duck += DUCK_TAIL
@@ -154,7 +212,15 @@ class Module(object):
             duck += random.choice(DUCK_MESSAGE)
 
         channel.send_message(duck)
-        channel.games["ducks"]["duck_spawned"] = 1
+
+        # Decoys take priority over regular ducks.
+        if game["decoy_requested"] == 1:
+            game["decoy_spawned"] = 1
+            game["decoy_requested"] = 0
+
+            game["next_duck_time"] = self.generate_next_duck_time()
+        else:
+            game["duck_spawned"] = 1
 
     def channel_message(self, event):
         if not event["channel"].get_setting("ducks-enabled", False):
@@ -167,8 +233,8 @@ class Module(object):
         user = event["user"]
         game = channel.games["ducks"]
 
-        if game["duck_spawned"] == 1 or channel.has_user(
-                event["user"]) == False:
+        if game["decoy_spawned"] == 1 or game["duck_spawned"] == 1 or \
+                channel.has_user(event["user"]) == False:
             return
 
         unique = game["unique_users"]
@@ -196,6 +262,8 @@ class Module(object):
         if self.is_duck_visible(event) == False:
             if self.should_kick(event):
                 self.kick_bef(event)
+                self.clear_ducks(channel)
+                event.eat()
             return
 
         channel.games["ducks"][
@@ -214,6 +282,7 @@ class Module(object):
         event["stdout"].write(msg)
 
         self.clear_ducks(channel)
+        event.eat()
 
     def shoot(self, event):
         channel = event["target"]
@@ -227,6 +296,8 @@ class Module(object):
         if self.is_duck_visible(event) == False:
             if self.should_kick(event):
                 self.kick_bang(event)
+                self.clear_ducks(channel)
+                event.eat()
             return
 
         channel.games["ducks"][
@@ -245,6 +316,7 @@ class Module(object):
         event["stdout"].write(msg)
 
         self.clear_ducks(channel)
+        event.eat()
 
     def duck_stats(self, event):
         user = event["user"]
@@ -285,6 +357,7 @@ class Module(object):
                  Utils.bold(tf), Utils.bold(cf), Utils.bold(channel))
 
         event["stdout"].write(Utils.bold(nick) + ": " + msg)
+        event.eat()
 
     def duck_enemies(self, event):
         the_enemy = event["server"].find_all_user_channel_settings("ducks-shot")
@@ -318,6 +391,7 @@ class Module(object):
         sentence += ", ".join(build)
 
         event["stdout"].write(sentence)
+        event.eat()
 
     def duck_friends(self, event):
         friends = event["server"].find_all_user_channel_settings(
@@ -353,3 +427,4 @@ class Module(object):
         sentence += ", ".join(build)
 
         event["stdout"].write(sentence)
+        event.eat()
diff --git a/modules/tweets.py b/modules/tweets.py
index cd32939b..7b8a69fc 100644
--- a/modules/tweets.py
+++ b/modules/tweets.py
@@ -11,6 +11,7 @@ REGEX_TWITTERURL = re.compile(
     "https?://(?:www\.)?twitter.com/[^/]+/status/(\d+)", re.I)
 
 class Module(object):
+    _name = "Twitter"
     def __init__(self, bot, events, exports):
         self.bot = bot
         self.events = events

From febb6d66e3fd29c5d18c4c49133b0aef9257f2e1 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sat, 22 Sep 2018 19:10:52 +0100
Subject: [PATCH 02/16] font reset at the beginning of commands to stop
 wonkiness with highlighting and fix duck decoys.

---
 modules/commands.py |  3 ++-
 modules/ducks.py    | 16 ++++++++++------
 2 files changed, 12 insertions(+), 7 deletions(-)

diff --git a/modules/commands.py b/modules/commands.py
index f0068c57..9df70ffc 100644
--- a/modules/commands.py
+++ b/modules/commands.py
@@ -29,7 +29,8 @@ class Out(object):
                     ].decode("utf8").lstrip())
             else:
                 self._text = ""
-            self.target.send_message(text, prefix="[%s] " % self.prefix())
+            self.target.send_message(text, prefix=Utils.FONT_RESET + "[%s] " %
+                                                         self.prefix())
     def set_prefix(self, prefix):
         self.module_name = prefix
     def has_text(self):
diff --git a/modules/ducks.py b/modules/ducks.py
index 7fa32910..ebef9006 100644
--- a/modules/ducks.py
+++ b/modules/ducks.py
@@ -100,6 +100,7 @@ class Module(object):
     def clear_ducks(self, channel):
         rand_time = self.generate_next_duck_time()
 
+        del channel.games["ducks"]
         channel.games["ducks"] = {'messages': 0, 'duck_spawned': 0,
                                   'unique_users': [],
                                   'next_duck_time': rand_time,
@@ -126,10 +127,11 @@ class Module(object):
         rand_time = random.randint(int(time()) + 1, int(time()) + 2)
         return rand_time
 
-    def is_duck_visible(self, event):
+    def is_duck_visible(self, event, decoy=False):
         channel = event["target"]
 
-        visible = channel.games["ducks"]["duck_spawned"]
+        visible = channel.games["ducks"]["decoy_spawned"] if \
+            decoy else channel.games["ducks"]["duck_spawned"]
         return visible
 
     def should_kick(self, event):
@@ -259,11 +261,12 @@ class Module(object):
         if self.is_duck_channel(channel) == False:
             return
 
-        if self.is_duck_visible(event) == False:
+        if self.is_duck_visible(event, False) == False:
             if self.should_kick(event):
                 self.kick_bef(event)
-                self.clear_ducks(channel)
                 event.eat()
+
+            self.clear_ducks(channel)
             return
 
         channel.games["ducks"][
@@ -293,11 +296,12 @@ class Module(object):
         if self.is_duck_channel(channel) == False:
             return
 
-        if self.is_duck_visible(event) == False:
+        if self.is_duck_visible(event, False) == False:
             if self.should_kick(event):
                 self.kick_bang(event)
-                self.clear_ducks(channel)
                 event.eat()
+
+            self.clear_ducks(channel)
             return
 
         channel.games["ducks"][

From 1a54c11df8210cd87d800ff23f09c07c7ff21d87 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sat, 22 Sep 2018 19:28:57 +0100
Subject: [PATCH 03/16] Fix ducks.py having a superfluous function

---
 modules/ducks.py | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/modules/ducks.py b/modules/ducks.py
index ebef9006..a25c05d0 100644
--- a/modules/ducks.py
+++ b/modules/ducks.py
@@ -70,9 +70,6 @@ class Module(object):
     def new_channel(self, event):
         self.bootstrap(event["channel"])
 
-    def has_command(self, command):
-        return command.lower() in self.events.on("received").on("command").get_children()
-
     def bootstrap(self, channel):
         self.init_game_var(channel)
         # getset

From 56afe77b5230a929c96f6fbc68d5699c214d10e4 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sat, 22 Sep 2018 19:32:54 +0100
Subject: [PATCH 04/16] Fix duck timing and stupid indenting.

---
 modules/ducks.py | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/modules/ducks.py b/modules/ducks.py
index a25c05d0..33d8e5e6 100644
--- a/modules/ducks.py
+++ b/modules/ducks.py
@@ -109,10 +109,8 @@ class Module(object):
         #   channel
         self.clear_ducks(channel)
 
-        min_unique = channel.get_setting("ducks-min-unique",
-                                         0)
-        min_messages = channel.get_setting("ducks-min-messages",
-                                           0)
+        min_unique = channel.get_setting("ducks-min-unique", 0)
+        min_messages = channel.get_setting("ducks-min-messages", 0)
 
         if min_unique == 0:
             channel.set_setting("ducks-min-unique", DUCK_MINIMUM_UNIQUE)
@@ -121,7 +119,7 @@ class Module(object):
             channel.set_setting("ducks-min-messages", DUCK_MINIMUM_MESSAGES)
 
     def generate_next_duck_time(self):
-        rand_time = random.randint(int(time()) + 1, int(time()) + 2)
+        rand_time = random.randint(int(time()) + 360, int(time()) + 1200)
         return rand_time
 
     def is_duck_visible(self, event, decoy=False):

From 9220f84c277df19f72195371fabd140883ad5708 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sat, 22 Sep 2018 20:08:07 +0100
Subject: [PATCH 05/16] clear channel.games properly.

---
 modules/ducks.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/modules/ducks.py b/modules/ducks.py
index 33d8e5e6..c2ae696c 100644
--- a/modules/ducks.py
+++ b/modules/ducks.py
@@ -97,7 +97,9 @@ class Module(object):
     def clear_ducks(self, channel):
         rand_time = self.generate_next_duck_time()
 
-        del channel.games["ducks"]
+        if hasattr(channel.games, "ducks"):
+            del channel.games["ducks"]
+
         channel.games["ducks"] = {'messages': 0, 'duck_spawned': 0,
                                   'unique_users': [],
                                   'next_duck_time': rand_time,

From e5283ed2e20e95df58c7a162fe76f88ebf3694e1 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 00:55:36 +0100
Subject: [PATCH 06/16] Update last.fm to include a youtube link. Also change
 the module name to last.fm

---
 modules/lastfm.py  | 15 +++++++++++++--
 modules/youtube.py | 18 ++++++++++++++++++
 2 files changed, 31 insertions(+), 2 deletions(-)

diff --git a/modules/lastfm.py b/modules/lastfm.py
index e448d9ea..3675bbd5 100644
--- a/modules/lastfm.py
+++ b/modules/lastfm.py
@@ -5,8 +5,10 @@ import Utils
 URL_SCROBBLER = "http://ws.audioscrobbler.com/2.0/"
 
 class Module(object):
+    _name = "last.fm"
     def __init__(self, bot, events, exports):
         self.bot = bot
+        self.events = events
 
         exports.add("set", {"setting": "lastfm",
             "help": "Set username on last.fm"})
@@ -34,6 +36,14 @@ class Module(object):
                 track_name = now_playing["name"]
                 artist = now_playing["artist"]["#text"]
 
+                ytquery = " - ".join([artist, track_name])
+
+                short_url = self.events.on(
+                    "get.youtubefromlastfm").call_for_result(
+                    query=ytquery)
+
+                short_url = " -- " + short_url if short_url else ""
+
                 info_page = Utils.get_url(URL_SCROBBLER, get_params={
                     "method": "track.getInfo", "artist": artist,
                     "track": track_name, "autocorrect": "1",
@@ -55,8 +65,9 @@ class Module(object):
                         "s" if play_count > 1 else "")
 
                 event["stdout"].write(
-                    "%s is now playing: %s - %s%s%s" % (
-                    shown_username, artist, track_name, play_count, tags))
+                    "%s is now playing: %s - %s%s%s%s" % (
+                    shown_username, artist, track_name, play_count, tags,
+                    short_url))
             else:
                 event["stderr"].write(
                     "The user '%s' has never scrobbled before" % (
diff --git a/modules/youtube.py b/modules/youtube.py
index 3d075f64..20dc072b 100644
--- a/modules/youtube.py
+++ b/modules/youtube.py
@@ -24,6 +24,8 @@ class Module(object):
             help="Find a video on youtube", usage="[query]")
         events.on("received.message.channel").hook(self.channel_message)
 
+        events.on("get.youtubefromlastfm").hook(self.get_yt_from_lastfm)
+
         exports.add("channelset", {"setting": "auto-youtube",
             "help": "Disable/Enable automatically getting info from "
             "youtube URLs", "validate": Utils.bool_or_none})
@@ -63,6 +65,22 @@ class Module(object):
                 video_title, video_duration, video_uploader, "{:,}".format(
                 int(video_views)), video_opinions, URL_YOUTUBESHORT % video_id)
 
+    def get_yt_from_lastfm(self, event):
+        search = event["query"]
+        video_id = ""
+
+        search_page = Utils.get_url(URL_YOUTUBESEARCH,
+            get_params={"q": search, "part": "snippet",
+            "maxResults": "1", "type": "video",
+            "key": self.bot.config["google-api-key"]},
+             json=True)
+
+        if search_page:
+            if search_page["pageInfo"]["totalResults"] > 0:
+                video_id = search_page["items"][0]["id"]["videoId"]
+                return "https://youtu.be/%s" % video_id
+
+
     def yt(self, event):
         video_id = None
         search = None

From d335deed40423c9a64de6bcffde301086e798822 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 01:39:38 +0100
Subject: [PATCH 07/16] make last.fm say "is now playing", if they're currently
 playing.

---
 modules/lastfm.py | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/modules/lastfm.py b/modules/lastfm.py
index 3675bbd5..35c903ba 100644
--- a/modules/lastfm.py
+++ b/modules/lastfm.py
@@ -1,6 +1,7 @@
 #--require-config lastfm-api-key
 
 import Utils
+import time
 
 URL_SCROBBLER = "http://ws.audioscrobbler.com/2.0/"
 
@@ -36,6 +37,16 @@ class Module(object):
                 track_name = now_playing["name"]
                 artist = now_playing["artist"]["#text"]
 
+                if '@attr' in now_playing:
+                    np = True
+                else:
+                    played = int(now_playing["date"]["uts"])
+                    timenow = int(time.time())
+                    np = bool(timenow - played > 240)
+
+                time_language = "last listened to" if np == False \
+                                                    else "is now playing"
+
                 ytquery = " - ".join([artist, track_name])
 
                 short_url = self.events.on(
@@ -65,8 +76,10 @@ class Module(object):
                         "s" if play_count > 1 else "")
 
                 event["stdout"].write(
-                    "%s is now playing: %s - %s%s%s%s" % (
-                    shown_username, artist, track_name, play_count, tags,
+                    "%s %s: %s - %s%s%s%s" % (
+                    shown_username, time_language, artist, track_name,
+                    play_count,
+                    tags,
                     short_url))
             else:
                 event["stderr"].write(

From bb15f8b8e2383a84b1e28bb6ee85f01687df819e Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 02:21:43 +0100
Subject: [PATCH 08/16] make last.fm say "is now playing", if they're currently
 playing.

---
 modules/lastfm.py | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/modules/lastfm.py b/modules/lastfm.py
index 35c903ba..0d6873a2 100644
--- a/modules/lastfm.py
+++ b/modules/lastfm.py
@@ -1,7 +1,7 @@
 #--require-config lastfm-api-key
 
 import Utils
-import time
+from datetime import datetime, timezone
 
 URL_SCROBBLER = "http://ws.audioscrobbler.com/2.0/"
 
@@ -41,11 +41,11 @@ class Module(object):
                     np = True
                 else:
                     played = int(now_playing["date"]["uts"])
-                    timenow = int(time.time())
-                    np = bool(timenow - played > 240)
+                    dts = int(datetime.now(tz=timezone.utc).timestamp())
+                    np = bool((dts - played) < 120)
 
-                time_language = "last listened to" if np == False \
-                                                    else "is now playing"
+                time_language = "is listening to" if np else "last " \
+                                                               + "listened to"
 
                 ytquery = " - ".join([artist, track_name])
 

From 0cb2c43ee317569aa7afe36acd03d059c3714b5e Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 07:04:50 +0100
Subject: [PATCH 09/16] Add setcoins command for admins, for people that abuse
 the coin game, or rewards and such.

---
 modules/coins.py | 23 +++++++++++++++++++++--
 1 file changed, 21 insertions(+), 2 deletions(-)

diff --git a/modules/coins.py b/modules/coins.py
index ce1bae57..d37832ca 100644
--- a/modules/coins.py
+++ b/modules/coins.py
@@ -26,6 +26,7 @@ THIRD_COLUMN = list(range(1, 37))[2::3]
 REGEX_STREET = re.compile("street([1-9]|1[0-2])$")
 
 class Module(object):
+
     def __init__(self, bot, events, exports):
         self.bot = bot
         events.on("received.command.coins").hook(self.coins,
@@ -57,6 +58,12 @@ class Module(object):
             self.send, min_args=2, help="Send coins to a user",
             usage="<nickname> <amount>", authenticated=True)
 
+        events.on("received.command.setcoins").hook(
+            self.set_coins, min_args=2,
+            help="Set a users coins. Do not abuse.",
+            usage="<nickname> <amount>", permission="setcoins",
+            authenticated=True)
+
         now = datetime.datetime.now()
         until_next_hour = 60-now.second
         until_next_hour += ((60-(now.minute+1))*60)
@@ -84,6 +91,18 @@ class Module(object):
             target.del_setting("coins")
             event["stdout"].write("Reset coins for %s" % target.nickname)
 
+    def set_coins(self, event):
+        target = event["server"].get_user(event["args_split"][0])
+        coins = event["args_split"][1]
+
+        if not coins.isdigit():
+            event["stderr"].write("Cannot set coins for %s, coins must be a " \
+                            + "whole number." % target.nickname)
+            return
+
+        target.set_setting("coins", "%s.0" % coins)
+        event["stdout"].write("Set %s's coins to %s" % (target.nickname, coins))
+
     def give_coins(self, event):
         target = event["server"].get_user(event["args_split"][0])
         coins = event["args_split"][1]
@@ -107,8 +126,8 @@ class Module(object):
 
         top_10 = sorted(all_coins.keys())
         top_10 = sorted(top_10, key=all_coins.get, reverse=True)[:10]
-        top_10 = ", ".join("%s (%s)" % (Utils.prevent_highlight(event[
-            "server"].get_user(nickname).nickname), "{0:.2f}".format(
+        top_10 = ", ".join("%s (%s)" % (Utils.bold(Utils.prevent_highlight(
+            event["server"].get_user(nickname).nickname)), "{0:.2f}".format(
             all_coins[nickname])) for nickname in top_10)
         event["stdout"].write("Richest users: %s" % top_10)
 

From fdcf3e45c70daffb314e3533fafb66aedbba6b1a Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 09:52:53 +0100
Subject: [PATCH 10/16] Add .randomword command, and change the api to use
 https

---
 modules/define.py | 61 +++++++++++++++++++++++++++++++++++++++++++----
 1 file changed, 56 insertions(+), 5 deletions(-)

diff --git a/modules/define.py b/modules/define.py
index 3451aa5f..ae1aa7b2 100644
--- a/modules/define.py
+++ b/modules/define.py
@@ -1,24 +1,42 @@
 #--require-config wordnik-api-key
 
 import Utils
+import time
 
-URL_WORDNIK = "http://api.wordnik.com:80/v4/word.json/%s/definitions"
+URL_WORDNIK = "https://api.wordnik.com/v4/word.json/%s/definitions"
+URL_WORDNIK_RANDOM = "https://api.wordnik.com/v4/words.json/randomWord"
+
+RANDOM_DELAY_SECONDS = 3
 
 class Module(object):
     def __init__(self, bot, events, exports):
         self.bot = bot
+        self.last_called = 0
+        self.events = events
+
         events.on("received.command.define").hook(self.define,
             help="Define a provided term", usage="<phrase>")
 
+        events.on("received.command.randomword").hook(self.random_word,
+            help="Generate a random word!")
+
+    def get_definition(self, event):
+        word = event["args"] if "args" in event else event
+
+
+        page = Utils.get_url(URL_WORDNIK % word, get_params={
+            "useCanonical": "true", "limit": 1,
+            "sourceDictionaries": "wiktionary", "api_key": self.bot.config[
+                "wordnik-api-key"]}, json=True)
+
+        return page
+
     def define(self, event):
         if event["args"]:
             word = event["args"]
         else:
             word = event["buffer"].get(from_self=False)
-        page = Utils.get_url(URL_WORDNIK % event["args"], get_params={
-            "useCanonical": "true", "limit": 1,
-            "sourceDictionaries": "wiktionary", "api_key": self.bot.config[
-                "wordnik-api-key"]}, json=True)
+        page = self.get_definition(event)
         if page:
             if len(page):
                 event["stdout"].write("%s: %s" % (page[0]["word"],
@@ -27,3 +45,36 @@ class Module(object):
                 event["stderr"].write("No definitions found")
         else:
             event["stderr"].write("Failed to load results")
+
+    def random_word(self, event):
+        if not self.last_called or (time.time()-self.last_called >=
+                                    RANDOM_DELAY_SECONDS):
+
+            self.last_called = time.time()
+
+            page = Utils.get_url(URL_WORDNIK_RANDOM, get_params={
+                "api_key":self.bot.config["wordnik-api-key"],
+                "min_dictionary_count":1},json=True)
+            if page:
+                if len(page):
+                    definition = self.get_definition(page["word"])
+
+                    if len(definition):
+                        definition = definition[0]
+                    else:
+                        self.events.on("send.stderr").call(module_name="Random",
+                                       target=event["target"],
+                                        message="Try again in a couple of seconds")
+                        return
+
+                    event["stdout"].set_prefix("Random")
+                    event["stdout"].write("Random Word: %s - Definition: %s" % (
+                        page["word"], definition["text"]))
+                else:
+                    event["stderr"].write("Something has gone terribly wrong")
+            else:
+                event["stderr"].write("Failed to load results")
+        else:
+            self.events.on("send.stderr").call(module_name="Random",
+                          target=event["target"],
+                          message="Try again in a couple of seconds")
\ No newline at end of file

From e8bdcf8f19b50555b18bb4993c0c5fe25e253952 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 10:16:43 +0100
Subject: [PATCH 11/16] Remove intrusive styling.

---
 modules/coins.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/modules/coins.py b/modules/coins.py
index d37832ca..d3fdf065 100644
--- a/modules/coins.py
+++ b/modules/coins.py
@@ -126,8 +126,8 @@ class Module(object):
 
         top_10 = sorted(all_coins.keys())
         top_10 = sorted(top_10, key=all_coins.get, reverse=True)[:10]
-        top_10 = ", ".join("%s (%s)" % (Utils.bold(Utils.prevent_highlight(
-            event["server"].get_user(nickname).nickname)), "{0:.2f}".format(
+        top_10 = ", ".join("%s (%s)" % (Utils.prevent_highlight(
+            event["server"].get_user(nickname).nickname), "{0:.2f}".format(
             all_coins[nickname])) for nickname in top_10)
         event["stdout"].write("Richest users: %s" % top_10)
 

From a37dfe530162d316f45b58aab4e4a1dcf160a152 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 10:20:20 +0100
Subject: [PATCH 12/16] Remove superfluous function.

---
 modules/coins.py | 17 -----------------
 1 file changed, 17 deletions(-)

diff --git a/modules/coins.py b/modules/coins.py
index d3fdf065..f1e3f826 100644
--- a/modules/coins.py
+++ b/modules/coins.py
@@ -58,11 +58,6 @@ class Module(object):
             self.send, min_args=2, help="Send coins to a user",
             usage="<nickname> <amount>", authenticated=True)
 
-        events.on("received.command.setcoins").hook(
-            self.set_coins, min_args=2,
-            help="Set a users coins. Do not abuse.",
-            usage="<nickname> <amount>", permission="setcoins",
-            authenticated=True)
 
         now = datetime.datetime.now()
         until_next_hour = 60-now.second
@@ -91,18 +86,6 @@ class Module(object):
             target.del_setting("coins")
             event["stdout"].write("Reset coins for %s" % target.nickname)
 
-    def set_coins(self, event):
-        target = event["server"].get_user(event["args_split"][0])
-        coins = event["args_split"][1]
-
-        if not coins.isdigit():
-            event["stderr"].write("Cannot set coins for %s, coins must be a " \
-                            + "whole number." % target.nickname)
-            return
-
-        target.set_setting("coins", "%s.0" % coins)
-        event["stdout"].write("Set %s's coins to %s" % (target.nickname, coins))
-
     def give_coins(self, event):
         target = event["server"].get_user(event["args_split"][0])
         coins = event["args_split"][1]

From 8b8c59ee213a4c006f90767fe379030d4a1e1524 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 10:23:12 +0100
Subject: [PATCH 13/16] Rename the functions to be more generic.

---
 modules/lastfm.py  | 2 +-
 modules/youtube.py | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/modules/lastfm.py b/modules/lastfm.py
index 0d6873a2..68b6249b 100644
--- a/modules/lastfm.py
+++ b/modules/lastfm.py
@@ -50,7 +50,7 @@ class Module(object):
                 ytquery = " - ".join([artist, track_name])
 
                 short_url = self.events.on(
-                    "get.youtubefromlastfm").call_for_result(
+                    "get.searchyoutube").call_for_result(
                     query=ytquery)
 
                 short_url = " -- " + short_url if short_url else ""
diff --git a/modules/youtube.py b/modules/youtube.py
index 20dc072b..c70e9ef8 100644
--- a/modules/youtube.py
+++ b/modules/youtube.py
@@ -24,7 +24,7 @@ class Module(object):
             help="Find a video on youtube", usage="[query]")
         events.on("received.message.channel").hook(self.channel_message)
 
-        events.on("get.youtubefromlastfm").hook(self.get_yt_from_lastfm)
+        events.on("get.searchyoutube").hook(self.search_video)
 
         exports.add("channelset", {"setting": "auto-youtube",
             "help": "Disable/Enable automatically getting info from "
@@ -65,7 +65,7 @@ class Module(object):
                 video_title, video_duration, video_uploader, "{:,}".format(
                 int(video_views)), video_opinions, URL_YOUTUBESHORT % video_id)
 
-    def get_yt_from_lastfm(self, event):
+    def search_video(self, event):
         search = event["query"]
         video_id = ""
 

From 1d3f05a8fe0ca9151909e70094a0cfbf75c0d412 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 11:01:24 +0100
Subject: [PATCH 14/16] Add .reloadallmodules, and fix permissions.

---
 modules/modules.py     | 37 +++++++++++++++++++++++++++++++++----
 modules/permissions.py |  4 ++--
 2 files changed, 35 insertions(+), 6 deletions(-)

diff --git a/modules/modules.py b/modules/modules.py
index 0f18b10f..109baeff 100644
--- a/modules/modules.py
+++ b/modules/modules.py
@@ -3,16 +3,23 @@
 class Module(object):
     def __init__(self, bot, events, exports):
         self.bot = bot
+        self.module_name = False
+        self.silent = False
+
         events.on("received.command.loadmodule").hook(self.load,
             min_args=1, permission="load-module", help="Load a module",
             usage="<module-name>")
         events.on("received.command.unloadmodule").hook(self.unload,
             min_args=1, permission="unload-module", help="Unload a module",
             usage="<module-name>")
+
         events.on("received.command.reloadmodule").hook(self.reload,
-            min_args=1, permission="reload-module", help="Reload a module",
+            min_args=1, permission="reload-module", help="Reoad a module",
             usage="<module-name>")
 
+        events.on("received.command.reloadallmodules").hook(self.reload_all,
+            permission="reload-module", help="Reload all modules")
+
         events.on("received.command.enablemodule").hook(self.enable,
             min_args=1, permission="enable-module", help="Enable a module",
             usage="<module-name>")
@@ -37,13 +44,35 @@ class Module(object):
         event["stdout"].write("Unloaded '%s'" % name)
 
     def reload(self, event):
-        name = event["args_split"][0].lower()
+        name = self.module_name if self.module_name != False else event[
+            "args_split"][0].lower()
         if not name in self.bot.modules.modules:
-            event["stderr"].write("Module '%s' isn't loaded" % name)
+            if self.silent == False:
+                event["stderr"].write("Module '%s' isn't loaded" % name)
             return
         self.bot.modules.unload_module(name)
         self.bot.modules.load_module(name)
-        event["stdout"].write("Reloaded '%s'" % name)
+
+        if self.silent == False:
+            event["stdout"].write("Reloaded '%s'" % name)
+
+    def reload_all(self, event):
+        modules_reloaded = []
+        self.silent = True
+
+        for name, value in self.bot.modules.modules.items():
+            if name in modules_reloaded:
+                pass
+
+            self.module_name = name
+            self.reload(event)
+            modules_reloaded.append(name)
+
+        event["stdout"].write("Reloaded modules: %s" % \
+                              " ".join(modules_reloaded))
+
+        self.silent = False
+        self.module_name = False
 
     def enable(self, event):
         name = event["args_split"][0].lower()
diff --git a/modules/permissions.py b/modules/permissions.py
index ff04f7f9..9a7c266e 100644
--- a/modules/permissions.py
+++ b/modules/permissions.py
@@ -177,7 +177,7 @@ class Module(object):
         target, registered, permissions = self._get_user_details(
             event["server"], event["args_split"][0])
 
-        if not registered:
+        if target.identified_account == None:
             event["stderr"].write("%s isn't registered" % target.nickname)
             return
 
@@ -194,7 +194,7 @@ class Module(object):
         target, registered, permissions = self._get_user_details(
             event["server"], event["args_split"][0])
 
-        if not registered:
+        if target.identified_account == None:
             event["stderr"].write("%s isn't registered" % target.nickname)
             return
 

From 36f0672f2b9fb4feb84ec7d25b52f39f37a334f8 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 11:06:15 +0100
Subject: [PATCH 15/16] Fix permission bug

---
 modules/permissions.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/permissions.py b/modules/permissions.py
index 9a7c266e..fe6a8977 100644
--- a/modules/permissions.py
+++ b/modules/permissions.py
@@ -199,7 +199,7 @@ class Module(object):
             return
 
         if not permission in permissions:
-            event["stderr"].write("%s already has permission '%s'" % (
+            event["stderr"].write("%s doesn't have permission '%s'" % (
                 target.nickname, permission))
         else:
             permissions.remove(permission)

From c3c3eba476b49944053e88b67fd2a622521d8d89 Mon Sep 17 00:00:00 2001
From: dngfx <294904+dngfx@users.noreply.github.com>
Date: Sun, 23 Sep 2018 11:09:46 +0100
Subject: [PATCH 16/16] Fix permission bug

---
 modules/permissions.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/modules/permissions.py b/modules/permissions.py
index fe6a8977..27c40485 100644
--- a/modules/permissions.py
+++ b/modules/permissions.py
@@ -198,7 +198,7 @@ class Module(object):
             event["stderr"].write("%s isn't registered" % target.nickname)
             return
 
-        if not permission in permissions:
+        if permission not in permissions:
             event["stderr"].write("%s doesn't have permission '%s'" % (
                 target.nickname, permission))
         else: