Edit report at https://bugs.php.net/bug.php?id=61366&edit=1
ID: 61366 User updated by: evert at rooftopsolutions dot nl Reported by: evert at rooftopsolutions dot nl Summary: DateInterval should support weeks and days combined Status: Open Type: Feature/Change Request Package: Date/time related Operating System: Any PHP Version: 5.3.10 Block user comment: N Private report: N New Comment: Well, I have several counter points: 1) If it's not supported, it should throw an exception. At the moment it fails silently. 2) There's no reason to support a superset of ISO 8601 3) I'm using this because I'm implementing an iCalendar parser. The DURATION property type is based on ISO 8601: http://tools.ietf.org/html/rfc5545#section-3.3.6 In the wild I'm finding duration values with mixed weeks and days. Now I need to pre-parse this string, do weeks * 7, then regenerate the (almost identical) string before I can add it to the constructor. I feel adding support for W and D at the same time does not pose additional confusion, and is a sensible feature. Previous Comments: ------------------------------------------------------------------------ [2012-03-14 01:51:40] uramihsayibok at gmail dot com ISO 8601 doesn't allow mixing weeks with anything else. If you need both then just do the math: 7 * weeks + days ------------------------------------------------------------------------ [2012-03-12 21:53:19] evert at rooftopsolutions dot nl Description: ------------ The DateInterval should support specifying weeks and days at the same time. This is documented to not be supported, but this is imho silly and confusing. Test script: --------------- <?php $di = new DateIterval('P1W1D'); echo $di->d; ?> Expected result: ---------------- 8 Actual result: -------------- 1 ------------------------------------------------------------------------ -- Edit this bug report at https://bugs.php.net/bug.php?id=61366&edit=1
