| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.4.7 |
| Bug #6781 | METAR Location ID check rejects valid ICAO codes | ||
|---|---|---|---|
| Submitted: | 2006-02-14 14:41 UTC | ||
| From: | sprice | Assigned: | eru |
| Status: | Closed | Package: | Services_Weather (version 1.3.2) |
| PHP Version: | Irrelevant | OS: | Mac |
| Roadmaps: | (Not assigned) | ||
[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.
Comments
[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...