From: Johannes Schlüter <johannes@php.net>
Date: Tue, 31 Jul 2007 22:47:25 +0000 (+0000)
Subject: - MFH: Add additional param to get_loaded_extensions() for returning Zend
X-Git-Tag: php-5.2.4RC1~19
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=ebaba407742e01902a30d1fd00209d90e3d5daa5;p=php

- MFH: Add additional param to get_loaded_extensions() for returning Zend
  extensions (Fixes #41278) [DOC]
---

diff --git a/Zend/tests/017.phpt b/Zend/tests/017.phpt
index e8a5cfe857..bdf2ea7273 100644
--- a/Zend/tests/017.phpt
+++ b/Zend/tests/017.phpt
@@ -10,9 +10,9 @@ var_dump(get_resource_type($fp));
 fclose($fp);
 var_dump(get_resource_type($fp));
 
-var_dump(get_loaded_extensions(true));
 var_dump(gettype(get_loaded_extensions()));
 var_dump(count(get_loaded_extensions()));
+var_dump(count(get_loaded_extensions(true)));
 
 define("USER_CONSTANT", "test");
 
@@ -50,11 +50,9 @@ Warning: Supplied argument is not a valid resource handle in %s on line %d
 bool(false)
 string(6) "stream"
 string(7) "Unknown"
-
-Warning: Wrong parameter count for get_loaded_extensions() in %s on line %d
-NULL
 string(5) "array"
 int(%d)
+int(2)
 
 Warning: Wrong parameter count for get_defined_constants() in %s on line %d
 NULL
diff --git a/Zend/zend_builtin_functions.c b/Zend/zend_builtin_functions.c
index 0d45d5a32e..e4fd10226f 100644
--- a/Zend/zend_builtin_functions.c
+++ b/Zend/zend_builtin_functions.c
@@ -25,6 +25,7 @@
 #include "zend_constants.h"
 #include "zend_ini.h"
 #include "zend_exceptions.h"
+#include "zend_extensions.h"
 
 #undef ZEND_TEST_EXCEPTIONS
 
@@ -1548,6 +1549,13 @@ static int add_extension_info(zend_module_entry *module, void *arg TSRMLS_DC)
 	return 0;
 }
 
+static int add_zendext_info(zend_extension *ext, void *arg TSRMLS_DC)
+{
+	zval *name_array = (zval *)arg;
+	add_next_index_string(name_array, ext->name, 1);
+	return 0;
+}
+
 static int add_constant_info(zend_constant *constant, void *arg TSRMLS_DC)
 {
 	zval *name_array = (zval *)arg;
@@ -1562,16 +1570,34 @@ static int add_constant_info(zend_constant *constant, void *arg TSRMLS_DC)
 }
 
 
-/* {{{ proto array get_loaded_extensions(void)
+/* {{{ proto array get_loaded_extensions([mixed categorize]) U
    Return an array containing names of loaded extensions */
 ZEND_FUNCTION(get_loaded_extensions)
 {
-	if (ZEND_NUM_ARGS() != 0) {
+	int argc = ZEND_NUM_ARGS();
+
+	if (argc != 0 && argc != 1) {
 		ZEND_WRONG_PARAM_COUNT();
 	}
 
 	array_init(return_value);
-	zend_hash_apply_with_argument(&module_registry, (apply_func_arg_t) add_extension_info, return_value TSRMLS_CC);
+
+	if (argc) {
+		zval *modules;
+		zval *extensions;
+
+		MAKE_STD_ZVAL(modules);
+		array_init(modules);
+		zend_hash_apply_with_argument(&module_registry, (apply_func_arg_t) add_extension_info, modules TSRMLS_CC);
+		add_assoc_zval_ex(return_value, "PHP Modules", sizeof("PHP Modules"), modules);
+
+		MAKE_STD_ZVAL(extensions);
+		array_init(extensions);
+		zend_llist_apply_with_argument(&zend_extensions, (llist_apply_with_arg_func_t) add_zendext_info, extensions TSRMLS_CC);
+		add_assoc_zval_ex(return_value, "Zend Extensions", sizeof("Zend Extensions"), extensions);
+	} else {
+		zend_hash_apply_with_argument(&module_registry, (apply_func_arg_t) add_extension_info, return_value TSRMLS_CC);
+	}
 }
 /* }}} */