Refactor and improvements

marvin_actions.py

@@ -31,7 +31,6 @@ def getAllActions():
         marvinSource,
         marvinBudord,
         marvinQuote,
-        marvinStats,
         marvinWeather,
         marvinSun,
         marvinSayHi,
@@ -123,7 +122,7 @@ def marvinExplainShell(row):
     if not query:
         return None
     cmd = " ".join(query)
-    url = "http://explainshell.com/explain?cmd="
+    url = "https://explainshell.com/explain?cmd="
     url += quote_plus(cmd, "/:")
     msg = getString("explainShell")
     return msg.format(url)
@@ -175,12 +174,12 @@ def videoOfToday():
     """
     Check what day it is and provide a url to a suitable video together with a greeting.
     """
-    dayNum = datetime.date.weekday(datetime.date.today()) + 1
-    msg = getString("weekdays", str(dayNum))
-    video = getString("video-of-today", str(dayNum))
+    weekday = datetime.date.today().strftime("%A")
+    day = getString("video-of-today", weekday)
+    msg = day.get("message")
 
-    if video:
-        msg += " En passande video är " + video
+    if day:
+        msg += " En passande video är " + day.get("url")
     else:
         msg += " Jag har ännu ingen passande video för denna dagen."
 
@@ -221,17 +220,6 @@ def marvinHelp(row):
     return msg
 
 
-def marvinStats(row):
-    """
-    Provide a link to the stats.
-    """
-    msg = None
-    if any(r in row for r in ["stats", "statistik", "ircstats"]):
-        msg = getString("ircstats")
-
-    return msg
-
-
 def marvinSayHi(row):
     """
     Say hi with a nice message.
@@ -255,21 +243,23 @@ def marvinLunch(row):
     Help decide where to eat.
     """
     lunchOptions = {
-        'stan centrum karlskrona kna': 'lunch-karlskrona',
-        'ängelholm angelholm engelholm': 'lunch-angelholm',
-        'hässleholm hassleholm': 'lunch-hassleholm',
-        'malmö malmo malmoe': 'lunch-malmo',
-        'göteborg goteborg gbg': 'lunch-goteborg'
+        'stan centrum karlskrona kna': 'karlskrona',
+        'ängelholm angelholm engelholm': 'angelholm',
+        'hässleholm hassleholm': 'hassleholm',
+        'malmö malmo malmoe': 'malmo',
+        'göteborg goteborg gbg': 'goteborg'
     }
 
-    if any(r in row for r in ["lunch", "mat", "äta", "luncha"]):
-        lunchStr = getString('lunch-message')
+    data = getString("lunch")
 
+    if any(r in row for r in ["lunch", "mat", "äta", "luncha"]):
+        places = data.get("location").get("bth")
         for keys, value in lunchOptions.items():
             if any(r in row for r in keys.split(" ")):
-                return lunchStr.format(getString(value))
+                places = data.get("location").get(value)
 
-        return lunchStr.format(getString('lunch-bth'))
+        lunchStr = getString("lunch", "message")
+        return lunchStr.format(places[random.randint(0, len(places) - 1)])
 
     return None
 
@@ -458,7 +448,7 @@ def marvinNameday(row):
     if any(r in row for r in ["nameday", "namnsdag"]):
         try:
             now = datetime.datetime.now()
-            raw_url = "http://api.dryg.net/dagar/v2.1/{year}/{month}/{day}"
+            raw_url = "https://api.dryg.net/dagar/v2.1/{year}/{month}/{day}"
             url = raw_url.format(year=now.year, month=now.month, day=now.day)
             r = requests.get(url, timeout=5)
             nameday_data = r.json()

marvin_strings.json

@@ -1,11 +1,9 @@
 {
-    "source": "I PHP-kurserna kan du länka till source.php. Annars delar du koden som en gist (https://gist.github.com) eller i CodeShare, (http://codeshare.io).",
+    "source": "I PHP-kurserna kan du länka till source.php. Annars delar du koden som en gist (https://gist.github.com) eller i CodeShare, (https://codeshare.io).",
 
-    "whois": "Jag är en tjänstvillig själ som gillar webbprogrammering. Jag bor på GitHub https://github.com/mosbth/irc2phpbb och du kan diskutera mig i forumet http://dbwebb.se/t/20",
+    "whois": "Jag är en tjänstvillig själ som gillar webbprogrammering. Jag bor på GitHub https://github.com/mosbth/irc2phpbb och du kan diskutera mig i forumet https://dbwebb.se/t/20",
 
-    "menu": "[ vem är | le | lunch [var] | citat | budord 1 - 5 | source | väder | solen | dagens video | nöje/paus/strip/comic [slump] | grill | birthday/födelsedag | nameday/namnsdag | stats | google/googla | explain/förklara | uptime | stream | princip | skämt/joke | hjälp ]",
-
-    "ircstats": "Statistik för kanalen finns här: http://dbwebb.se/irssistats/db-o-webb.html",
+    "menu": "[ vem är | le | lunch [var] | citat | budord 1 - 5 | source | väder | solen | dagens video | nöje/paus/strip/comic [slump] | grill | birthday/födelsedag | nameday/namnsdag |  google/googla | explain/förklara | uptime | stream | princip | skämt/joke | hjälp ]",
 
     "google": [
         "Googla {}",
@@ -62,125 +60,133 @@
 
     "budord": {
         "#1": "Ställ din fråga, länka till exempel och källkod. Häng kvar och vänta på svar.",
-        "#2": "Var inte rädd för att fråga och fråga tills du får svar: http://dbwebb.se/f/6249",
-        "#3": "Öva dig ställa smarta frågor: http://dbwebb.se/f/7802",
-        "#4": "When in doubt - gör ett testprogram. http://dbwebb.se/f/13570",
-        "#5": "Hey Luke - use the source! http://catb.org/jargon/html/U/UTSL.html"
-    },
-
-
-    "weekdays": {
-        "1": "Idag är det måndag.",
-        "2": "Idag är det tisdag.",
-        "3": "Idag är det onsdag.",
-        "4": "Idag är det torsdag.",
-        "5": "Idag är det fredag.",
-        "6": "Idag är det lördag.",
-        "7": "Idag är det söndag."
+        "#2": "Var inte rädd för att fråga och fråga tills du får svar: https://dbwebb.se/f/6249",
+        "#3": "Öva dig ställa smarta frågor: https://dbwebb.se/f/7802",
+        "#4": "When in doubt - gör ett testprogram. https://dbwebb.se/f/13570",
+        "#5": "Hey Luke - use the source! https://catb.org/jargon/html/U/UTSL.html"
     },
 
     "video-of-today": {
-        "1": "https://www.youtube.com/watch?v=HbJ6Jsy0JfY  https://www.youtube.com/watch?v=-Kobdb37Cwc eller https://www.youtube.com/watch?v=d-diB65scQU",
-        "2": "https://www.youtube.com/watch?v=sL4znuvCuRg",
-        "3": "https://www.youtube.com/watch?v=F3jnymeJof4",
-        "4": "https://www.youtube.com/watch?v=9Tg_zvDg3Xo",
-        "5": "https://www.youtube.com/watch?v=kfVsfOSbJY0",
-        "6": "https://www.youtube.com/watch?v=GVCzdpagXOQ",
-        "7": "https://www.youtube.com/watch?v=c3BvW56tjB0"
+      "Monday": {
+        "url": "https://www.youtube.com/watch?v=HbJ6Jsy0JfY  https://www.youtube.com/watch?v=-Kobdb37Cwc eller https://www.youtube.com/watch?v=d-diB65scQU",
+        "message": "Idag är det måndag."
+      },
+      "Tuesday": {
+        "url": "https://www.youtube.com/watch?v=sL4znuvCuRg",
+        "message": "Idag är det tisdag."
+      },
+      "Wednesday": {
+        "url": "https://www.youtube.com/watch?v=F3jnymeJof4",
+        "message": "Idag är det onsdag."
+      },
+      "Thursday": {
+        "url": "https://www.youtube.com/watch?v=9Tg_zvDg3Xo",
+        "message": "Idag är det torsdag."
+      },
+      "Friday": {
+        "url": "https://www.youtube.com/watch?v=kfVsfOSbJY0",
+        "message": "Idag är det fredag."
+      },
+      "Saturday": {
+        "url": "https://www.youtube.com/watch?v=GVCzdpagXOQ",
+        "message": "Idag är det lördag."
+      },
+      "Sunday": {
+        "url": "https://www.youtube.com/watch?v=c3BvW56tjB0",
+        "message": "Idag är det söndag."
+      }
     },
 
-    "lunch-message": [
-        "Ska vi ta {}?",
-        "Jag är lite sugen på {}?",
-        "Jag tänkte käka på {}, ska du med?",
-        "{}, där är det mysigt, ska vi ta där?"
-    ],
-
-    "lunch-karlskrona": [
-        "Olles krovbar",
-        "Lila thai stället",
-        "donken",
-        "tex mex stället vid subway",
-        "Subway",
-        "Nya peking",
-        "kebab house",
-        "Royal thai",
-        "thai stället vid hemmakväll",
-        "Gelato",
-        "Indian garden",
-        "Sumo sushi",
-        "Pasterian i stan",
-        "Biobaren",
-        "Michelangelo"
-    ],
-
-    "lunch-bth": [
-        "thairestaurangen vid korsningen",
-        "fiket jämte demolabbet",
-        "Indiska",
-        "Pappa curry",
-        "boden uppe på parkeringen",
-        "Bergåsa kebab",
-        "Pasterian",
-        "Villa Oscar",
-        "Eat here",
-        "Bistro J"
-    ],
-
-    "lunch-angelholm": [
-        "Kirris",
-        "Passagen",
-        "Ica (mmmm broccoli)",
-        "Åkanten (Om sommar och sol)",
-        "Top Falafel",
-        "Subway",
-        "Acapulco",
-        "Kebab på torget",
-        "Ostbutiken",
-        "Hummerhuset",
-        "Klitterhus (fancy stuff)"
-    ],
-
-    "lunch-hassleholm": [
-        "pastavagnen på torget",
-        "Freds",
-        "mcDonalds",
-        "subway",
-        "kinabuffé på Cats",
-        "valentino",
-        "lotterilådan",
-        "casablance",
-        "det där stället i gallerian",
-        "infinity",
-        "östervärn",
-        "argentina",
-        "T4"
-    ],
-
-    "lunch-malmo": [
-        "afrikanskt på Calabass (http://www.calabass.com/)",
-        "mexikanskt på Zócalo (http://zocalo.se/restaurangerna/malmo/)",
-        "hamburgare på Surf Shack (http://surfshacksmashburgers.com/)",
-        "hamburgare på Casual Street Food (http://www.casualstreetfood.se/)",
-        "hamburgare på Tommi's (http://www.tommisburgerjoint.se/)",
-        "kebab på Döner Kebab (http://www.triangeln.com/doner_kebab)"
-    ],
-
-    "lunch-goteborg": [
-        "halvspecial på Preem",
-        "grillat på Odinsgrillen (http://www.odinsparkgrill.se/)",
-        "hamburgare på Burgersson Frigga (http://www.burgersson.se/)",
-        "hamburgare på Tugg (http://gbg.tuggburgers.se/)",
-        "japanskt på Mikado (http://www.mikadogoteborg.se/)",
-        "pizza på I love pizza (http://www.ilovepizza.se/)",
-        "burrito på TomToms Burritos (http://www.tomtoms.se/)",
-        "steak på Brasa (http://www.restaurangbrasa.se/)",
-        "något fint på Post-hotellet (http://www.nordabargrill.se/lunch/)",
-        "något på John Scott's Place (http://johnscotts.se/palace/)",
-        "husman på Vällagat (http://www.vallagat.se/)",
-        "husman på AtWork (http://atwork.nu/restaurang/)",
-        "sallad på MiXiT (http://mixitgbg.se/mixit-garda/)"
-    ],
+    "lunch": {
+        "message" : [
+            "Ska vi ta {}?",
+            "Jag är lite sugen på {}?",
+            "Jag tänkte käka på {}, ska du med?",
+            "{}, där är det mysigt, ska vi ta där?"
+        ],
+        "location": {
+            "karlskrona": [
+                "Olles krovbar",
+                "Lila thai stället",
+                "donken",
+                "tex mex stället vid subway",
+                "Subway",
+                "Nya peking",
+                "kebab house",
+                "Royal thai",
+                "thai stället vid hemmakväll",
+                "Gelato",
+                "Indian garden",
+                "Sumo sushi",
+                "Pasterian i stan",
+                "Biobaren",
+                "Michelangelo"
+            ],
+            "bth": [
+                "thairestaurangen vid korsningen",
+                "fiket jämte demolabbet",
+                "Indiska",
+                "Pappa curry",
+                "boden uppe på parkeringen",
+                "Bergåsa kebab",
+                "Pasterian",
+                "Villa Oscar",
+                "Eat here",
+                "Bistro J"
+            ],
+            "angelholm": [
+                "Kirris",
+                "Passagen",
+                "Ica (mmmm broccoli)",
+                "Åkanten (Om sommar och sol)",
+                "Top Falafel",
+                "Subway",
+                "Acapulco",
+                "Kebab på torget",
+                "Ostbutiken",
+                "Hummerhuset",
+                "Klitterhus (fancy stuff)"
+            ],
+            "hassleholm": [
+                "pastavagnen på torget",
+                "Freds",
+                "mcDonalds",
+                "subway",
+                "kinabuffé på Cats",
+                "valentino",
+                "lotterilådan",
+                "casablance",
+                "det där stället i gallerian",
+                "infinity",
+                "östervärn",
+                "argentina",
+                "T4"
+            ],
+            "malmo": [
+                "afrikanskt på Calabass (https://www.calabass.com/)",
+                "mexikanskt på Zócalo (https://zocalo.se/restaurangerna/malmo/)",
+                "hamburgare på Surf Shack (https://surfshacksmashburgers.com/)",
+                "hamburgare på Casual Street Food (https://www.casualstreetfood.se/)",
+                "hamburgare på Tommi's (https://www.tommisburgerjoint.se/)",
+                "kebab på Döner Kebab (https://www.triangeln.com/doner_kebab)"
+            ],
+            "goteborg": [
+                "halvspecial på Preem",
+                "grillat på Odinsgrillen (https://www.odinsparkgrill.se/)",
+                "hamburgare på Burgersson Frigga (https://www.burgersson.se/)",
+                "hamburgare på Tugg (https://gbg.tuggburgers.se/)",
+                "japanskt på Mikado (https://www.mikadogoteborg.se/)",
+                "pizza på I love pizza (https://www.ilovepizza.se/)",
+                "burrito på TomToms Burritos (https://www.tomtoms.se/)",
+                "steak på Brasa (https://www.restaurangbrasa.se/)",
+                "något fint på Post-hotellet (https://www.nordabargrill.se/lunch/)",
+                "något på John Scott's Place (https://johnscotts.se/palace/)",
+                "husman på Vällagat (https://www.vallagat.se/)",
+                "husman på AtWork (https://atwork.nu/restaurang/)",
+                "sallad på MiXiT (https://mixitgbg.se/mixit-garda/)"
+            ]
+        }
+    },
 
     "sun": {
         "url": "https://api.sunrisesunset.io/json?lat=56.182244&lng=15.5882305",
@@ -189,9 +195,9 @@
     },
 
     "commitstrip": {
-        "message": "Ta en paus och hämta energi med dagens comic strip från commitStrip: {url}, kolla lite gifs på http://thecodinglove.com/, läs dagens artikel på http://thedailywtf.com/ och slösurfa https://www.reddit.com/r/programmerhumor",
-        "url": "http://www.commitstrip.com/en/",
-        "urlPage": "http://www.commitstrip.com/en/page/",
+        "message": "Ta en paus och hämta energi med dagens comic strip från commitStrip: {url}, kolla lite gifs på https://thecodinglove.com/, läs dagens artikel på https://thedailywtf.com/ och slösurfa https://www.reddit.com/r/programmerhumor",
+        "url": "https://www.commitstrip.com/en/",
+        "urlPage": "https://www.commitstrip.com/en/page/",
         "first": 1,
         "last": 688
     },
@@ -220,7 +226,7 @@
     },
 
     "birthday": {
-        "url": "http://dbwebb.se/forum/",
+        "url": "https://dbwebb.se/forum/",
         "somebody": "Idag gratulerar vi {}",
         "nobody": "Ingen som fyller år idag.",
         "error": "Något gick snett, jag har inte en susning om någon fyller år eller inte."
@@ -231,7 +237,7 @@
         "error": "Något gick snett, jag har inte en susning om någon har namnsdag idag eller inte."
     },
     "uptime": {
-        "info": "Kolla in statsen för uptime-tävlingen på http://uptime.dbwebb.se/"
+        "info": "Kolla in statsen för uptime-tävlingen på https://uptime.dbwebb.se/"
     },
     "stream": {
         "info": "Streamen finns på https://dbwebb.se/stream och sparas i spellistan 'Streamade föreläsningar/seminarier' https://www.youtube.com/playlist?list=PLKtP9l5q3ce8Bhxn22oBYWZZ4-3ms1rMl"
@@ -256,7 +262,7 @@
         "error": "Chuck Norris har inte tid underhålla er idag!"
     },
     "commit": {
-        "url": "http://whatthecommit.com/index.txt",
+        "url": "https://whatthecommit.com/index.txt",
         "error": "Du får komma på ett själv. Jag är trasig för tillfället!"
     }
 }