From e0f73716465ec67615f52f4e66f2e6d997c19303 Mon Sep 17 00:00:00 2001 From: Stefan Idler Date: Wed, 7 Jun 2017 22:02:18 +0200 Subject: [PATCH] BUG: system | Rights.php -> group ids are casted to string before trim()ing them --- module_system/system/Rights.php | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/module_system/system/Rights.php b/module_system/system/Rights.php index 3ac82f06a..47d1f9568 100644 --- a/module_system/system/Rights.php +++ b/module_system/system/Rights.php @@ -108,16 +108,16 @@ private function writeSingleRecord(string $strSystemid, array $arrRights): bool //Splitting up the rights $arrParams = array(); $arrParams[] = (int)$arrRights[self::$STR_RIGHT_INHERIT]; - $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_VIEW]) ? trim($arrRights[self::$STR_RIGHT_VIEW], ",") : "").","; - $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_EDIT]) ? trim($arrRights[self::$STR_RIGHT_EDIT], ",") : "").","; - $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_DELETE]) ? trim($arrRights[self::$STR_RIGHT_DELETE], ",") : "").","; - $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT]) ? trim($arrRights[self::$STR_RIGHT_RIGHT], ",") : "").","; - $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT1]) ? trim($arrRights[self::$STR_RIGHT_RIGHT1], ",") : "").","; - $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT2]) ? trim($arrRights[self::$STR_RIGHT_RIGHT2], ",") : "").","; - $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT3]) ? trim($arrRights[self::$STR_RIGHT_RIGHT3], ",") : "").","; - $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT4]) ? trim($arrRights[self::$STR_RIGHT_RIGHT4], ",") : "").","; - $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT5]) ? trim($arrRights[self::$STR_RIGHT_RIGHT5], ",") : "").","; - $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_CHANGELOG]) ? trim($arrRights[self::$STR_RIGHT_CHANGELOG], ",") : "").","; + $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_VIEW]) ? trim("".$arrRights[self::$STR_RIGHT_VIEW], ",") : "").","; + $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_EDIT]) ? trim("".$arrRights[self::$STR_RIGHT_EDIT], ",") : "").","; + $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_DELETE]) ? trim("".$arrRights[self::$STR_RIGHT_DELETE], ",") : "").","; + $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT]) ? trim("".$arrRights[self::$STR_RIGHT_RIGHT], ",") : "").","; + $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT1]) ? trim("".$arrRights[self::$STR_RIGHT_RIGHT1], ",") : "").","; + $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT2]) ? trim("".$arrRights[self::$STR_RIGHT_RIGHT2], ",") : "").","; + $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT3]) ? trim("".$arrRights[self::$STR_RIGHT_RIGHT3], ",") : "").","; + $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT4]) ? trim("".$arrRights[self::$STR_RIGHT_RIGHT4], ",") : "").","; + $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_RIGHT5]) ? trim("".$arrRights[self::$STR_RIGHT_RIGHT5], ",") : "").","; + $arrParams[] = ",". (!empty($arrRights[self::$STR_RIGHT_CHANGELOG]) ? trim("".$arrRights[self::$STR_RIGHT_CHANGELOG], ",") : "").","; $arrParams[] = $strSystemid; $strQuery = "UPDATE "._dbprefix_."system