Merge https://github.com/sbp/phenny

modules/head.py

@@ -168,7 +168,7 @@ def noteuri(phenny, input):
    if not hasattr(phenny.bot, 'last_seen_uri'): 
       phenny.bot.last_seen_uri = {}
    phenny.bot.last_seen_uri[input.sender] = uri
-noteuri.rule = r'.*(http://[^<> "\x01]+)[,.]?'
+noteuri.rule = r'.*(http[s]?://[^<> "\x01]+)[,.]?'
 noteuri.priority = 'low'
 
 if __name__ == '__main__': 

modules/oblique.py

@@ -34,10 +34,15 @@ def service(phenny, input, command, args):
    template = template.replace('${nick}', urllib.quote(input.nick))
    uri = template.replace('${sender}', urllib.quote(input.sender))
 
+   info = web.head(uri)
+   if isinstance(info, list): 
+      info = info[0]
+   if not 'text/plain' in info.get('content-type', '').lower(): 
+      return phenny.reply("Sorry, the service didn't respond in plain text.")
    bytes = web.get(uri)
    lines = bytes.splitlines()
    if not lines: 
-      return phenny.reply('Sorry, the service is broken.')
+      return phenny.reply("Sorry, the service didn't respond any output.")
    phenny.say(lines[0][:350])
 
 def refresh(phenny): 

modules/tell.py

@@ -29,7 +29,8 @@ def loadReminders(fn):
    for line in f: 
       line = line.strip()
       if line: 
-         tellee, teller, verb, timenow, msg = line.split('\t', 4)
+         try: tellee, teller, verb, timenow, msg = line.split('\t', 4)
+         except ValueError: continue # @@ hmm
          result.setdefault(tellee, []).append((teller, verb, timenow, msg))
    f.close()
    return result

modules/translate.py

@@ -46,6 +46,8 @@ def tr(phenny, context):
 
    if input != output: 
       msg = translate(phrase, input, output)
+      if isinstance(msg, str): 
+         msg = msg.decode('utf-8')
       if msg: 
          msg = web.decode(msg) # msg.replace(''', "'")
          msg = '"%s" (%s to %s, translate.google.com)' % (msg, input, output)