From bca265b82bc7578e4149ab83eada7c60b1649779 Mon Sep 17 00:00:00 2001
From: Kamil Tekiela <tekiela246@gmail.com>
Date: Sat, 8 Apr 2023 22:27:24 +0100
Subject: [PATCH] Cast POST values to expected types

Signed-off-by: Kamil Tekiela <tekiela246@gmail.com>
---
 .../classes/Controllers/Table/FindReplaceController.php  | 6 +++---
 psalm-baseline.xml                                       | 9 ++++-----
 2 files changed, 7 insertions(+), 8 deletions(-)

diff --git a/libraries/classes/Controllers/Table/FindReplaceController.php b/libraries/classes/Controllers/Table/FindReplaceController.php
index 4c61e40e85..ca088f6b93 100644
--- a/libraries/classes/Controllers/Table/FindReplaceController.php
+++ b/libraries/classes/Controllers/Table/FindReplaceController.php
@@ -146,7 +146,7 @@ class FindReplaceController extends AbstractController
             && $_POST['useRegex'] === 'on';
 
         $preview = $this->getReplacePreview(
-            $_POST['columnIndex'],
+            (int) $_POST['columnIndex'],
             $_POST['find'],
             $_POST['replaceWith'],
             $useRegex,
@@ -158,10 +158,10 @@ class FindReplaceController extends AbstractController
     public function replaceAction(): void
     {
         $this->replace(
-            $_POST['columnIndex'],
+            (int) $_POST['columnIndex'],
             $_POST['findString'],
             $_POST['replaceWith'],
-            $_POST['useRegex'],
+            (bool) $_POST['useRegex'],
             $this->connectionCharSet,
         );
         $this->response->addHTML(
diff --git a/psalm-baseline.xml b/psalm-baseline.xml
index c859b9bc4f..7db1f39b4c 100644
--- a/psalm-baseline.xml
+++ b/psalm-baseline.xml
@@ -3591,11 +3591,6 @@
     <InvalidArrayOffset>
       <code><![CDATA[$GLOBALS['errorUrl']]]></code>
     </InvalidArrayOffset>
-    <InvalidScalarArgument>
-      <code><![CDATA[$_POST['columnIndex']]]></code>
-      <code><![CDATA[$_POST['columnIndex']]]></code>
-      <code><![CDATA[$_POST['useRegex']]]></code>
-    </InvalidScalarArgument>
     <MixedArgument>
       <code>$column</code>
       <code>$column</code>
@@ -3641,6 +3636,10 @@
     <PossiblyUnusedParam>
       <code>$request</code>
     </PossiblyUnusedParam>
+    <RiskyCast>
+      <code><![CDATA[$_POST['columnIndex']]]></code>
+      <code><![CDATA[$_POST['columnIndex']]]></code>
+    </RiskyCast>
   </file>
   <file src="libraries/classes/Controllers/Table/GetFieldController.php">
     <PossiblyInvalidArgument>