php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Doc Bug #54630 listed as a PHP 5 >= 5.1.0 when should PHP 5 >= 5.2.0
Submitted: 2011-04-29 06:58 UTC Modified: 2011-04-29 13:42 UTC
From: daniel dot black at ovee dot com dot au Assigned: pierrick (profile)
Status: Closed Package: Documentation problem
PHP Version: trunk-SVN-2011-04-29 (SVN) OS:
Private report: No CVE-ID: None
 [2011-04-29 06:58 UTC] daniel dot black at ovee dot com dot au
Description:
------------
---
From manual page: http://www.php.net/function.timezone-identifiers-
list#Description
---
timezone_identifiers_list

The dependency info should the the same as the real function it is an alias for.

Expected result:
----------------
	PHP 5 >= 5.2.0 
in description


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2011-04-29 13:12 UTC] salathe@php.net
-Status: Open +Status: Wont fix
 [2011-04-29 13:12 UTC] salathe@php.net
The function timezone_identifiers_list() was available (experimentally) from 
5.1.0, the associated method DateTimeZone::listIdentifiers() was only added in 
5.2.0.  The versions should be kept as-is, even if they are confusing.
 [2011-04-29 13:14 UTC] pierrick@php.net
Automatic comment from SVN on behalf of pierrick
Revision: http://svn.php.net/viewvc/?view=revision&revision=310649
Log: Fixed bug #54630
 [2011-04-29 13:15 UTC] pierrick@php.net
-Status: Wont fix +Status: Closed -Assigned To: +Assigned To: pierrick
 [2011-04-29 13:15 UTC] pierrick@php.net
This bug has been fixed in the documentation's XML sources. Since the
online and downloadable versions of the documentation need some time
to get updated, we would like to ask you to be a bit patient.

Thank you for the report, and for helping us make our documentation better.


 [2011-04-29 13:23 UTC] pierrick@php.net
Sorry Peter,

Did not noticed your answer before submitting my patch.
I compiled a 5.1 version by default and it was not available in it.

pierrick@magtured:~/php-src/tags/php_5_1_0$ sapi/cli/php -r 
'var_dump(timezone_identifiers_list());'

Fatal error: Call to undefined function timezone_identifiers_list() in Command 
line code on line 1

I think it is really confusing for people furthermore since the documentation 
says that this is an alias of DateTimeZone::listIdentifiers() which is 
documented as PHP 5 >= 5.2.0.

And the issue is the same with :
- timezone_abbreviations_list
- timezone_name_get
- timezone_offset_get
- timezone_open
 [2011-04-29 13:25 UTC] salathe@php.net
Experimental support was added in 5.1.0 as noted on 
http://php.net/datetime.installation

Whether we document the functions as available from then, or from when the 
compiler flag was not longer necessary, is a good question. Currently, the 
versions are not consistent with one or the other.
 [2011-04-29 13:42 UTC] pierrick@php.net
As discussed with Peter, those functions will now be documented as PHP5 >= 5.2.0 
since they are aliases of DateTimeZone methods documented as PHP5 >= 5.2.0.

We will still keep the note in the date installation about the CFLAGS=-
DEXPERIMENTAL_DATE_SUPPORT=1 to enable those in 5.1.0
 [2020-02-07 06:08 UTC] phpdocbot@php.net
Automatic comment on behalf of pierrick
Revision: http://git.php.net/?p=doc/en.git;a=commit;h=e5f0d0ecb2f07ffde3ccb3b20e1b96108627183a
Log: Fixed bug #54630
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Thu May 02 04:01:30 2024 UTC