From 1a7db40a1fba2f7d79f09b309bac6f957d70893f Mon Sep 17 00:00:00 2001 From: "Christoph M. Becker" Date: Thu, 23 Jul 2015 21:31:58 +0200 Subject: [PATCH] Fix #53854: Missing constants for compression type The constants have already been added long ago. This patch just adds a PHPT which checks the recognition of the respective compression methods. Unfortunately, I've not been able to assemble a zip with all compression methods. --- ext/zip/tests/compression_methods.phpt | 35 +++++++++++++++++++++++++ ext/zip/tests/compression_methods.zip | Bin 0 -> 2402 bytes 2 files changed, 35 insertions(+) create mode 100644 ext/zip/tests/compression_methods.phpt create mode 100644 ext/zip/tests/compression_methods.zip diff --git a/ext/zip/tests/compression_methods.phpt b/ext/zip/tests/compression_methods.phpt new file mode 100644 index 0000000000..084fa848f1 --- /dev/null +++ b/ext/zip/tests/compression_methods.phpt @@ -0,0 +1,35 @@ +--TEST-- +Recognition of compression methods +--DESCRIPTION-- +This test is supposed to cover all compression methods that are recognized by +libzip, but for now only 6 methods are available in compression_methods.zip. +The zip and the test should be extended, if possible. +--SKIPIF-- + +--FILE-- + ZipArchive::CM_STORE, + 'deflate' => ZipArchive::CM_DEFLATE, + 'deflate64' => ZipArchive::CM_DEFLATE64, + 'bzip2' => ZipArchive::CM_BZIP2, + 'lzma' => ZipArchive::CM_LZMA, + 'ppmd' => ZipArchive::CM_PPMD +); +$zip = new ZipArchive(); +$zip->open(__DIR__ . '/compression_methods.zip'); +foreach ($methods as $filename => $method) { + echo "$filename: "; + var_dump($zip->statName($filename)['comp_method'] === $method); +} +$zip->close(); +?> +--EXPECT-- +store: bool(true) +deflate: bool(true) +deflate64: bool(true) +bzip2: bool(true) +lzma: bool(true) +ppmd: bool(true) diff --git a/ext/zip/tests/compression_methods.zip b/ext/zip/tests/compression_methods.zip new file mode 100644 index 0000000000000000000000000000000000000000..44ed8ff5ed2ee214eee7232bed52002de954127d GIT binary patch literal 2402 zcmeHIX;c$e6n+Uo79+9*MKt6YQ{_yg^*4^xElYpS;9Kq@9CaUM%HiAyG7s|CyE<9|XE%2N}w~#ev z2bd^Uip;D7HwrJGbPCF%o$QgaIAK0Tp>;anJd<;4+=P{Rkts^Y#N=?M#=C3MBiE(g zcI*|Vo7OZ|^7XAG6ZORgiM4z0-ArPp7?r7O9X2d??r6WX$Bo0!tO~nadX9fW@SyG` z8-E+cFPO2hWXvJMw6CTjkFqNpj_J)G=kLE z0Q^ZccqCKipc*=~cWUU8qrw=ea=OPViHnnauw7tqvQx7N0GF(IH(kik2lLANp(8B{ zIOMrm0+62qBmfi^|{`5{)!+G|Hsy<@EUN>_Y6$0PYZ%UPE26Q{6Ra@h7{ zXvpYr`xBgGr*S-O^@5IrJ0zv63;Qzz8kj#G>rA+_=iY5jo3Ndo-juM--6PR>|ITH|0e%|YT;*p*o~(!Tb$BF~)%e_--dT|KIE zyE;Z!hb9$dJuAy}sMKhX)D8$nIzE#vPejFzWp41CUom#O z+G82-lr~uM5`}R--)=o^1NT}t)1I7-THm#+T-%8WgOs{go_8-=ZP)8)y);%n zVtbzHzUrFSJm2`}%+0aRtUC<5=dF1LdDi|VU2&F4aY*q_iHTG&$&!s)%&F7S@6yJ+s zzcG7E_-&q}bk7BHQQBOajU^9{q}wd6!*G3=*Wyj%w8FRpSa3>O(dRN|kB?gJOkDp@M2D4iOB+xlp8#t5F=q333n- zlWRm$`8tToq_~O;)u>@n34pM$|Iy=LXJZMhcZM+I)Nzu z&H)NBWSty=h)lYkh$aD06+${2#EVeK2qjJ+Jm`+Y2vntjs6tKcMnob)O|8da7(oUP z#RS=?n9+fG6oD#KNMJk|NCpi>WSj^LN7Q1tju4`%bt+WOC8H%hArMExr5Hlas}Z>f zC&>M391gDq?_cKr*XhHZ;rTD2(dUEV>Zk3%T?7EWw^TJXeBYx13y@QEt0b(dmiDow zPI^2)zW`5Ye5b-k zmQukS441Hm0(X1>p5^?h&@3=qsu&7=o$)?#424Y(y)#V2x+AEg9zp&_q=fURvH|`8 Domp{m literal 0 HcmV?d00001 -- 2.49.0