summaryrefslogtreecommitdiff
path: root/ext/dom
diff options
context:
space:
mode:
authorNikita Popov <nikita.ppv@gmail.com>2020-04-28 15:10:23 +0200
committerNikita Popov <nikita.ppv@gmail.com>2020-04-28 15:10:23 +0200
commitbbda71b63d5dd36ba4e6e183fce4e4eab154b3e2 (patch)
tree91875bd82f33fccaeecf4f604c5c17c9a02474af /ext/dom
parent7d1c8064912da100be0106774b9ba08ec1a88ee6 (diff)
parent75470bc312ef2e151696ea5c1ee026c089944623 (diff)
downloadphp-git-bbda71b63d5dd36ba4e6e183fce4e4eab154b3e2.tar.gz
Merge branch 'PHP-7.4'
* PHP-7.4: Revert "Fix #79065: DOM classes do not expose properties to Reflection"
Diffstat (limited to 'ext/dom')
-rw-r--r--ext/dom/php_dom.c23
-rw-r--r--ext/dom/tests/bug79065.phpt30
2 files changed, 0 insertions, 53 deletions
diff --git a/ext/dom/php_dom.c b/ext/dom/php_dom.c
index 67961b303a..c7b2103a0f 100644
--- a/ext/dom/php_dom.c
+++ b/ext/dom/php_dom.c
@@ -385,28 +385,6 @@ static int dom_property_exists(zend_object *object, zend_string *name, int check
}
/* }}} */
-/* {{{ dom_get_properties */
-static HashTable *dom_get_properties(zend_object *object)
-{
- dom_object *obj = php_dom_obj_from_obj(object);
- HashTable *props = zend_std_get_properties(object);
-
- if (obj->prop_handler != NULL) {
- zend_string *key;
- dom_prop_handler *hnd;
-
- ZEND_HASH_FOREACH_STR_KEY_PTR(obj->prop_handler, key, hnd) {
- zval val;
-
- if (hnd->read_func(obj, &val) == SUCCESS) {
- zend_hash_update(props, key, &val);
- }
- } ZEND_HASH_FOREACH_END();
- }
- return props;
-}
-/* }}} */
-
static HashTable* dom_get_debug_info_helper(zend_object *object, int *is_temp) /* {{{ */
{
dom_object *obj = php_dom_obj_from_obj(object);
@@ -590,7 +568,6 @@ PHP_MINIT_FUNCTION(dom)
dom_object_handlers.get_property_ptr_ptr = dom_get_property_ptr_ptr;
dom_object_handlers.clone_obj = dom_objects_store_clone_obj;
dom_object_handlers.has_property = dom_property_exists;
- dom_object_handlers.get_properties = dom_get_properties;
dom_object_handlers.get_debug_info = dom_get_debug_info;
memcpy(&dom_nnodemap_object_handlers, &dom_object_handlers, sizeof(zend_object_handlers));
diff --git a/ext/dom/tests/bug79065.phpt b/ext/dom/tests/bug79065.phpt
deleted file mode 100644
index 5b92d5c749..0000000000
--- a/ext/dom/tests/bug79065.phpt
+++ /dev/null
@@ -1,30 +0,0 @@
---TEST--
-Bug #79065 (DOM classes do not expose properties to Reflection)
---SKIPIF--
-<?php
-if (!extension_loaded('dom')) die('skip dom extension not available');
-?>
---FILE--
-<?php
-$dom = new DOMDocument;
-$dom->loadHTML('<b>test</b>');
-var_dump(count(get_object_vars($dom)));
-
-$ro = new ReflectionObject($dom);
-var_dump(count($ro->getProperties()));
-var_dump($ro->hasProperty("textContent"));
-$rp = $ro->getProperty("textContent");
-var_dump($rp);
-var_dump($rp->getValue($dom));
-?>
---EXPECTF--
-int(38)
-int(38)
-bool(true)
-object(ReflectionProperty)#%d (2) {
- ["name"]=>
- string(11) "textContent"
- ["class"]=>
- string(11) "DOMDocument"
-}
-string(4) "test"