From d9eeb2e6e659486ab9d2d15b720714b1ef5503dd Mon Sep 17 00:00:00 2001 From: Sebastian Bergmann Date: Thu, 3 Sep 2009 14:02:51 +0000 Subject: [PATCH] Make invokeArgs() error messages consistent with invoke(). --- ext/reflection/php_reflection.c | 6 +++--- .../tests/ReflectionMethod_invokeArgs_error3.phpt | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/ext/reflection/php_reflection.c b/ext/reflection/php_reflection.c index 6374eaa3a1..ba165dbd1e 100644 --- a/ext/reflection/php_reflection.c +++ b/ext/reflection/php_reflection.c @@ -2422,11 +2422,11 @@ ZEND_METHOD(reflection_method, invokeArgs) (mptr->common.fn_flags & ZEND_ACC_ABSTRACT)) { if (mptr->common.fn_flags & ZEND_ACC_ABSTRACT) { zend_throw_exception_ex(reflection_exception_ptr, 0 TSRMLS_CC, - "Trying to invoke abstract method %s::%s", + "Trying to invoke abstract method %s::%s()", mptr->common.scope->name, mptr->common.function_name); } else { zend_throw_exception_ex(reflection_exception_ptr, 0 TSRMLS_CC, - "Trying to invoke %s method %s::%s from scope %s", + "Trying to invoke %s method %s::%s() from scope %s", mptr->common.fn_flags & ZEND_ACC_PROTECTED ? "protected" : "private", mptr->common.scope->name, mptr->common.function_name, Z_OBJCE_P(getThis())->name); @@ -2453,7 +2453,7 @@ ZEND_METHOD(reflection_method, invokeArgs) if (!object) { efree(params); zend_throw_exception_ex(reflection_exception_ptr, 0 TSRMLS_CC, - "Trying to invoke non static method %s::%s without an object", + "Trying to invoke non static method %s::%s() without an object", mptr->common.scope->name, mptr->common.function_name); return; } diff --git a/ext/reflection/tests/ReflectionMethod_invokeArgs_error3.phpt b/ext/reflection/tests/ReflectionMethod_invokeArgs_error3.phpt index 834f35d668..f39dd46819 100644 --- a/ext/reflection/tests/ReflectionMethod_invokeArgs_error3.phpt +++ b/ext/reflection/tests/ReflectionMethod_invokeArgs_error3.phpt @@ -111,9 +111,9 @@ NULL NULL Private method: -string(84) "Trying to invoke private method TestClass::privateMethod from scope ReflectionMethod" +string(86) "Trying to invoke private method TestClass::privateMethod() from scope ReflectionMethod" Abstract method: -string(51) "Trying to invoke abstract method AbstractClass::foo" +string(53) "Trying to invoke abstract method AbstractClass::foo()" Warning: ReflectionMethod::invokeArgs() expects exactly 2 parameters, 1 given in %s on line %d -- 2.50.1