Skip to content

Commit 9d58172

Browse files
committed
PHP 8.0 | Squiz/OperatorBracket: allow for nullsafe object operator
Includes unit test.
1 parent 85f3521 commit 9d58172

File tree

4 files changed

+7
-0
lines changed

4 files changed

+7
-0
lines changed

src/Standards/Squiz/Sniffs/Formatting/OperatorBracketSniff.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,7 @@ public function process(File $phpcsFile, $stackPtr)
140140
T_SELF,
141141
T_STATIC,
142142
T_OBJECT_OPERATOR,
143+
T_NULLSAFE_OBJECT_OPERATOR,
143144
T_DOUBLE_COLON,
144145
T_OPEN_SQUARE_BRACKET,
145146
T_CLOSE_SQUARE_BRACKET,
@@ -283,6 +284,7 @@ public function addMissingBracketsError($phpcsFile, $stackPtr)
283284
T_SELF => true,
284285
T_STATIC => true,
285286
T_OBJECT_OPERATOR => true,
287+
T_NULLSAFE_OBJECT_OPERATOR => true,
286288
T_DOUBLE_COLON => true,
287289
T_MODULUS => true,
288290
T_ISSET => true,

src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.inc

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,3 +170,5 @@ $value = (binary) $blah + b"binary $foo";
170170

171171
$test = (1 * static::TEST);
172172
$test = myfunc(1 * static::TEST);
173+
174+
$errorPos = $params[$x]?->getLine() + $commentStart;

src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.inc.fixed

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,3 +170,5 @@ $value = ((binary) $blah + b"binary $foo");
170170

171171
$test = (1 * static::TEST);
172172
$test = myfunc(1 * static::TEST);
173+
174+
$errorPos = ($params[$x]?->getLine() + $commentStart);

src/Standards/Squiz/Tests/Formatting/OperatorBracketUnitTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,7 @@ public function getErrorList($testFile='OperatorBracketUnitTest.inc')
6666
163 => 2,
6767
165 => 2,
6868
169 => 1,
69+
174 => 1,
6970
];
7071
break;
7172
case 'OperatorBracketUnitTest.js':

0 commit comments

Comments
 (0)