php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Doc Bug #74054 New DateInterval properties based on `DateInterval::__debuginfo`
Submitted: 2017-02-07 08:24 UTC Modified: 2022-06-05 14:24 UTC
Votes:1
Avg. Score:5.0 ± 0.0
Reproduced:1 of 1 (100.0%)
Same Version:1 (100.0%)
Same OS:1 (100.0%)
From: a dot schilder at gmx dot de Assigned: derick (profile)
Status: Closed Package: Date/time related
PHP Version: Next Major Version OS: irrelevant
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: a dot schilder at gmx dot de
New email:
PHP Version: OS:

 

 [2017-02-07 08:24 UTC] a dot schilder at gmx dot de 
Description:
------------
The following properties are missing in the documentation (http://www.php.net/class.dateinterval):
- 'f' (decimal fraction of second)
- 'weekday' (0 = undefined, 1 to 7 = Monday to Sunday?)
- 'weekday_behavior' (?)
- 'first_last_day_of' (?)
- 'special_type' (?)
- 'special_amount' (?)
- 'have_weekday_relative' (?)
- 'have_special_relative' (?)

Especially the 'f' property is important when comparing two values, and it should be noted that the decimal fraction of the seconds was ignored in previous PHP versions.



Expected result:
----------------
Documented properties

Actual result:
--------------
Undocumented properties

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2017-02-07 12:04 UTC] cmb@php.net
-Package: Documentation problem +Package: Date/time related
 [2020-02-16 19:27 UTC] carusogabriel@php.net
-Summary: DateInterval property documentation incomplete +Summary: New DateInterval properties based on `DateInterval::__debuginfo` -Type: Documentation Problem +Type: Feature/Change Request -PHP Version: 7.1.1 +PHP Version: Next Major Version
 [2020-02-16 19:27 UTC] carusogabriel@php.net 
Apart from `f`, all the properties that you mentioned they do not exist: https://3v4l.org/2vfiq.

I don't think this is a Doc Bug, but rather a Feature Request than.
 [2020-02-16 22:19 UTC] cmb@php.net 
> Apart from `f`, all the properties that you mentioned they do
> not exist […]

That depends on the state of the object.  After it has been
var_dump()ed, for example, the properties do exist, see
<https://3v4l.org/Xaov2>.

I doubt that we want to document these properties, though.
 [2020-02-16 22:22 UTC] nikic@php.net 
Huh, I thought we had fixed these "properties appear after var_dump" cases... I guess not for the cases where there are both real properties and debug properties.
 [2020-02-16 22:48 UTC] carusogabriel@php.net 
Related To: Bug #68940
 [2021-04-07 16:40 UTC] cmb@php.net 
Related To: Bug #76588
 [2022-06-05 14:22 UTC] git@php.net 
Automatic comment on behalf of derickr
Revision: https://github.com/php/doc-en/commit/4586854a24b1ca532b5c9ba2410db1f1eee338ed
Log: Fixed bug #74054: New DateInterval properties based on
 [2022-06-05 14:22 UTC] git@php.net
-Status: Open +Status: Closed
 [2022-06-05 14:24 UTC] derick@php.net
-Type: Feature/Change Request +Type: Documentation Problem -Assigned To: +Assigned To: derick
 [2022-06-05 14:24 UTC] derick@php.net 
The fix for this bug has been committed.
If you are still experiencing this bug, try to check out latest source from https://github.com/php/php-src and re-test.
Thank you for the report, and for helping us make PHP better.

PHP 8.2 handles this now in a much saner way, and henceforth I changed this back into a Documentation bug, which I can also close as the documentation for this change has now been merged into the documentation's sources.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Sat Dec 21 14:01:32 2024 UTC