Skip to content

Commit 1d499c2

Browse files
committed
AC-662: Create phpcs static check for AutogeneratedClassNotInConstructorTest
1 parent c06cbc2 commit 1d499c2

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

Magento2/Sniffs/PHP/AutogeneratedClassNotInConstructorSniff.php

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
use PHP_CodeSniffer\Sniffs\Sniff;
1212

1313
/**
14-
* Detects possible usage of 'var' language construction.
14+
* Detects bad usages of ObjectManager in constructor.
1515
*/
1616
class AutogeneratedClassNotInConstructorSniff implements Sniff
1717
{
@@ -104,13 +104,10 @@ private function isObjectManagerGetInstance(File $phpcsFile, int $stackPtr): boo
104104
{
105105
$prev = $phpcsFile->findPrevious(T_STRING, $stackPtr - 1);
106106
$next = $phpcsFile->findNext(T_STRING, $stackPtr + 1);
107-
if ($prev &&
107+
return $prev &&
108108
$next &&
109109
$phpcsFile->getTokens()[$prev]['content'] === 'ObjectManager' &&
110-
$phpcsFile->getTokens()[$next]['content'] === 'getInstance') {
111-
return true;
112-
}
113-
return false;
110+
$phpcsFile->getTokens()[$next]['content'] === 'getInstance';
114111
}
115112

116113
/**

0 commit comments

Comments
 (0)