php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Doc Bug #54747 Default value for touch() function
Submitted: 2011-05-16 16:33 UTC Modified: 2011-05-17 22:52 UTC
From: gwarnants at gmail dot com Assigned: salathe
Status: Closed Package: Documentation problem
PHP Version: Irrelevant OS:
Private report: No CVE-ID:
 [2011-05-16 16:33 UTC] gwarnants at gmail dot com
Description:
------------
---
From manual page: http://www.php.net/function.touch#Description
---

Third parameter $atime is indicated as optional. Details says that if not present it will be set to "time". So, prototype should be written accordingly :

bool touch ( string $filename [, int $time = time() [, int $atime = time() ]] )


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2011-05-17 22:52 UTC] salathe@php.net
Automatic comment from SVN on behalf of salathe
Revision: http://svn.php.net/viewvc/?view=revision&revision=311148
Log: clarify default value for atime parameter for touch() (doc #54747)
 [2011-05-17 22:52 UTC] salathe@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: salathe
 [2011-05-17 22:52 UTC] salathe@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.

The $atime parameter defaults to the value passed to the $time parameter, or the 
latter's default if that too is not passed.  If anything, that would make the 
prototype look like:

bool touch ( string $filename [, int $time = time() [, int $atime = $time ]] )

Which I fear would cause more confusion than omitting the default value.

I have however updated the description to be more clear about $atime using 
$time's value if present.
 
PHP Copyright © 2001-2017 The PHP Group
All rights reserved.
Last updated: Wed Aug 23 02:01:33 2017 UTC