Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:113574
To: internals@lists.php.net
References: <BFDC5ABE-3155-4C4D-84DE-0FB1D9B37D32@9dev.de>
 <alpine.DEB.2.23.453.2103150917420.2842@singlemalt.home.derickrethans.nl>
 <AFD3BCAD-F628-460C-B450-EC8CCEE179BD@9dev.de>
 <alpine.DEB.2.23.453.2103161145500.2842@singlemalt.home.derickrethans.nl>
 <CB06AC6B-E914-4C85-89EC-0C1DA0789AD4@9dev.de>
 <alpine.DEB.2.23.453.2103161529460.2842@singlemalt.home.derickrethans.nl>
 <05166649-DFD5-4B93-8D4C-1E19CE129EFF@9dev.de>
Message-ID: <5a226984-dc4e-535a-ba01-62daacbe2107@gmail.com>
Date: Tue, 16 Mar 2021 19:04:33 +0000
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.8.1
MIME-Version: 1.0
In-Reply-To: <05166649-DFD5-4B93-8D4C-1E19CE129EFF@9dev.de>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 8bit
Content-Language: en-GB
Subject: Re: [PHP-DEV] Add __toString() to DateInterval
From: rowan.collins@gmail.com (Rowan Tommins)

On 16/03/2021 18:56, Moritz Friedrich wrote:
> Again, I’m not attached to `__toString()`. Having another helper method or a special formatting constant, as suggested by others in this thread, would likewise be a viable option. I’d merely prefer to have a built-in way of parsing*and*  composing time intervals as per the ISO specification, making the API consistent and easing communication with non-PHP systems.


Would a reasonable way forward be to have such a named method, and have 
it throw an exception if the instance is not representable in that form?

That way, you'd be guaranteed that new 
DateInterval($period->toIsoPeriodString()) would result in an equivalent 
object, rather than discarding special relative values.

Regards,

-- 
Rowan Tommins
[IMSoP]