Home  |  Linux  | Mysql  | PHP  | XML
Date:Wed Mar 14 09:19:41 2012
Subject:Req #61366 [Opn]: DateInterval should support weeks and days combined
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
Navigate in group php.bugs at sever news.php.net
Previous Next





  
© No Copyright
You are free to use Anything, but please consult your advocate before doing so as this website
also list content from other sources which may be copyrighted.