| Summary | WeatherUnderground forecast only reports three days |
| Queue | Horde Framework Packages |
| Queue Version | Git master |
| Type | Bug |
| State | Resolved |
| Priority | 1. Low |
| Owners | mrubinsk (at) horde (dot) org |
| Requester | arjen+horde (at) de-korte (dot) org |
| Created | 02/18/2012 (4889 days ago) |
| Due | |
| Updated | 02/18/2012 (4889 days ago) |
| Assigned | |
| Resolved | 02/18/2012 (4889 days ago) |
| Github Issue Link | |
| Github Pull Request | |
| Milestone | |
| Patch | Yes |
commit 92dd38c2c78ab8380bc435848f3ae7f582d13eac
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date: Sat Feb 18 12:13:20 2012 -0500
Allow emulating a smaller forecast length.
Emulate shorter forecast length by requesting longer forecast and setting
a top to the number of days returned. Implemented now that
WeatherUnderground
has done away with previously supported 5 day forecast. We now emulate it
with 7 day forecast and limitDays() call.
Bug: 11010.../lib/Horde/Service/Weather/Forecast/Base.php | 20
+++++++++++++++++++-
.../Horde/Service/Weather/WeatherUnderground.php | 3 ++-
framework/Service_Weather/package.xml | 12 ++++++++----
3 files changed, 29 insertions(+), 6 deletions(-)
http://git.horde.org/horde-git/-/commit/92dd38c2c78ab8380bc435848f3ae7f582d13eac
Assigned to Michael Rubinsky
State ⇒ Resolved
now only reports three days, so we can't use that for a five day
forecast anymore.
so it might be better to use 'forecast10day' instead (for anything
longer than 3 days).
commit 92dd38c2c78ab8380bc435848f3ae7f582d13eac
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date: Sat Feb 18 12:13:20 2012 -0500
Allow emulating a smaller forecast length.
Emulate shorter forecast length by requesting longer forecast and setting
a top to the number of days returned. Implemented now that
WeatherUnderground
has done away with previously supported 5 day forecast. We now emulate it
with 7 day forecast and limitDays() call.
Bug: 11010.../lib/Horde/Service/Weather/Forecast/Base.php | 20
+++++++++++++++++++-
.../Horde/Service/Weather/WeatherUnderground.php | 3 ++-
framework/Service_Weather/package.xml | 12 ++++++++----
3 files changed, 29 insertions(+), 6 deletions(-)
http://git.horde.org/horde-git/-/commit/92dd38c2c78ab8380bc435848f3ae7f582d13eac
Priority ⇒ 1. Low
New Attachment: WeatherUnderground.diff
Patch ⇒ Yes
Milestone ⇒
Queue ⇒ Horde Framework Packages
Summary ⇒ WeatherUnderground forecast only reports three days
Type ⇒ Bug
State ⇒ Unconfirmed
only reports three days, so we can't use that for a five day forecast
anymore.
The seven day forecast is no longer mentioned in the documentation, so
it might be better to use 'forecast10day' instead (for anything longer
than 3 days).