Changeset 356 for trunk/Modules/User/User.php

Timestamp:

Jan 18, 2012, 10:21:43 AM (13 years ago)

Author:

chronos

Message:

• Upraveno: Zrychlení kontroly uživatelských oprávnění pro opakované kontroly se stejnými podmínkami.

File:

• trunk/Modules/User/User.php (modified) (3 diffs)

trunk/Modules/User/User.php

@@ -34 +34 @@
   var $AnonymousUserId = 98;
   var $OnlineStateTimeout = 600; // in seconds
+  var $PermissionCache = array();
 
   function __construct($Database, $System)
@@ -226 +227 @@
   function CheckPermission($Module, $Operation, $ItemType = '', $ItemIndex = 0)
   {
-        $DbResult = $this->Database->select('PermissionOperation', 'Id', '`Module`="'.$Module.'" AND `Item`="'.$ItemType.'" AND `ItemId`='.$ItemIndex.' AND `Operation`="'.$Operation.'"');
-    if($DbResult->num_rows > 0)
-    {
-      $DbRow = $DbResult->fetch_array();
-      $OperationId = $DbRow['Id'];
-
+    // First try to check cache
+    if(in_array(array($Module, $Operation, $ItemType, $ItemType), $this->PermissionCache)) 
+    {
+      $OperationId = array_search(array($Module, $Operation, $ItemType, $ItemType), $this->PermissionCache);
+      $PermissionExists = is_numeric($OperationId);
+    } else 
+    {    
+      // If no permission combination exists in cache, do new check of database items
+      $DbResult = $this->Database->select('PermissionOperation', 'Id', '`Module`="'.$Module.'" AND `Item`="'.$ItemType.'" AND `ItemId`='.$ItemIndex.' AND `Operation`="'.$Operation.'"');
+      if($DbResult->num_rows > 0)
+      {
+        $DbRow = $DbResult->fetch_array();
+        $OperationId = $DbRow['Id'];
+        $this->PermissionCache[$DbRow['Id']] = array($Module, $Operation, $ItemType, $ItemType);        
+        $PermissionExists = true;
+      } else 
+      {          
+        $this->PermissionCache[$DbRow['Id'].'_'] = array($Module, $Operation, $ItemType, $ItemType);        
+        $PermissionExists = false;
+      }
+    } 
+    
+    if($PermissionExists)
+    {
       // Check user-operation relation
       $DbResult = $this->Database->select('PermissionUserAssignment', '*', '`User`="'.$this->User['Id'].'" AND `GroupOrOperation`="'.$OperationId.'" AND `Type`="Operation"');
@@ -240 +259 @@
       while($DbRow = $DbResult->fetch_array())
       {
-         if($this->CheckGroupPermission($DbRow['GroupOrOperation'], $OperationId) == true) return(true);
+       if($this->CheckGroupPermission($DbRow['GroupOrOperation'], $OperationId) == true) return(true);
       }
       return(false);