Skip Menu |

This queue is for tickets about the WWW-Wunderground-API CPAN distribution.

Report information
The Basics
Id: 71737
Status: resolved
Priority: 0/
Queue: WWW-Wunderground-API

People
Owner: Nobody in particular
Requestors: wsnyder [...] wsnyder.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in: 0.01
Fixed in: 0.02



Subject: Doc fixes & JSON
Nice module - does exactly what I needed, thanks. Minor nits on the usage section, I tried this example: my $wun = new WWW::Wunderground::API(01111); print 'The temperature is: '.$wun->data->temp_f; print 'XML source:'.$wun->xml; And was confused by the result, then realized the 01111 is octal of course; so it would be nice to change your docs to save the next person a few minutes to this: #or zipcode my $wun = new WWW::Wunderground::API('22030'); #or airport identifier my $wun = new WWW::Wunderground::API('KIAD'); print 'The temperature is: '.$wun->data->temp_f."\n"; print 'XML source:'.$wun->xml."\n"; Also added the newlines to make it look nicer. Finally I was looking for the XML docs on wunderground, and I think the XML format is deprecated and JSON should be used instead, see http://wiki.wunderground.com/index.php/API_-_XML . Fine for now, but... Thanks.
From: nebulous
On Mon Oct 17 09:07:35 2011, WSNYDER wrote: Show quoted text
> Nice module - does exactly what I needed, thanks. > > Minor nits on the usage section, I tried this example: > > my $wun = new WWW::Wunderground::API(01111); > print 'The temperature is: '.$wun->data->temp_f; > print 'XML source:'.$wun->xml; > > And was confused by the result, then realized the 01111 is octal of > course
Hah, good catch. Updating docs. Show quoted text
> Finally I was looking for the XML docs on wunderground, and I think
the Show quoted text
> XML format is deprecated and JSON should be used instead, see > http://wiki.wunderground.com/index.php/API_-_XML . Fine for now,
but... Show quoted text
>
I suppose this is good and bad. I much prefer JSON, but hate forcing users to create an API key. I'll see if I can add it in as an option. Thanks for the report.
Changed documentation and added JSON API support as requested in version 0.02