Merge pull request #11 from inpsyde/hacktoberfest-make-qa-pass

Psalm suppress for RedundantCondition & MixedArgument
inpsyde · Oct 16, 2024 · 1350ada · 1350ada
2 parents 6dbd993 + 82c4a9a
commit 1350ada
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,4 @@
+/.idea/
 /vendor/
 phpunit.xml
 .phpunit.result.cache

diff --git a/src/WpContext.php b/src/WpContext.php
@@ -107,10 +107,9 @@ final public static function determine(): WpContext
      */
     private static function isRestRequest(): bool
     {
-        if (
-            (defined('REST_REQUEST') && REST_REQUEST)
-            || !empty($_GET['rest_route']) // phpcs:ignore
-        ) {
+        /** @psalm-suppress RedundantCondition */
+        $isRestRequest = defined('REST_REQUEST') && REST_REQUEST;
+        if ($isRestRequest || !empty($_GET['rest_route'])) { // phpcs:ignore
             return true;
         }
 
@@ -372,6 +371,7 @@ private function addActionHooks(): void
         ];
 
         foreach ($this->actionCallbacks as $action => $callback) {
+            /** @psalm-suppress MixedArgument */
             add_action($action, $callback, PHP_INT_MIN);
         }
     }
@@ -385,6 +385,7 @@ private function addActionHooks(): void
     private function removeActionHooks(): void
     {
         foreach ($this->actionCallbacks as $action => $callback) {
+            /** @psalm-suppress MixedArgument */
             remove_action($action, $callback, PHP_INT_MIN);
         }
         $this->actionCallbacks = [];