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 13:33 UTC Modified: 2009-03-12 18:18 UTC
From: mrubinsk Assigned: Eru
Status: Closed Package: Services_Weather (version 1.4.3)
PHP Version: Irrelevant OS:
Roadmaps: (Not assigned)    
Subscription  


 [2009-01-14 13: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 09: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 10: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 11:56 UTC] mrubinsk (Michael Rubinsky)
Patch attached, as well as a bare bones PHPUnit test that tests the time related parsing.
 [2009-03-10 17:08 UTC] mrubinsk (Michael Rubinsky)
-PHP Version: 5.2.8 +PHP Version: Irrelevant
Ping?
 [2009-03-12 17:59 UTC] eru (Alexander Wirtz)
-Status: Feedback +Status: Assigned -Assigned To: +Assigned To: Eru
Pong.
 [2009-03-12 18: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!