|
php.net | support | documentation | report a bug | advanced search | search howto | statistics | random bug | login |
[2020-01-11 21:42 UTC] jimbo2150 at gmail dot com
Description:
------------
A relative date format used in DateInterval produces and infinite loop when used with DatePeriod.
An example is looking for the first Monday in January of each year. When iterated via DatePeriod the year will not get iterated and the correct relative date for current year keeps getting returned creating an infinite loop.
Test script:
---------------
// Looking for every Monday of January each year
$fmj = DateInterval::createFromDateString('first Monday of January');
$period = new DatePeriod(new DateTime('2020-01-01'), $fmj, new DateTime('2022-01-11'));
// Infinite loop, first Monday of January of current year keeps getting iterated
foreach ($period as $dt) {
var_dump($dt->format("l Y-m-d H:i:s"));
}
Expected result:
----------------
Return each first Monday of January for the next 2 years.
string(30) "Wednesday 2020-01-01 00:00:00"
string(27) "Monday 2020-01-06 00:00:00"
string(27) "Monday 2021-01-04 00:00:00"
string(27) "Monday 2022-01-03 00:00:00"
Actual result:
--------------
Infinite loop where each date is the first Monday of the current year's January.
string(30) "Wednesday 2020-01-01 00:00:00"
string(27) "Monday 2020-01-06 00:00:00"
string(27) "Monday 2020-01-06 00:00:00"
string(27) "Monday 2020-01-06 00:00:00"
string(27) "Monday 2020-01-06 00:00:00"
string(27) "Monday 2020-01-06 00:00:00"
string(27) "Monday 2020-01-06 00:00:00"
string(27) "Monday 2020-01-06 00:00:00"
string(27) "Monday 2020-01-06 00:00:00"
...
{infinite loop}
PatchesPull RequestsHistoryAllCommentsChangesGit/SVN commits
|
|||||||||||||||||||||||||||||||||||||
Copyright © 2001-2025 The PHP GroupAll rights reserved. |
Last updated: Mon Dec 08 23:00:02 2025 UTC |
Looks like it can't determine how long "first Monday of January" is. echo $fmj->format('%y years, %m months, %d days, %h hours, %i minutes, %s seconds'); What numbers should it be using for those fields?Not sure what you mean by the numbers you are looking for. I got closer with this code (added "next year" to the DatePeriod object, but the result is erratic (not sure what is being calculated): $fmj = DateInterval::createFromDateString('first Monday of January next year'); $period = new DatePeriod(new DateTime('first Monday of January 2020'), $fmj, new DateTime('2022-01-31')); foreach ($period as $dt) { var_dump($dt->format("l Y-m-d H:i:s")); } // Strange, incorrect output (not the first Monday of January 2021 or 2022): string(26) "Monday 2020-01-06 00:00:00" string(29) "Wednesday 2021-01-06 00:00:00" string(27) "Tuesday 2022-01-04 00:00:00"$timezone = new DateTimeZone( "UTC" ); $fmj = DateInterval::createFromDateString( 'first Monday of January 2020' ); //Monday from start 2019 only $period = new DatePeriod( new DateTime( '2019-01-01 00:00:00.000000', $timezone ), $fmj, new DateTime( '2022-01-11 00:00:00.000000', $timezone ) ); $y = 0; foreach ( $period as $dt ) { $display[$y++] = $dt->format( "l Y-m-d H:i:s" ); //stop total_recurrence if( $y == 5 ) { break; } } Manually stop infinite loop :) The relative date, first Monday of jenaury is extended for the current year but for the parametres start and end of function DatePeriod , only uses the start, there are two bugs in the php source code, the reference is not counted if the code does not return the expected result, the second bug is stopping on start.DateInterval can indicate the progress of the year, while in the second form it can be applied to the current year. In the second you can use modify or directly in the constructor, since you are not using a range between two dates. The progress of the year DateInterval::createFromDateString ('first Monday of January next year');. Specific year not supported by Dateinterval new DateTime ('first monday of January', new DateTimeZone ("UTC")); without specifying is the same as saying the current year. Currently DataPeriod incorrectly calculates the distance between two relative dates, expressed previously. Fifth I ask that Dateinterval throw an exception if with that relative date. https://3v4l.org/uJpiUI am marking this as "won't fix" as this is not doable with the current architecture. You will have to make your own loop, and use the ->modify() method more than once in each iteration to obtain this result: $current = new DateTimeImmutable('2020-01-01'); while ( $current < new DateTimeImmutable('2022-01-11') ) { $current = $current->modify('monday'); echo $current->format('Y-m-d'), "\n"; $current = $current->modify('next year January 1st'); }