MFH: - Fix isset/empty($ZipArchive->property)

MFH: - Add test

diff --git a/ext/zip/php_zip.c b/ext/zip/php_zip.c
index ec25cc6efc9b2cfb1f1615e77d5637e0f16fe657..59336158dee8bdcf640e20f14c612928dc60e9fc 100644 (file)
--- a/ext/zip/php_zip.c
+++ b/ext/zip/php_zip.c
@@ -449,6 +449,55 @@ static zval* php_zip_read_property(zval *object, zval *member, int type TSRMLS_D
 }
 /* }}} */
 
+static int php_zip_has_property(zval *object, zval *member, int type TSRMLS_DC) /* {{{ */
+{
+       ze_zip_object *obj;
+       zval tmp_member;
+       zip_prop_handler *hnd;
+       zend_object_handlers *std_hnd;
+       int ret, retval = 0;
+
+       if (member->type != IS_STRING) {
+               tmp_member = *member;
+               zval_copy_ctor(&tmp_member);
+               convert_to_string(&tmp_member);
+               member = &tmp_member;
+       }
+
+       ret = FAILURE;
+       obj = (ze_zip_object *)zend_objects_get_address(object TSRMLS_CC);
+
+       if (obj->prop_handler != NULL) {
+               ret = zend_hash_find(obj->prop_handler, Z_STRVAL_P(member), Z_STRLEN_P(member)+1, (void **) &hnd);
+       }
+
+       if (ret == SUCCESS) {
+               zval *tmp;
+
+               if (type == 2) {
+                       retval = 1;
+               } else if (php_zip_property_reader(obj, hnd, &tmp, 1 TSRMLS_CC) == SUCCESS) {
+                       tmp->refcount = 1;
+                       tmp->is_ref = 0;
+                       if (type == 1) {
+                               retval = zend_is_true(tmp);
+                       } else if (type == 0) {
+                               retval = (Z_TYPE_P(tmp) != IS_NULL);
+                       }
+                       zval_ptr_dtor(&tmp);
+               }
+       } else {
+               std_hnd = zend_get_std_object_handlers();
+               retval = std_hnd->has_property(object, member, type TSRMLS_CC);
+       }
+
+       if (member == &tmp_member) {
+               zval_dtor(member);
+       }
+       return retval;
+}
+/* }}} */
+
 static HashTable *php_zip_get_properties(zval *object TSRMLS_DC)/* {{{ */
 {
        ze_zip_object *obj;
@@ -1965,6 +2014,7 @@ static PHP_MINIT_FUNCTION(zip)
 
        zip_object_handlers.get_properties = php_zip_get_properties;
        zip_object_handlers.read_property       = php_zip_read_property;
+       zip_object_handlers.has_property        = php_zip_has_property;
 
        INIT_CLASS_ENTRY(ce, "ZipArchive", zip_class_functions);
        ce.create_object = php_zip_object_new;

diff --git a/ext/zip/tests/oo_properties.phpt b/ext/zip/tests/oo_properties.phpt
new file mode 100644 (file)
index 0000000..cc7ceca
--- /dev/null
+++ b/ext/zip/tests/oo_properties.phpt
@@ -0,0 +1,59 @@
+--TEST--
+ziparchive::properties isset()/empty() checks
+--SKIPIF--
+<?php
+/* $Id$ */
+if(!extension_loaded('zip')) die('skip');
+?>
+--FILE--
+<?php
+
+$dirname = dirname(__FILE__) . '/';
+$file = $dirname . '__property_test.zip';
+
+copy($dirname . 'test_with_comment.zip', $file);
+
+$zip = new ZipArchive;
+if (!$zip->open($file)) {
+       exit('failed');
+}
+
+printf("zip->status (%d):\n\tempty(): %d\n\tisset(): %d\n", $zip->status, empty($zip->status), isset($zip->status));
+printf("zip->numFiles (%d):\n\tempty(): %d\n\tisset(): %d\n", $zip->numFiles, empty($zip->numFiles), isset($zip->numFiles));
+printf("zip->bogus (%d):\n\tempty(): %d\n\tisset(): %d\n", $zip->bogus, empty($zip->bogus), isset($zip->bogus));
+
+
+$zip->addEmptyDir('emptydir');
+
+printf("zip->status (%d):\n\tempty(): %d\n\tisset(): %d\n", $zip->status, empty($zip->status), isset($zip->status));
+printf("zip->numFiles (%d):\n\tempty(): %d\n\tisset(): %d\n", $zip->numFiles, empty($zip->numFiles), isset($zip->numFiles));
+printf("zip->filename (%d):\n\tempty(): %d\n\tisset(): %d\n", strlen($zip->filename), empty($zip->filename), isset($zip->filename));
+printf("zip->comment (%d):\n\tempty(): %d\n\tisset(): %d\n", strlen($zip->comment), empty($zip->comment), isset($zip->comment));
+
+var_dump(unlink($file));
+?>
+--EXPECTF--
+zip->status (0):
+       empty(): 1
+       isset(): 1
+zip->numFiles (4):
+       empty(): 0
+       isset(): 1
+
+Notice: Undefined property:  ZipArchive::$bogus in %s on line %d
+zip->bogus (0):
+       empty(): 1
+       isset(): 0
+zip->status (0):
+       empty(): 1
+       isset(): 1
+zip->numFiles (5):
+       empty(): 0
+       isset(): 1
+zip->filename (0):
+       empty(): 1
+       isset(): 1
+zip->comment (19):
+       empty(): 0
+       isset(): 1
+