Merge pull request #101 from mambax7/feature/check_for_permissionHandle

feature/if there is no permissionHandler, it crashes
XOOPS · Oct 2, 2023 · 2844df8 · 2844df8
2 parents 6cb3473 + 6c4bf3a
commit 2844df8
Showing 1 changed file with 11 additions and 7 deletions.
diff --git a/src/Module/Helper/Permission.php b/src/Module/Helper/Permission.php
@@ -68,13 +68,17 @@ public function checkPermission($gperm_name, $gperm_itemid, $trueifadmin = true)
         $gperm_itemid = (int) $gperm_itemid;
         $gperm_groupid = $this->getUserGroups();
 
-        return $this->permissionHandler->checkRight(
-            $gperm_name,
-            $gperm_itemid,
-            $gperm_groupid,
-            $this->mid,
-            (bool) $trueifadmin
-        );
+        if ($this->permissionHandler) {
+            return $this->permissionHandler->checkRight(
+                $gperm_name,
+                $gperm_itemid,
+                $gperm_groupid,
+                $this->mid,
+                (bool)$trueifadmin
+            );
+        } else {
+            return false;
+        }
     }
 
     /**