Attributes on class const / prop groups trigger a compile error.

Author: kooldev

Zend/tests/attributes/001_placement.phpt

@@ -7,10 +7,10 @@ Attributes can be placed on all supported elements.
 class Foo
 {
     <<A1(2)>>
-    public const FOO = 'foo', BAR = 'bar';
+    public const FOO = 'foo';
 
     <<A1(3)>>
-    public $x, $y;
+    public $x;
 
     <<A1(4)>>
     public function foo(<<A1(5)>> $a, <<A1(6)>> $b) { }
@@ -30,9 +30,7 @@ $ref = new \ReflectionClass(Foo::class);
 $sources = [
     $ref,
     $ref->getReflectionConstant('FOO'),
-    $ref->getReflectionConstant('BAR'),
     $ref->getProperty('x'),
-    $ref->getProperty('y'),
     $ref->getMethod('foo'),
     $ref->getMethod('foo')->getParameters()[0],
     $ref->getMethod('foo')->getParameters()[1],
@@ -71,9 +69,6 @@ array(1) {
   int(2)
 }
 
-string(23) "ReflectionClassConstant"
-int(0)
-
 string(18) "ReflectionProperty"
 int(1)
 string(2) "A1"
@@ -82,9 +77,6 @@ array(1) {
   int(3)
 }
 
-string(18) "ReflectionProperty"
-int(0)
-
 string(16) "ReflectionMethod"
 int(1)
 string(2) "A1"

Zend/tests/attributes/014_class_const_group.phpt

@@ -0,0 +1,14 @@
+--TEST--
+Attributes cannot be applied to groups of class constants.
+--FILE--
+<?php
+
+class C1
+{
+	<<A1>>
+	public const A = 1, B = 2;
+}
+
+?>
+--EXPECTF--
+Fatal error: Cannot apply attributes to a group of constants in %s

Zend/tests/attributes/015_property_group.phpt

@@ -0,0 +1,14 @@
+--TEST--
+Attributes cannot be applied to groups of properties.
+--FILE--
+<?php
+
+class C1
+{
+	<<A1>>
+	public $x, $y;
+}
+
+?>
+--EXPECTF--
+Fatal error: Cannot apply attributes to a group of properties in %s

Zend/zend_ast.c

@@ -1590,7 +1590,7 @@ static ZEND_COLD void zend_ast_export_ex(smart_str *str, zend_ast *ast, int prio
 		case ZEND_AST_CLASS_CONST_DECL:
 			smart_str_appends(str, "const ");
 			goto simple_list;
-		case ZEND_AST_CLASS_CONST_DECL_ATTRIBUTES:
+		case ZEND_AST_CLASS_CONST_GROUP:
 			if (ast->child[1]) {
 				zend_ast_export_attributes(str, ast->child[1], indent, 1);
 			}
@@ -2192,7 +2192,7 @@ zend_ast * ZEND_FASTCALL zend_ast_with_attributes(zend_ast *ast, zend_ast *attr)
 	case ZEND_AST_PARAM:
 		ast->child[3] = attr;
 		break;
-	case ZEND_AST_CLASS_CONST_DECL_ATTRIBUTES:
+	case ZEND_AST_CLASS_CONST_GROUP:
 		ast->child[1] = attr;
 		break;
 	EMPTY_SWITCH_DEFAULT_CASE()

Zend/zend_ast.h

@@ -139,7 +139,7 @@ enum _zend_ast_kind {
 	ZEND_AST_USE_ELEM,
 	ZEND_AST_TRAIT_ALIAS,
 	ZEND_AST_GROUP_USE,
-	ZEND_AST_CLASS_CONST_DECL_ATTRIBUTES,
+	ZEND_AST_CLASS_CONST_GROUP,
 	ZEND_AST_ATTRIBUTE,
 
 	/* 3 child nodes */

Zend/zend_compile.c

@@ -2367,7 +2367,7 @@ static inline zend_bool zend_is_variable_or_call(zend_ast *ast) /* {{{ */
 static inline zend_bool zend_is_unticked_stmt(zend_ast *ast) /* {{{ */
 {
 	return ast->kind == ZEND_AST_STMT_LIST || ast->kind == ZEND_AST_LABEL
-		|| ast->kind == ZEND_AST_PROP_DECL || ast->kind == ZEND_AST_CLASS_CONST_DECL_ATTRIBUTES
+		|| ast->kind == ZEND_AST_PROP_DECL || ast->kind == ZEND_AST_CLASS_CONST_GROUP
 		|| ast->kind == ZEND_AST_USE_TRAIT || ast->kind == ZEND_AST_METHOD;
 }
 /* }}} */
@@ -6520,8 +6520,6 @@ void zend_compile_prop_decl(zend_ast *ast, zend_ast *type_ast, uint32_t flags, H
 		}
 
 		zend_declare_typed_property(ce, name, &value_zv, flags, doc_comment, attributes, type);
-
-		attributes = NULL;
 	}
 }
 /* }}} */
@@ -6534,6 +6532,11 @@ void zend_compile_prop_group(zend_ast *list) /* {{{ */
 	zend_ast *prop_ast = list->child[1];
 
 	if (list->child[2]) {
+		if (zend_ast_get_list(prop_ast)->children > 1) {
+			zend_error_noreturn(E_COMPILE_ERROR, "Cannot apply attributes to a group of properties");
+			return;
+		}
+
 		attributes = create_attribute_array();
 		zend_compile_attributes(attributes, list->child[2], 0, ZEND_ATTRIBUTE_TARGET_PROPERTY);
 	}
@@ -6567,6 +6570,11 @@ void zend_compile_class_const_decl(zend_ast *ast, zend_ast *attr_ast) /* {{{ */
 	}
 
 	if (attr_ast) {
+		if (list->children > 1) {
+			zend_error_noreturn(E_COMPILE_ERROR, "Cannot apply attributes to a group of constants");
+			return;
+		}
+
 		attributes = create_attribute_array();
 		zend_compile_attributes(attributes, attr_ast, 0, ZEND_ATTRIBUTE_TARGET_CLASS_CONST);
 	}
@@ -6586,8 +6594,6 @@ void zend_compile_class_const_decl(zend_ast *ast, zend_ast *attr_ast) /* {{{ */
 
 		zend_const_expr_to_zval(&value_zv, value_ast);
 		zend_declare_class_constant_ex(ce, name, &value_zv, ast->attr, doc_comment, attributes);
-
-		attributes = NULL;
 	}
 }
 /* }}} */
@@ -8885,7 +8891,7 @@ void zend_compile_stmt(zend_ast *ast) /* {{{ */
 		case ZEND_AST_PROP_GROUP:
 			zend_compile_prop_group(ast);
 			break;
-		case ZEND_AST_CLASS_CONST_DECL_ATTRIBUTES:
+		case ZEND_AST_CLASS_CONST_GROUP:
 			zend_compile_class_const_decl(ast->child[0], ast->child[1]);
 			break;
 		case ZEND_AST_USE_TRAIT:

Zend/zend_language_parser.y

@@ -786,7 +786,7 @@ annotated_class_statement:
 			{ $$ = zend_ast_create(ZEND_AST_PROP_GROUP, $2, $3, NULL);
 			  $$->attr = $1; }
 	|	method_modifiers T_CONST class_const_list ';'
-			{ $$ = zend_ast_create(ZEND_AST_CLASS_CONST_DECL_ATTRIBUTES, $3, NULL); $3->attr = $1; }
+			{ $$ = zend_ast_create(ZEND_AST_CLASS_CONST_GROUP, $3, NULL); $3->attr = $1; }
 	|	method_modifiers function returns_ref identifier backup_doc_comment '(' parameter_list ')'
 		return_type backup_fn_flags method_body backup_fn_flags
 			{ $$ = zend_ast_create_decl(ZEND_AST_METHOD, $3 | $1 | $12, $2, $5,