Changeset 26331


Ignore:
Timestamp:
06/12/10 15:25:00 (7 years ago)
Author:
flack
Message:

make sanity checks less redundant, refs #1848

File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/developers/flack/acltuning/midcom.core/midcom/services/auth/acl.php

    r26329 r26331  
    744744    {
    745745        $cache_id = $user_id . '::' . $object_guid;
    746         if (array_key_exists($cache_id, self::$_privileges_cache))
    747         {
     746
     747        if (!array_key_exists($cache_id, self::$_privileges_cache))
     748        {
     749            if (   empty($object_guid)
     750                || empty($object_class))
     751            {
     752                /* No idea if there should be some special log message written */
     753                return array();
     754            }
     755
     756            if (!class_exists($object_class))
     757            {
     758                $_MIDCOM->generate_error(MIDCOM_ERRCRIT, "class '{$object_class}' does not exist");
     759                // This will exit()
     760            }
     761
    748762            $this->_load_privileges_byguid($object_guid, $object_class, $user_id);
    749763        }
     
    767781    private function _load_privileges_byguid($object_guid, $object_class, $user_id)
    768782    {
    769         /* No idea if there should be some special log message written */
    770         if (   empty($object_guid)
    771             || empty($object_class))
    772         {
    773             return array();
    774         }
    775 
    776         if (!class_exists($object_class))
    777         {
    778             $_MIDCOM->generate_error(MIDCOM_ERRCRIT, "class '{$object_class}' does not exist");
    779             // This will exit()
    780         }
    781 
    782         // Check for a cache Hit.
    783783        $cache_id = $user_id . '::' . $object_guid;
    784 
    785         if (array_key_exists($cache_id, self::$_privileges_cache))
    786         {
    787             return;
    788         }
    789784
    790785        self::$_privileges_cache[$cache_id] = array();
Note: See TracChangeset for help on using the changeset viewer.