| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.4.7 |
| Bug #15637 | New TAF format breaks date groups | ||
|---|---|---|---|
| Submitted: | 2009-01-14 18:33 UTC | ||
| From: | mrubinsk | Assigned: | Eru |
| Status: | Closed | Package: | Services_Weather (version 1.4.3) |
| PHP Version: | Irrelevant | OS: | |
| Roadmaps: | (Not assigned) | ||
[2009-01-14 18:33 UTC] mrubinsk
(Michael Rubinsky)
Description:
------------
TAF format update to conform to the ICAO Annex 3 standards, in which date groups (such as the valid from/to times) include the day of month as well.
Example:
KXXX 241732Z 2418/2524 ..... would indicate a valid time FROM the 24th day at 1800 UTC TO the 25th day at 2400 UTC
or a FM grouping:
FM242300
would represent a change starting at the 24th day at 2300 UTC
See http://aviationweather.gov/notice/taf30.php for more examples.
Comments
[2009-01-15 14:23 UTC] doconnor
(Daniel O'Connor)
Hey Michael, can you turn this into some simple executable test cases or patches by chance?
IE:
assert($foo->render() == "KXXX 241732Z 2418/2524"); // Verify we're outputting valid stuff!
[2009-01-15 15:29 UTC] mrubinsk
(Michael Rubinsky)
Sure. I'll put a test case together as soon as I have a chance to. I'll also *try* to get a working patch together if I can... :)
[2009-01-16 16:56 UTC] mrubinsk
(Michael Rubinsky)
Patch attached, as well as a bare bones PHPUnit test that tests the time related parsing.
[2009-03-10 21:08 UTC] mrubinsk
(Michael Rubinsky)
-PHP Version: 5.2.8
+PHP Version: Irrelevant
Ping? [2009-03-12 21:59 UTC] eru
(Alexander Wirtz)
-Status: Feedback
+Status: Assigned
-Assigned To:
+Assigned To: Eru
Pong. [2009-03-12 22:18 UTC] eru
(Alexander Wirtz)
-Status: Assigned
+Status: Closed
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.
Many thanks for the patch, Michael!