BUG: system | Rights.php -> group ids are casted to string before tri…

…m()ing them
kajona · Jun 7, 2017 · e0f7371 · e0f7371
1 parent fdd8700
commit e0f7371
Showing 1 changed file with 10 additions and 10 deletions.
diff --git 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