Doc #71064 [Ana->Csd]: open($string, ZipArchive::OVERWRITE) does not create non-existing archives
Edit report at https://bugs.php.net/bug.php?id=71064&edit=1
ID: 71064
Updated by: cmb@php.net
Reported by: php at rachuth dot de
Summary: open($string, ZipArchive::OVERWRITE) does not create
non-existing archives
-Status: Analyzed
+Status: Closed
Type: Documentation Problem
Package: Zip Related
Operating System: *
PHP Version: 5.6.16
Assigned To: cmb
Block user comment: N
Private report: N
New Comment:
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.
Previous Comments:
------------------------------------------------------------------------
[2018-05-08 22:00:02] cmb@php.net
Automatic comment from SVN on behalf of cmb
Revision: http://svn.php.net/viewvc/?view=revision&revision=344995
Log: Fix #71064: open($string, ZipArchive::OVERWRITE) does not create non-existing archives
We're taking the wording from the zip_open() documentation[1], and don't
care to document that ZipArchive::OVERWRITE implied ZipArchive::CREATE
in some formerly bundled libzips.
[1] <https://libzip.org/documentation/zip_open.html>
------------------------------------------------------------------------
[2018-05-08 20:36:02] cmb@php.net
Since system libzip is actually preferred in the meantime[1], we
won't customize the bundled libzip anymore. Thus, I'm switching
to doc issue.
[1] <https://github.com/php/php-src/commit/8857161bf2936bb807ac99c991509821bf7f1100#diff-18e07cc7a4680f60b944b30302e4bfa9>
------------------------------------------------------------------------
[2016-09-05 17:13:55] cmb@php.net
Actually, the bundled libzip had been patched in the past due to
bug #47667, and this later has be re-done again[1]. This patch
causes different behavior with the bundled and a system libzip,
so, in my opinion, it is good that we don't patch libzip anymore.
I guess that this difference is the cause of the confusion
regarding several user notes[2].
Considering that the behavioral change is already around for
nearly 2 years (and always has been for system libzip), it might
be best to stick with the current behavior, and update the manual
appropriately.
[1] <https://github.com/php/php-src/commit/b8cdc731>
[2] <http://php.net/manual/en/ziparchive.open.php>
------------------------------------------------------------------------
[2015-12-17 05:44:08] pajoye@php.net
Thanks for the report.
Sorry to have missed this BC.
------------------------------------------------------------------------
[2015-12-16 19:34:23] jkeller at berkeley dot edu
workaround is to use ZipArchive::CREATE | ZipArchive::OVERWRITE instead of ZipArchive::OVERWRITE
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=71064
--
Edit this bug report at https://bugs.php.net/bug.php?id=71064&edit=1
Thread (4 messages)