Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#10231 closed Patch - Bug Fix (fixed) won't show list of cities during adding new location

Reported by: j.novak@… Owned by: beirdo
Priority: minor Milestone:
Component: Plugin - MythWeather Version: 0.24.1
Severity: medium Keywords:
Cc: Ticket locked: no

Description won't show list of cities during adding new location. The reason is that changed format of page which script parsed. You can check it with ./ -l liberec - It won't return any location.

I found that there is no way how to update script to parse homepage correctly, so I updated script to use JSON API at to find cities as home page does.

It adds dependency on JSON perl module.

Note: During work on code I found that code uses -D parameter for debugging, but command line parser don't accept it. I updated it too.

Attachments (1) (3.1 KB) - added by j.novak@… 9 years ago.
Patch to

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by j.novak@…

Attachment: added

Patch to

comment:1 Changed 9 years ago by Github

Milestone: unknown0.25
Resolution: fixed
Status: newclosed

Fix yrnoxml weather script

Seems has changed their site around a bit, in particular the searching. This now should work properly again.

Fixes #10231.

Based on the patch in the ticket, from j.novak@… but reformatted and reworked a bit.

Branch: master Changeset: 75bde2a1bea691c9b1374f2378ecb60bad1d79e7

comment:2 Changed 9 years ago by beirdo


Also backported to fixes-0.24

comment:3 Changed 9 years ago by stuartm

Milestone: 0.24.2

Milestone 0.24.2 deleted

Note: See TracTickets for help on using tickets.