Bug #6781

METAR Location ID check rejects valid ICAO codes

Details

Submitted	2006-02-14 14:41 UTC
From	seth at pricepages dot org
Assigned	eru
Status	Closed
Package	Services_Weather
PHP Version	Irrelevant
OS	Mac
Roadmaps	(Not assigned)

Comments

[2006-02-14 14:41 UTC] seth at pricepages dot org

Description:
------------
In Metar.php, in function _checkLocationID(), the call to
ctype_alpha() rejects valid ICAO codes. Rejected codes
include K99S, K15C, K07S, K43S, and K28Y.

I'm just removing that call to ctype_alpha() for now. A
proper check will most likely include some research and a
regex.

[2006-03-08 12:41 UTC] eru at php dot net

This bug has been fixed in CVS.

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.

It was never the goal to check the code for existance, just to validate, if it's plausible. So a replacement with ctype_alnum is sufficient.

Btw.: sorry for the delay...