Resolve #926

[maxpowa]

Also cleaned up dict value retrieval a little, the .get() calls were a bit unnecessary.

[dgw]

Should probably be squashed into a single commit that says "Removed feedparser dependency (closes #926)" or something like that. But 👍

[maxpowa]

But muh commiting from github web

[dgw]

That sounds painful lol

[embolalia]

The get calls are necessary, because the result will not always have those keys in it. get will return None, rather than raising a KeyError.

[maxpowa]

Ah good call, didn't even think about that. Should I edit this PR or create a new one to add them back in?

[embolalia]

You can just edit this one. Thanks!

[embolalia]

I got this exception while testing this with .weather new york, ny:

Traceback (most recent call last):
  File "/usr/local/lib/python3.4/dist-packages/xmltodict.py", line 246, in parse
    parser.ParseFile(xml_input)
TypeError: argument must have 'read' attribute

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/embo/src/willie/sopel/bot.py", line 262, in call
    exit_code = func(sopel, trigger)
  File "/home/embo/src/willie/sopel/modules/weather.py", line 147, in weather
    parsed = xmltodict.parse(url).get('rss')
  File "/usr/local/lib/python3.4/dist-packages/xmltodict.py", line 248, in parse
    parser.Parse(xml_input, True)
xml.parsers.expat.ExpatError: not well-formed (invalid token): line 1, column 5

[maxpowa]

Yeaaaah, don't ask how that one slipped through.

[embolalia]

Thanks!