Skip to content

Commit 3fe6883

Browse files
committed
#107: [New Rule] Implement sniff for constants PHPDoc formatting
- added strict comparison - changed sequence in the ruleset.xml
1 parent 2cb805a commit 3fe6883

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

Magento2/Sniffs/Commenting/ConstantsPHPDocFormattingSniff.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,8 @@ public function process(File $phpcsFile, $stackPtr)
3131
{
3232
$tokens = $phpcsFile->getTokens();
3333

34-
if ($tokens[$stackPtr]['code'] != T_CONST
35-
&& !($tokens[$stackPtr]['content'] == 'define' && $tokens[$stackPtr+1]['code'] == T_OPEN_PARENTHESIS)
34+
if ($tokens[$stackPtr]['code'] !== T_CONST
35+
&& !($tokens[$stackPtr]['content'] === 'define' && $tokens[$stackPtr + 1]['code'] === T_OPEN_PARENTHESIS)
3636
) {
3737
return;
3838
}

Magento2/ruleset.xml

+4-4
Original file line numberDiff line numberDiff line change
@@ -44,10 +44,6 @@
4444
<severity>10</severity>
4545
<type>error</type>
4646
</rule>
47-
<rule ref="Magento2.Commenting.ConstantsPHPDocFormatting">
48-
<severity>5</severity>
49-
<type>warning</type>
50-
</rule>
5147
<rule ref="Magento2.PHP.FinalImplementation">
5248
<severity>10</severity>
5349
<type>error</type>
@@ -500,6 +496,10 @@
500496
</rule>
501497

502498
<!-- Severity 5 warnings: PHPDoc formatting and commenting issues. -->
499+
<rule ref="Magento2.Commenting.ConstantsPHPDocFormatting">
500+
<severity>5</severity>
501+
<type>warning</type>
502+
</rule>
503503
<rule ref="Squiz.Commenting.DocCommentAlignment">
504504
<severity>5</severity>
505505
<type>warning</type>

0 commit comments

Comments
 (0)