Update Magento2/Sniffs/Commenting/ClassPropertyPHPDocFormattingSniff.php

svera · sivaschenko · web-flow · commit 2a57af8e3db5 · 2021-09-14T09:00:30.000+02:00
Co-authored-by: Sergii Ivashchenko &lt;serg.ivashchenko@gmail.com&gt;
diff --git a/Magento2/Sniffs/Commenting/ClassPropertyPHPDocFormattingSniff.php b/Magento2/Sniffs/Commenting/ClassPropertyPHPDocFormattingSniff.php
@@ -141,11 +141,9 @@ public function processMemberVar(File $phpcsFile, $stackPtr)
                 /x';
         $varTagParts = preg_split($re, $tokens[$string]['content']);
 
-        foreach ($varTagParts as $part) {
-            if (stripos($tokens[$isShortDescriptionPreviousVar]['content'], $part) === false) {
+            if (stripos($tokens[$isShortDescriptionPreviousVar]['content'], implode(' ', $varTagParts)) === false) {
                 return;
             }
-        }
         $error = 'Short description duplicates class property name.';
         $phpcsFile->addWarning($error, $isShortDescriptionPreviousVar, 'AlreadyHaveMeaningfulNameVar');
     }

Original file line number	Diff line number	Diff line change
`@@ -141,11 +141,9 @@ public function processMemberVar(File $phpcsFile, $stackPtr)`
`141`	`141`	`/x';`
`142`	`142`	`$varTagParts = preg_split($re, $tokens[$string]['content']);`
`143`	`143`
`144`		`- foreach ($varTagParts as $part) {`
`145`		`- if (stripos($tokens[$isShortDescriptionPreviousVar]['content'], $part) === false) {`
	`144`	`+ if (stripos($tokens[$isShortDescriptionPreviousVar]['content'], implode(' ', $varTagParts)) === false) {`
`146`	`145`	`return;`
`147`	`146`	`}`
`148`		`- }`
`149`	`147`	`$error = 'Short description duplicates class property name.';`
`150`	`148`	`$phpcsFile->addWarning($error, $isShortDescriptionPreviousVar, 'AlreadyHaveMeaningfulNameVar');`
`151`	`149`	`}`