Fixed bug GH-8943 Reflection::getModifiersNames() with readonly modifier

Author: adoy

ext/reflection/php_reflection.c

@@ -1550,6 +1550,10 @@ ZEND_METHOD(Reflection, getModifierNames)
 	if (modifiers & ZEND_ACC_STATIC) {
 		add_next_index_stringl(return_value, "static", sizeof("static")-1);
 	}
+
+	if (modifiers & ZEND_ACC_READONLY) {
+		add_next_index_stringl(return_value, "readonly", sizeof("readonly")-1);
+	}
 }
 /* }}} */
 

ext/reflection/tests/Reflection_getModifierNames_001.phpt

@@ -0,0 +1,23 @@
+--TEST--
+Reflection::getModifierNames() basic
+--FILE--
+<?php
+
+function printModifiers($modifiers) {
+	echo implode(',', Reflection::getModifierNames($modifiers)), PHP_EOL;
+}
+
+printModifiers(ReflectionProperty::IS_PRIVATE);
+printModifiers(ReflectionProperty::IS_PROTECTED | ReflectionProperty::IS_STATIC);
+printModifiers(ReflectionProperty::IS_PUBLIC | ReflectionProperty::IS_READONLY);
+printModifiers(ReflectionClass::IS_EXPLICIT_ABSTRACT);
+printModifiers(ReflectionMethod::IS_ABSTRACT | ReflectionMethod::IS_FINAL);
+printModifiers(ReflectionProperty::IS_PUBLIC | ReflectionProperty::IS_STATIC | ReflectionProperty::IS_READONLY);
+?>
+--EXPECT--
+private
+protected,static
+public,readonly
+abstract
+abstract,final
+public,static,readonly