Allow the if-true-return-false pattern

Eventjet/ruleset.xml

@@ -265,9 +265,6 @@
     <!-- Require new instances with parentheses -->
     <rule ref="SlevomatCodingStandard.ControlStructures.NewWithParentheses"/>
 
-    <!-- Forbid usage of conditions when a simple return can be used -->
-    <rule ref="SlevomatCodingStandard.ControlStructures.UselessIfConditionWithReturn"/>
-
     <!-- Forbid unused variables passed to closures via `use` -->
     <rule ref="SlevomatCodingStandard.Functions.UnusedInheritedVariablePassedToClosure"/>
 

php-cs-fixer-rules.php

@@ -121,7 +121,6 @@
     'phpdoc_var_annotation_correct_order' => false,
     'php_unit_data_provider_static' => true,
     'self_accessor' => true,
-    'simplified_if_return' => true,
     'single_quote' => true,
     'single_space_around_construct' => true,
     // Loose comparison is lint territory — PHPStan and Psalm both flag it. Kept

tests/fixtures/valid/useless-if-with-return.php

@@ -0,0 +1,22 @@
+<?php
+
+declare(strict_types=1);
+
+// The `if (cond) { return true; } return false;` shape is allowed — the rules
+// that would collapse it (`SlevomatCodingStandard.ControlStructures.UselessIfConditionWithReturn`
+// and `simplified_if_return`) are off. The pattern is useful in functions
+// where several branches each return a boolean: collapsing only the last
+// branch would make it visually inconsistent with the ones above it.
+function canProceed(int $value, bool $locked): bool
+{
+    if ($locked) {
+        return false;
+    }
+    if ($value < 0) {
+        return false;
+    }
+    if ($value > 0) {
+        return true;
+    }
+    return false;
+}