Weather source html has changed, breaking the weather plugin

Bug #716141 reported by marcog
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ibid
Fix Committed
Medium
Stefano Rivera

Bug Description

23:54 < marcog> tibid: weather for cape town
23:54 < tibid> marcog: That didn't go down very well. Burp.
23:54 < marcog> tibid: last exception
23:54 < tibid> marcog: message event u'tibid: weather for cape town' triggered KeyError:
               u'sunrise'

It looks like it might be a simple change of case.

Related branches

Revision history for this message
Stefano Rivera (stefanor) wrote :

I can't reproduce this.

Changed in ibid:
status: New → Incomplete
Changed in ibid:
milestone: 0.1.1 → none
Revision history for this message
Stefano Rivera (stefanor) wrote :

It's back again. The cause is an unexpected </table>

Changed in ibid:
milestone: none → 0.1.2
status: Incomplete → Confirmed
Revision history for this message
Stefano Rivera (stefanor) wrote :

And gone again, but here's the sample

Revision history for this message
michael perigard (overprescribed) wrote : Re: [Bug 716141] Re: Weather source html has changed, breaking the weather plugin

I'm on vacation atm, but I've already written a parser for this data
that uses the XML API as opposed to screen scraping the mobile page.
I'll submit a merge request when I get back.

Revision history for this message
Max Rabkin (max-rabkin) wrote :

Michael: any progress on this? If you're not working on it, perhaps the best thing is to dump a patch here.

Changed in ibid:
assignee: nobody → Stefano Rivera (stefanor)
milestone: 0.1.2 → 0.2.0
status: Confirmed → In Progress
Changed in ibid:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.