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)    

Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
Solve the problem : 16 - 15 = ?

 [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 for more examples.


 [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
 [2009-03-12 21:59 UTC] eru (Alexander Wirtz)
-Status: Feedback +Status: Assigned -Assigned To: +Assigned To: Eru
 [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 by the end of next Sunday (CET). If this was a problem with the 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!