From: Marcus Boerger <helly@php.net>
Date: Tue, 28 Oct 2003 18:46:37 +0000 (+0000)
Subject: Give some freedon to c iterators but not in userspace.
X-Git-Tag: RELEASE_2_0_0RC1~32
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=851a02e87f0a820a32eb02651cd80fa4efdc46d5;p=php

Give some freedon to c iterators but not in userspace.
---

diff --git a/Zend/zend_interfaces.c b/Zend/zend_interfaces.c
index 04cc74ea41..082653fedf 100755
--- a/Zend/zend_interfaces.c
+++ b/Zend/zend_interfaces.c
@@ -302,8 +302,14 @@ static int zend_implement_traversable(zend_class_entry *interface, zend_class_en
 /* {{{ zend_implement_aggregate */
 static int zend_implement_aggregate(zend_class_entry *interface, zend_class_entry *class_type TSRMLS_DC)
 {
-	if (class_type->get_iterator && class_type->get_iterator != zend_user_get_new_iterator) {
-		return FAILURE;
+	if (class_type->get_iterator) {
+		if (class_type->type == ZEND_INTERNAL_CLASS) {
+			/* inheritance ensures the class has necessary userland methods */
+			return SUCCESS;
+		} else if (class_type->get_iterator != zend_user_get_new_iterator) {
+			/* c-level get_iterator cannot be changed */
+			return FAILURE;
+		}
 	}
 	class_type->iterator_funcs.zf_new_iterator = NULL;
 	class_type->get_iterator = zend_user_get_new_iterator;
@@ -315,7 +321,13 @@ static int zend_implement_aggregate(zend_class_entry *interface, zend_class_entr
 static int zend_implement_iterator(zend_class_entry *interface, zend_class_entry *class_type TSRMLS_DC)
 {
 	if (class_type->get_iterator && class_type->get_iterator != zend_user_get_iterator) {
-		return FAILURE;
+		if (class_type->type == ZEND_INTERNAL_CLASS) {
+			/* inheritance ensures the class has the necessary userland methods */
+			return SUCCESS;
+		} else if (class_type->get_iterator != zend_user_get_new_iterator) {
+			/* c-level get_iterator cannot be changed */
+			return FAILURE;
+		}
 	}
 	class_type->get_iterator = zend_user_get_iterator;
 	class_type->iterator_funcs.zf_has_more = NULL;