From de4990a0fd34c2662ab4d01ac1d29a1a4c1fcb3e Mon Sep 17 00:00:00 2001 From: atm-irvine <165771178+atm-irvine@users.noreply.github.com> Date: Tue, 27 May 2025 16:04:25 +0200 Subject: [PATCH] Fix fatal error by validating array key type before using in empty() (#34295) --- htdocs/comm/action/class/actioncomm.class.php | 3 +++ 1 file changed, 3 insertions(+) diff --git a/htdocs/comm/action/class/actioncomm.class.php b/htdocs/comm/action/class/actioncomm.class.php index d198de99b1a..5eca0fd136f 100644 --- a/htdocs/comm/action/class/actioncomm.class.php +++ b/htdocs/comm/action/class/actioncomm.class.php @@ -1239,6 +1239,9 @@ class ActionComm extends CommonObject if (!is_array($val)) { // For backward compatibility when val=id $val = array('id' => $val); } + if (!isset($val['id']) || !is_scalar($val['id'])) { + continue; + } if (!empty($already_inserted[$val['id']])) { continue; }