Req #79442 [Opn->Wfx]: Modify does not support multiple modifications at once

From: Date: Sat, 04 Jun 2022 17:20:48 +0000
Subject: Req #79442 [Opn->Wfx]: Modify does not support multiple modifications at once
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-241663@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=79442&edit=1

 ID:                 79442
 Updated by:         derick@php.net
 Reported by:        michael dot vorisek at email dot cz
 Summary:            Modify does not support multiple modifications at
                     once
-Status:             Open
+Status:             Wont fix
 Type:               Feature/Change Request
 Package:            Date/time related
 PHP Version:        7.4.4
 Block user comment: N
 Private report:     N

 New Comment:

The architecture currently does not allow for this, and hence I can not add this. You will have to
split it yourself and call modify() multiple times.


Previous Comments:
------------------------------------------------------------------------
[2020-04-01 08:39:17] derick@php.net

> Is it currently possible to use multiple modifications to a date like "first day of -2
> months, +15 days" in one modify() call?

No, that's not possible.

> Is it currently possible to to write one modify for "15th day of -2 months"?

I don't think so, but I will check and see whether it's easy to add.

I also see that you're using "clone" here. If instead of \DateTime you use
\DateTimeImmutable, that clone isn't needed, and you will have less issues running two
->modify() calls (as you don't need to clone twice).

------------------------------------------------------------------------
[2020-04-01 08:04:42] michael dot vorisek at email dot cz

We are using modify string from external source where we have one field for it. Basically we do
$date->modify($modString)->format().

If my use case is not supported, I can explode the modify string and call modify() multiple times,
but if there is currently a way to write "15th day of -2 months" please let me know.

"first day of" works, so to be consistent "nth day of" should work too

------------------------------------------------------------------------
[2020-04-01 07:56:31] sjon@php.net

what's wrong with two modify calls? It will allow you to do as many modifications as you want,
I don't see how this qualifies as a bug. This works fine:

$d = new \DateTime('2020-03-28 12:00:00');
echo $d ->modify('first day of -2 months')
        ->modify('+15 days')
        ->format('j.n.Y');

------------------------------------------------------------------------
[2020-04-01 07:38:07] michael dot vorisek at email dot cz

Description:
------------
Is it currently possible to use multiple modifications to a date like "first day of -2 months,
+15 days" in one modify() call?

Is it currently possible to to write one modify for "15th day of -2 months"?

Test script:
---------------
$d = new \DateTime('2020-03-28 12:00:00');

// first day of the month, ok
echo ((clone $d)->modify('last day of -2 months')->format('j.n.Y')) .
"\n";

// 15th day of the month - not working
echo ((clone $d)->modify('first day of -2 months, +15
days')->format('j.n.Y')) . "\n"; // bad result
echo ((clone $d)->modify('15th day of -2 months')->format('j.n.Y')) .
"\n"; // not parseable modify string

Expected result:
----------------
31.1.2020
15.1.2020
15.1.2020

Actual result:
--------------
31.1.2020
31.1.2020
Warning: DateTime::modify(): Failed to parse time string (15th day of -2 months) at position 0 (1)
Fatal error: Uncaught Error: Call to a member function format() on bool


------------------------------------------------------------------------



--
Edit this bug report at https://bugs.php.net/bug.php?id=79442&edit=1


Thread (6 messages)

« previous php.bugs (#241663) next »