From: Marcus Boerger <helly@php.net>
Date: Sat, 22 Jan 2005 12:29:13 +0000 (+0000)
Subject: - MFH bugfix #31651
X-Git-Tag: php-5.0.4RC1~276
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ef9d20d63817fc9eb6382c3a2af9abe8a40dbe47;p=php

- MFH bugfix #31651
---

diff --git a/Zend/zend_API.c b/Zend/zend_API.c
index 09feb17fcb..08032604b1 100644
--- a/Zend/zend_API.c
+++ b/Zend/zend_API.c
@@ -687,6 +687,16 @@ ZEND_API void zend_merge_properties(zval *obj, HashTable *properties, int destro
 }
 
 
+ZEND_API void zend_update_class_constants(zend_class_entry *class_type TSRMLS_DC)
+{
+	if (!class_type->constants_updated) {
+		zend_hash_apply_with_argument(&class_type->default_properties, (apply_func_arg_t) zval_update_constant, (void *) 1 TSRMLS_CC);
+		zend_hash_apply_with_argument(class_type->static_members, (apply_func_arg_t) zval_update_constant, (void *) 1 TSRMLS_CC);
+		class_type->constants_updated = 1;
+	}
+}
+
+
 /* This function requires 'properties' to contain all props declared in the
  * class and all props being public. If only a subset is given or the class 
  * has protected members then you need to merge the properties seperately by
@@ -701,11 +711,7 @@ ZEND_API int _object_and_properties_init(zval *arg, zend_class_entry *class_type
 		zend_error(E_ERROR, "Cannot instantiate %s %s", what, class_type->name);
 	}
 
-	if (!class_type->constants_updated) {
-		zend_hash_apply_with_argument(&class_type->default_properties, (apply_func_arg_t) zval_update_constant, (void *) 1 TSRMLS_CC);
-		zend_hash_apply_with_argument(class_type->static_members, (apply_func_arg_t) zval_update_constant, (void *) 1 TSRMLS_CC);
-		class_type->constants_updated = 1;
-	}
+	zend_update_class_constants(class_type TSRMLS_CC);
 	
 	arg->type = IS_OBJECT;
 	if (class_type->create_object == NULL) {
diff --git a/Zend/zend_API.h b/Zend/zend_API.h
index 6a4754ed21..b8bb18b7c4 100644
--- a/Zend/zend_API.h
+++ b/Zend/zend_API.h
@@ -188,6 +188,7 @@ ZEND_API int zend_declare_property_null(zend_class_entry *ce, char *name, int na
 ZEND_API int zend_declare_property_long(zend_class_entry *ce, char *name, int name_length, long value, int access_type TSRMLS_DC);
 ZEND_API int zend_declare_property_string(zend_class_entry *ce, char *name, int name_length, char *value, int access_type TSRMLS_DC);
 
+ZEND_API void zend_update_class_constants(zend_class_entry *class_type TSRMLS_DC);
 ZEND_API void zend_update_property(zend_class_entry *scope, zval *object, char *name, int name_length, zval *value TSRMLS_DC);
 ZEND_API void zend_update_property_null(zend_class_entry *scope, zval *object, char *name, int name_length TSRMLS_DC);
 ZEND_API void zend_update_property_long(zend_class_entry *scope, zval *object, char *name, int name_length, long value TSRMLS_DC);
diff --git a/Zend/zend_reflection_api.c b/Zend/zend_reflection_api.c
index a2d0e618b5..18c790670f 100644
--- a/Zend/zend_reflection_api.c
+++ b/Zend/zend_reflection_api.c
@@ -2128,11 +2128,7 @@ ZEND_METHOD(reflection_class, getStaticProperties)
 	METHOD_NOTSTATIC_NUMPARAMS(0);	
 	GET_REFLECTION_OBJECT_PTR(ce);
 
-	if (!ce->constants_updated) {
-		zend_hash_apply_with_argument(&ce->default_properties, (apply_func_arg_t) zval_update_constant, (void *) 1 TSRMLS_CC);
-		zend_hash_apply_with_argument(ce->static_members, (apply_func_arg_t) zval_update_constant, (void *) 1 TSRMLS_CC);
-		ce->constants_updated = 1;
-	}
+	zend_update_class_constants(ce TSRMLS_CC);
 
 	array_init(return_value);
 	zend_hash_copy(Z_ARRVAL_P(return_value), ce->static_members, (copy_ctor_func_t) zval_add_ref, (void *) &tmp_copy, sizeof(zval *));
@@ -2151,6 +2147,8 @@ ZEND_METHOD(reflection_class, getDefaultProperties)
 	GET_REFLECTION_OBJECT_PTR(ce);
 	array_init(return_value);
 
+	zend_update_class_constants(ce TSRMLS_CC);
+
 	count = zend_hash_num_elements(&ce->default_properties);
 	if (count > 0) {
 		HashPosition pos;
diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c
index a2d0e618b5..18c790670f 100644
--- a/ext/reflection/php_reflection.c
+++ b/ext/reflection/php_reflection.c
@@ -2128,11 +2128,7 @@ ZEND_METHOD(reflection_class, getStaticProperties)
 	METHOD_NOTSTATIC_NUMPARAMS(0);	
 	GET_REFLECTION_OBJECT_PTR(ce);
 
-	if (!ce->constants_updated) {
-		zend_hash_apply_with_argument(&ce->default_properties, (apply_func_arg_t) zval_update_constant, (void *) 1 TSRMLS_CC);
-		zend_hash_apply_with_argument(ce->static_members, (apply_func_arg_t) zval_update_constant, (void *) 1 TSRMLS_CC);
-		ce->constants_updated = 1;
-	}
+	zend_update_class_constants(ce TSRMLS_CC);
 
 	array_init(return_value);
 	zend_hash_copy(Z_ARRVAL_P(return_value), ce->static_members, (copy_ctor_func_t) zval_add_ref, (void *) &tmp_copy, sizeof(zval *));
@@ -2151,6 +2147,8 @@ ZEND_METHOD(reflection_class, getDefaultProperties)
 	GET_REFLECTION_OBJECT_PTR(ce);
 	array_init(return_value);
 
+	zend_update_class_constants(ce TSRMLS_CC);
+
 	count = zend_hash_num_elements(&ce->default_properties);
 	if (count > 0) {
 		HashPosition pos;