fix(condition): infinite loop detection

pluginsGLPI · Feb 14, 2023 · 172d5e8 · 172d5e8
1 parent 9af4714
commit 172d5e8
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/ajax/showfields.php b/ajax/showfields.php
@@ -58,6 +58,9 @@
 try {
     $visibility = PluginFormcreatorFields::updateVisibility($_POST);
 } catch (Exception $e) {
+    echo json_encode([
+        'error' => $e->getMessage(),
+    ]);
     http_response_code(500);
     exit();
 }

diff --git a/inc/fields.class.php b/inc/fields.class.php
@@ -140,7 +140,8 @@ public static function isVisible(PluginFormcreatorConditionnableInterface $item,
       /** @var CommonDBTM $item */
       $itemtype = get_class($item);
       $itemId = $item->getID();
-      if (!isset(self::$visibility[$itemtype][$itemId])) {
+
+      if (!isset(self::$visibility[$itemtype]) || !array_key_exists($itemId, self::$visibility[$itemtype])) {
          self::$visibility[$itemtype][$itemId] = null;
       } else if (self::$visibility[$itemtype][$itemId] !== null) {
          return self::$visibility[$itemtype][$itemId];