cadey
/
h
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Don't include " dontsave" when displaying weather API failures 

The weather plugin will reply with "Could not find weather for <input>" when the API response doesn't include the "current_observation" key. However, when the user appends " dontsave" to the query that gets reported back to the user and confuses the shit out of everyone.
This commit is contained in:
Alex Vidal 2014-12-30 14:24:57 -06:00
parent 94343e7891
commit 1000be3ea1
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
plugins/weather.py
View File

@ -22,11 +22,12 @@ def weather(inp, chan='', nick='', reply=None, db=None, api_key=None):
loc = None loc = None
dontsave = True dontsave = True
else: else:
loc = inp dontsave = inp.endswith(" dontsave")
# strip off the " dontsave" text if it exists and set it back to `inp` so we don't report it
dontsave = loc.endswith(" dontsave") # back to the user incorrectly
if dontsave: if dontsave:
loc = loc[:-9].strip().lower() inp = inp[:-9].strip().lower()
loc = inp
if not loc: # blank line if not loc: # blank line
loc = db.execute( loc = db.execute(