php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Doc Bug #64570 Phar::extractTo() documentation page has phar.readonly=0 stated as requrement
Submitted: 2013-04-02 23:32 UTC Modified: 2016-06-25 15:42 UTC
From: goran dot radulovic at gmail dot com Assigned: cmb (profile)
Status: Closed Package: PHAR related
PHP Version: Irrelevant OS: MacOS X
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: goran dot radulovic at gmail dot com
New email:
PHP Version: OS:

 

 [2013-04-02 23:32 UTC] goran dot radulovic at gmail dot com
Description:
------------
---
From manual page: http://www.php.net/phar.extractto#refsect1-phar.extractto-
description
---

I don't see any particular reason for that requirement phar.readonly. I tried 
using this method with phar.readonly set to both 1 and 0 and in both cases it 
works perfectly.

As for functionality, when using this method original phar file is not affected 
and modified in any way, and for that reason phar.readonly shoud not be relevant.

Also on the same page, on method params section block. Description of the first 
metod parameter is 'Path within an archive to the file to delete.' which is also 
mistake. First parameter is destination directory, and extracing phar file to 
directory has nothing to do with deleting files.

This leads me to conclusion that this is some mistake in documentation. However i 
must be sure before heavily relay on this method in our application.

Thank you


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2013-04-02 23:49 UTC] goran dot radulovic at gmail dot com
By mistake, i wrote "phar.readonly = 1", when in fact i meant to write 
"phar.readonly = 0" in bug title.

Sorry
 [2016-06-25 15:28 UTC] cmb@php.net
-Status: Open +Status: Verified -Package: Documentation problem +Package: PHAR related -Assigned To: +Assigned To: cmb
 [2016-06-25 15:28 UTC] cmb@php.net
Indeed, that are ovbiously copy&paste errors.
 [2016-06-25 15:30 UTC] cmb@php.net
-Summary: Phar::extractTo() documentation page has phar.readonly = 1stated as requrement +Summary: Phar::extractTo() documentation page has phar.readonly=0 stated as requrement
 [2016-06-25 15:36 UTC] cmb@php.net
Automatic comment from SVN on behalf of cmb
Revision: http://svn.php.net/viewvc/?view=revision&revision=339493
Log: Fix #64570: Phar::extractTo() documentation page has phar.readonly=0 stated as requrement
 [2016-06-25 15:42 UTC] cmb@php.net
-Status: Verified +Status: Closed
 [2016-06-25 15:42 UTC] cmb@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.
 [2020-02-07 06:07 UTC] phpdocbot@php.net
Automatic comment on behalf of cmb
Revision: http://git.php.net/?p=doc/en.git;a=commit;h=e4725d85c5d955053a090d2e14fc5c5d446dc128
Log: Fix #64570: Phar::extractTo() documentation page has phar.readonly=0 stated as requrement
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Fri May 17 05:01:31 2024 UTC