Skip to content

Commit 3bd51bd

Browse files
dantleechdantleech
committed
Merge branch 'constant-types'
2 parents bdc7009 + d31c085 commit 3bd51bd

23 files changed

+297
-7
lines changed

‎.github/workflows/main.yml‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,8 @@ jobs:
3131
- PHP_VERSION: '7.4'
3232
- PHP_VERSION: '8.0'
3333
- PHP_VERSION: '8.1'
34-
- PHP_VERSION: '8.2.0beta2'
34+
- PHP_VERSION: '8.2'
35+
- PHP_VERSION: '8.3'
3536

3637
# Steps represent a sequence of tasks that will be executed as part of the job
3738
steps:

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
# Tolerant PHP Parser
2-
[![Build Status](https://travis-ci.org/Microsoft/tolerant-php-parser.svg?branch=master)](https://travis-ci.org/Microsoft/tolerant-php-parser)
2+
[![CI](https://github.com/microsoft/tolerant-php-parser/actions/workflows/main.yml/badge.svg?branch=main)](https://github.com/microsoft/tolerant-php-parser/actions/workflows/main.yml)
33

44
This is an early-stage PHP parser designed, from the beginning, for IDE usage scenarios (see [Design Goals](#design-goals) for more details). There is
55
still a ton of work to be done, so at this point, this repo mostly serves as

‎src/Node/ClassConstDeclaration.php‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
use Microsoft\PhpParser\ModifiedTypeInterface;
1010
use Microsoft\PhpParser\ModifiedTypeTrait;
1111
use Microsoft\PhpParser\Node;
12+
use Microsoft\PhpParser\Node\DelimitedList\QualifiedNameList;
1213
use Microsoft\PhpParser\Token;
1314

1415
class ClassConstDeclaration extends Node implements ModifiedTypeInterface {
@@ -23,6 +24,9 @@ class ClassConstDeclaration extends Node implements ModifiedTypeInterface {
2324
/** @var Token */
2425
public $constKeyword;
2526

27+
/** @var QualifiedNameList|null */
28+
public $typeDeclarationList;
29+
2630
/** @var DelimitedList\ConstElementList */
2731
public $constElements;
2832

@@ -33,6 +37,7 @@ class ClassConstDeclaration extends Node implements ModifiedTypeInterface {
3337
'attributes',
3438
'modifiers',
3539
'constKeyword',
40+
'typeDeclarationList',
3641
'constElements',
3742
'semicolon'
3843
];

‎src/Parser.php‎

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1419,7 +1419,7 @@ private function parseStringLiteralExpression2($parentNode): StringLiteral {
14191419
case TokenKind::DollarOpenBraceToken:
14201420
case TokenKind::OpenBraceDollarToken:
14211421
$expression->children[] = $this->eat(TokenKind::DollarOpenBraceToken, TokenKind::OpenBraceDollarToken);
1422-
/**
1422+
/**
14231423
* @phpstan-ignore-next-line "Strict comparison using
14241424
* === between 403|404 and 408 will always evaluate to
14251425
* false" is wrong because those tokens were eaten above
@@ -2804,10 +2804,6 @@ private function parseListIntrinsicExpression($parentNode) {
28042804
return $listExpression;
28052805
}
28062806

2807-
private function isArrayElementStart($token) {
2808-
return ($this->isArrayElementStartFn())($token);
2809-
}
2810-
28112807
private function isArrayElementStartFn() {
28122808
return function ($token) {
28132809
return $token->kind === TokenKind::AmpersandToken || $token->kind === TokenKind::DotDotDotToken || $this->isExpressionStart($token);
@@ -3372,6 +3368,16 @@ private function parseClassConstDeclaration($parentNode, $modifiers) {
33723368

33733369
$classConstDeclaration->modifiers = $modifiers;
33743370
$classConstDeclaration->constKeyword = $this->eat1(TokenKind::ConstKeyword);
3371+
// Handle class constant declarations such as `const X|Y Z = 123;` or `const X = 123;`.
3372+
// This is similar to lookahead().
3373+
$startPos = $this->lexer->getCurrentPosition();
3374+
$startToken = $this->token;
3375+
$classConstDeclaration->typeDeclarationList = $this->tryParseParameterTypeDeclarationList($classConstDeclaration);
3376+
if (in_array($this->token->kind, [TokenKind::EqualsToken, TokenKind::CommaToken, TokenKind::SemicolonToken]) && $this->lexer->getCurrentPosition() <= $startPos + 1) {
3377+
$classConstDeclaration->typeDeclarationList = null;
3378+
$this->lexer->setCurrentPosition($startPos);
3379+
$this->token = $startToken;
3380+
}
33753381
$classConstDeclaration->constElements = $this->parseConstElements($classConstDeclaration);
33763382
$classConstDeclaration->semicolon = $this->eat1(TokenKind::SemicolonToken);
33773383

‎tests/cases/parser/classConstDeclaration1.php.tree‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@
4141
"kind": "ConstKeyword",
4242
"textLength": 5
4343
},
44+
"typeDeclarationList": null,
4445
"constElements": {
4546
"ConstElementList": {
4647
"children": [

‎tests/cases/parser/classConstDeclaration10.php.tree‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,7 @@
4646
"kind": "ConstKeyword",
4747
"textLength": 5
4848
},
49+
"typeDeclarationList": null,
4950
"constElements": {
5051
"ConstElementList": {
5152
"children": [

‎tests/cases/parser/classConstDeclaration11.php‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
<?php
2+
class A {
3+
const int|string a = 3;
4+
}

‎tests/cases/parser/classConstDeclaration11.php.diag‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
[]

‎tests/cases/parser/classConstDeclaration11.php.tree‎

Lines changed: 109 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,109 @@
1+
{
2+
"SourceFileNode": {
3+
"statementList": [
4+
{
5+
"InlineHtml": {
6+
"scriptSectionEndTag": null,
7+
"text": null,
8+
"scriptSectionStartTag": {
9+
"kind": "ScriptSectionStartTag",
10+
"textLength": 6
11+
}
12+
}
13+
},
14+
{
15+
"ClassDeclaration": {
16+
"attributes": null,
17+
"abstractOrFinalModifier": null,
18+
"modifiers": [],
19+
"classKeyword": {
20+
"kind": "ClassKeyword",
21+
"textLength": 5
22+
},
23+
"name": {
24+
"kind": "Name",
25+
"textLength": 1
26+
},
27+
"classBaseClause": null,
28+
"classInterfaceClause": null,
29+
"classMembers": {
30+
"ClassMembersNode": {
31+
"openBrace": {
32+
"kind": "OpenBraceToken",
33+
"textLength": 1
34+
},
35+
"classMemberDeclarations": [
36+
{
37+
"ClassConstDeclaration": {
38+
"attributes": null,
39+
"modifiers": [],
40+
"constKeyword": {
41+
"kind": "ConstKeyword",
42+
"textLength": 5
43+
},
44+
"typeDeclarationList": {
45+
"QualifiedNameList": {
46+
"children": [
47+
{
48+
"kind": "IntReservedWord",
49+
"textLength": 3
50+
},
51+
{
52+
"kind": "BarToken",
53+
"textLength": 1
54+
},
55+
{
56+
"kind": "StringReservedWord",
57+
"textLength": 6
58+
}
59+
]
60+
}
61+
},
62+
"constElements": {
63+
"ConstElementList": {
64+
"children": [
65+
{
66+
"ConstElement": {
67+
"name": {
68+
"kind": "Name",
69+
"textLength": 1
70+
},
71+
"equalsToken": {
72+
"kind": "EqualsToken",
73+
"textLength": 1
74+
},
75+
"assignment": {
76+
"NumericLiteral": {
77+
"children": {
78+
"kind": "IntegerLiteralToken",
79+
"textLength": 1
80+
}
81+
}
82+
}
83+
}
84+
}
85+
]
86+
}
87+
},
88+
"semicolon": {
89+
"kind": "SemicolonToken",
90+
"textLength": 1
91+
}
92+
}
93+
}
94+
],
95+
"closeBrace": {
96+
"kind": "CloseBraceToken",
97+
"textLength": 1
98+
}
99+
}
100+
}
101+
}
102+
}
103+
],
104+
"endOfFileToken": {
105+
"kind": "EndOfFileToken",
106+
"textLength": 0
107+
}
108+
}
109+
}

‎tests/cases/parser/classConstDeclaration12.php‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
<?php
2+
class A {
3+
const int| a = 3;
4+
}

0 commit comments

Comments
 (0)