summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--NEWS2
-rw-r--r--Zend/tests/bug70957.phpt22
-rw-r--r--Zend/zend_compile.c2
3 files changed, 25 insertions, 1 deletions
diff --git a/NEWS b/NEWS
index 189d9e0301..27c215b23a 100644
--- a/NEWS
+++ b/NEWS
@@ -3,6 +3,8 @@ PHP NEWS
?? ??? 2015, PHP 5.6.17
- Core:
+ . Fixed bug #70957 (self::class can not be resolved with reflection for
+ abstract class). (Laruence)
. Fixed bug #70944 (try{ } finally{} can create infinite chains of
exceptions). (Laruence)
diff --git a/Zend/tests/bug70957.phpt b/Zend/tests/bug70957.phpt
new file mode 100644
index 0000000000..e6a38f7641
--- /dev/null
+++ b/Zend/tests/bug70957.phpt
@@ -0,0 +1,22 @@
+--TEST--
+Bug #70957 (self::class can not be resolved with reflection for abstract class)
+--FILE--
+<?php
+
+abstract class Foo
+{
+ function bar($a = self::class) {}
+}
+
+trait T {
+ public function bar() {
+ }
+}
+
+class Bar extends Foo
+{
+ use T;
+}
+?>
+--EXPECTF--
+Strict Standards: Declaration of T::bar() should be compatible with Foo::bar($a = 'Foo') in %sbug70957.php on line %d
diff --git a/Zend/zend_compile.c b/Zend/zend_compile.c
index fca4e2a04d..03c2f5a31e 100644
--- a/Zend/zend_compile.c
+++ b/Zend/zend_compile.c
@@ -2214,7 +2214,7 @@ void zend_do_resolve_class_name(znode *result, znode *class_name, int is_static
if (!CG(active_class_entry)) {
zend_error_noreturn(E_COMPILE_ERROR, "Cannot access self::class when no class scope is active");
}
- if (CG(active_class_entry)->ce_flags & ZEND_ACC_TRAIT) {
+ if ((CG(active_class_entry)->ce_flags & ZEND_ACC_TRAIT) == ZEND_ACC_TRAIT) {
constant_name.op_type = IS_CONST;
ZVAL_STRINGL(&constant_name.u.constant, "class", sizeof("class")-1, 1);
zend_do_fetch_constant(result, class_name, &constant_name, ZEND_RT, 1 TSRMLS_CC);