Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#8056 closed patch (worksforme)

BBC weather web page probably changed

Reported by: ido roseman <ido_roseman@…> Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: Plugin - MythWeather Version: unknown
Severity: medium Keywords:
Cc: Ticket locked: no

Description

upgrading my system i could not find tel-aviv in mythweather. after some changes in /usr/share/mythtv/mythweather/scripts/uk_bbc/BBCLocation.pm all is well again :-) didn't save original file so can't diff, but here it is ... line 15 should be

my $base_url = 'http://news.bbc.co.uk/weather/forecast/75';

line 69 should be

elsif (/<a id=\"result_\d*\" class\=\"result\" href \=\"\/weather\/$

Change History (4)

comment:1 Changed 9 years ago by Joe Ripley <vitaminjoe@…>

This works fine for me. Seems like you're using a version of the grabber script older than [22988] that doesn't have patch #7758 applied.

You can verify your changes against the current revision of the code here: BBCLocation.pm

-- Joe Ripley vitaminjoe@…

comment:2 Changed 9 years ago by paulh

Status: newinfoneeded_new

ido, can you still reproduce this in current trunk?

comment:3 Changed 9 years ago by paulh

Resolution: worksforme
Status: infoneeded_newclosed

Searching for Tel Aviv is working fine in current trunk.

comment:4 Changed 9 years ago by anonymous

I was spending the last couple of days compiling myth from svn. unfortunately mythweather is not working so I can't test it myself. but if it works for you, I don't see a reason why it's not work for me. thanks

Note: See TracTickets for help on using tickets.