SlevomatCodingStandard.Functions.DisallowEmptyFunction: Private or protected constructor should not be reported

Author: kukulich

SlevomatCodingStandard/Sniffs/Functions/DisallowEmptyFunctionSniff.php

@@ -7,7 +7,13 @@
 use PHP_CodeSniffer\Util\Tokens;
 use SlevomatCodingStandard\Helpers\FunctionHelper;
 use SlevomatCodingStandard\Helpers\TokenHelper;
+use function in_array;
+use const T_CLOSE_CURLY_BRACKET;
 use const T_FUNCTION;
+use const T_OPEN_CURLY_BRACKET;
+use const T_PRIVATE;
+use const T_PROTECTED;
+use const T_SEMICOLON;
 use const T_WHITESPACE;
 
 class DisallowEmptyFunctionSniff implements Sniff
@@ -36,6 +42,16 @@ public function process(File $phpcsFile, $functionPointer): void
 		}
 
 		if (FunctionHelper::getName($phpcsFile, $functionPointer) === '__construct') {
+			$previousPointer = TokenHelper::findPrevious(
+				$phpcsFile,
+				[T_PRIVATE, T_SEMICOLON, T_CLOSE_CURLY_BRACKET, T_OPEN_CURLY_BRACKET],
+				$functionPointer - 1,
+			);
+
+			if ($previousPointer !== null && in_array($tokens[$previousPointer]['code'], [T_PRIVATE, T_PROTECTED], true)) {
+				return;
+			}
+
 			$propertyPromotion = TokenHelper::findNext(
 				$phpcsFile,
 				Tokens::$scopeModifiers,

tests/Sniffs/Functions/data/disallowEmptyFunctionNoErrors.php

@@ -54,3 +54,12 @@ public function __construct(private $a, public $b)
 	}
 
 }
+
+class Foo
+{
+
+	private function __construct()
+	{
+	}
+
+}