Edit report at http://pear.php.net/bugs/bug.php?id=16802&edit=1
ID: 16802
Updated by: [email protected]
Reported By: jan at horde dot org
Summary: Location used without being processed
-Status: Open
+Status: Closed
Type: Bug
Package: Services_Weather
Package Version: 1.4.5
PHP Version: Irrelevant
-Assigned To:
+Assigned To: eru
Roadmap Versions:
New Comment:
-Status: Open
+Status: Closed
-Assigned To:
+Assigned To: eru
This bug has been fixed in SVN.
If this was a documentation problem, the fix will appear on pear.php.net
by the end of next Sunday (CET).
If this was a problem with the pear.php.net website, the change should
be live shortly.
Otherwise, the fix will appear in the package's next release.
Thank you for the report and for helping us make PEAR better.
Thanks for the report and especially the patch!
Previous Comments:
------------------------------------------------------------------------
[2009-12-08 15:13:17] yunosh
You've got to run the script twice, because the errors are only
triggered after a cache hit.
------------------------------------------------------------------------
[2009-12-08 15:12:40] yunosh
Added #patch bug:16802;patch:location_test.php;revision:1260285160;.
------------------------------------------------------------------------
[2009-12-08 14:05:26] doconnor
Any chance of a minor executable test script to fully demonstrate the
problem?
------------------------------------------------------------------------
[2009-11-18 23:12:19] yunosh
Added #patch bug:16802;patch:location.diff;revision:1258585939;.
------------------------------------------------------------------------
[2009-11-18 23:12:06] yunosh
Description:
------------
getWeather() and getForecast() of the weather.com driver use the
$_location property which is only being set by the getLocation() method.
So unless you call that information first, you get PHP notices and
broken behavior.
------------------------------------------------------------------------
--
Edit this bug report at http://pear.php.net/bugs/bug.php?id=16802&edit=1
lmpx.com only provides a reader for public news (NNTP) servers. It is not
affiliated with the servers or forums shown here and is not responsible for
the content of articles, which is written by their respective authors.