fix psalm-internal risky casts

kkmuffme · Sep 15, 2022 · 4e1a654 · 4e1a654
1 parent 0ca91f8
commit 4e1a654
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/Psalm/Internal/Cli/LanguageServer.php b/src/Psalm/Internal/Cli/LanguageServer.php
@@ -298,7 +298,7 @@ function () use ($current_dir, $options, $vendor_dir): ?\Composer\Autoload\Class
             $find_unused_code = 'auto';
         }
 
-        if (isset($options['disable-on-change'])) {
+        if (isset($options['disable-on-change']) && is_numeric($options['disable-on-change'])) {
             $project_analyzer->onchange_line_limit = (int) $options['disable-on-change'];
         }
 

diff --git a/src/Psalm/Internal/Cli/Psalter.php b/src/Psalm/Internal/Cli/Psalter.php
@@ -230,7 +230,7 @@ function () use ($current_dir, $options, $vendor_dir): ?\Composer\Autoload\Class
             chdir($current_dir);
         }
 
-        $threads = isset($options['threads']) ? (int)$options['threads'] : 1;
+        $threads = isset($options['threads']) && is_numeric($options['threads']) ? (int)$options['threads'] : 1;
 
         if (isset($options['no-cache'])) {
             $providers = new Providers(

diff --git a/src/Psalm/Internal/Cli/Refactor.php b/src/Psalm/Internal/Cli/Refactor.php
@@ -284,7 +284,7 @@ function () use ($current_dir, $options, $vendor_dir): ?\Composer\Autoload\Class
             chdir($current_dir);
         }
 
-        $threads = isset($options['threads'])
+        $threads = isset($options['threads']) && is_numeric($options['threads'])
             ? (int)$options['threads']
             : max(1, ProjectAnalyzer::getCpuCount() - 2);