[cvs] commit: horde/lib/Block weatherdotcom.php
Chuck Hagenbuch
chuck at horde.org
Sat Jan 24 15:24:46 PST 2004
chuck 2004/01/24 15:24:46 PST
Modified files:
lib/Block weatherdotcom.php
Log:
1. The weather.com logo and an attribution to weather.com for providing the
data are required on the displayed page. Both are supposed to link to the
weather.com site. I've moved the logo from the title to the bottom of the
block, and added the attribution next to it.
2. It turns out that weather.com accepts a zip code as a location ID. I added a
check to see if the user entered a zip code; if so, no search is necessary and
the zip code becomes the location ID. This may also require modification of
the validation routine when the blocks use horde_form validation. I also added
some error-checking to catch and report invalid locations. This should also
become unnecessary (or redundant) after horde_form validation is used.
3. Display the update time as local time (instead of GMT).
4. The forecast for the current day (not night) is no longer valid after 14:00
local time, so don't display it. Also, use day headers "Today" and "Tomorrow"
(instead of the day name).
Submitted By: Rick Emery <rick at emery.homelinux.net>
Revision Changes Path
1.4 +61 -27 horde/lib/Block/weatherdotcom.php
Chora Links:
http://cvs.horde.org/diff.php/horde/lib/Block/weatherdotcom.php?r1=1.3&r2=1.4&ty=u
More information about the cvs
mailing list