config.json.example

@@ -152,7 +152,11 @@
       "not_fresh_reply": "It's not from today but here it is: {url}"
     },
     "plus": {
-      "commands": ["plus"]
+      "commands": ["plus"],
+      "aliases": {
+        "plus": "more"
+      },
+      "shortcut": "+"
     },
     "randomchoice": {
       "commands": ["choose"],

edmond/plugins/journee_mondiale.py

@@ -1,5 +1,4 @@
 import datetime
-from typing import cast
 
 # BS is optional and only for scrapping journee-mondiale.com, thus why we do not
 # mark the dependencies flag here.
@@ -9,7 +8,6 @@ except ImportError:
     BeautifulSoup = None
 
 from edmond.plugin import Plugin
-from edmond.plugins.plus import PlusPlugin
 from edmond.utils import http_get
 
 
@@ -33,7 +31,6 @@ class JourneeMondialePlugin(Plugin):
     """
 
     REQUIRED_CONFIGS = ["commands", "dates", "no_entry_reply"]
-    JMCOM_URL = "https://www.journee-mondiale.com"
 
     def __init__(self, bot):
         super().__init__(bot)
@@ -48,7 +45,7 @@ class JourneeMondialePlugin(Plugin):
 
         jmcom_reply = ""
         if self.config.get("jmcom", False) is True:
-            jmcom_reply = self.get_jmcom_days(event.target)
+            jmcom_reply = self.get_jmcom_days()
             if jmcom_reply:
                 self.bot.say(event.target, jmcom_reply)
 
@@ -63,16 +60,15 @@ class JourneeMondialePlugin(Plugin):
         today_obs = map(
             lambda line: line.split(maxsplit=1)[1],
             filter(
-                lambda line: line.startswith(date_tag),
+                lambda line: line.startswith(date_tag), self.config["dates"]
-                self.config["dates"],
             ),
         )
         days = ", ".join(today_obs)
-        return days
+        return reply
 
-    def get_jmcom_days(self, target) -> str:
+    def get_jmcom_days(self) -> str:
         """Get international days from journee-mondiale.com."""
-        response = http_get(JourneeMondialePlugin.JMCOM_URL)
+        response = http_get(self.config["url"])
         if not response:
             return ""
 
@@ -81,22 +77,13 @@ class JourneeMondialePlugin(Plugin):
         try:
             items = soup.find("div", id="journeesDuJour").find_all("article")
             for item in items:
-                entries.append(
+                entries.append({
-                    {
+                    "url": item.find("a").href,
-                        "url": item.find("a")["href"],
                     "title": item.find("h2").string,
-                    }
+                })
-                )
         except (ValueError, KeyError):
             return ""
 
-        if plus_plugin := cast(PlusPlugin, self.bot.get_plugin("plus")):
+        # TODO add plus plugin support
 
-            def handler(plus_event):
+        return ", ".join(map(lambda i: i["title"], entries))
-                urls = map(lambda i: i["url"], entries)
-                self.bot.say(plus_event.target, " — ".join(urls))
-
-            plus_plugin.add_handler(target, handler)
-
-        days = ", ".join(map(lambda i: i["title"], entries))
-        return days

edmond/plugins/plus.py

@@ -20,12 +20,16 @@ class PlusPlugin(Plugin):
     def __init__(self, bot):
         super().__init__(bot)
         self.handlers = {}
+        self.shortcut = self.config.get("shortcut")
 
     def on_pubmsg(self, event):
-        if not self.should_handle_command(event.arguments[0], no_content=True):
+        if (
-            return False
+            (self.shortcut and event.arguments[0].strip() == self.shortcut)
+            or self.should_handle_command(event.arguments[0], no_content=True)
+        ):
             self.process_handler(event)
             return True
+        return False
 
     def process_handler(self, event):
         if handler := self.handlers.pop(event.target, None):