Changeset 26277


Ignore:
Timestamp:
06/05/10 09:20:54 (7 years ago)
Author:
flack
Message:

small readability improvements, refs #1848

Location:
branches/ragnaroek/midcom/midcom.core/midcom
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • branches/ragnaroek/midcom/midcom.core/midcom/core/privilege.php

    r25323 r26277  
    386386    public static function get_content_privileges($guid)
    387387    {
    388         $all_privileges = midcom_core_privilege::get_all_privileges($guid);
     388        $all_privileges = self::get_all_privileges($guid);
    389389
    390390        $return = Array();
     
    762762            // The privilege applies if we arrive here, so we merge it into the current collection
    763763            // taking its scope into account.
    764             if ($privilege->assignee == 'EVERYONE')
    765             {
    766                 $scope = MIDCOM_PRIVILEGE_SCOPE_EVERYONE;
    767             }
    768             else if ($privilege->assignee == 'USERS')
    769             {
    770                 $scope = MIDCOM_PRIVILEGE_SCOPE_USERS;
    771             }
    772             else if ($privilege->assignee == 'ANONYMOUS')
    773             {
    774                 $scope = MIDCOM_PRIVILEGE_SCOPE_ANONYMOUS;
    775             }
    776             else if ($privilege->assignee == 'OWNER')
    777             {
    778                 $scope = MIDCOM_PRIVILEGE_SCOPE_OWNER;
    779             }
    780             else
    781             {
    782                 $assignee = $privilege->get_assignee();
    783                 if (! $assignee)
    784                 {
    785                     debug_push_class(__CLASS__, __FUNCTION__);
    786                     debug_print_r('Could not resolve the assignee of this privilege, skipping it:', $privilege);
    787                     debug_pop();
    788                     // Skip broken privileges.
    789                     continue;
    790                 }
    791                 $scope = $assignee->scope;
     764            switch ($privilege->assignee)
     765            {
     766                case 'EVERYONE':
     767                    $scope = MIDCOM_PRIVILEGE_SCOPE_EVERYONE;
     768                    break;
     769                case 'USERS':
     770                    $scope = MIDCOM_PRIVILEGE_SCOPE_USERS;
     771                    break;
     772                case 'ANONYMOUS':
     773                    $scope = MIDCOM_PRIVILEGE_SCOPE_ANONYMOUS;
     774                    break;
     775                case 'OWNER':
     776                    $scope = MIDCOM_PRIVILEGE_SCOPE_OWNER;
     777                    break;
     778                default:
     779                    $assignee = $privilege->get_assignee();
     780                    if (! $assignee)
     781                    {
     782                        debug_push_class(__CLASS__, __FUNCTION__);
     783                        debug_print_r('Could not resolve the assignee of this privilege, skipping it:', $privilege);
     784                        debug_pop();
     785                        // Skip broken privileges.
     786                        continue;
     787                    }
     788                    $scope = $assignee->scope;
     789
     790                    break;
    792791            }
    793792
     
    840839
    841840        // printing debug-statements if object doesn't override base privileges
    842         foreach ($base_privileges as $name => $value)
    843         {
    844             if (!isset($collected_privileges[$name]))
    845             {
    846                 debug_push_class(__CLASS__, __FUNCTION__);
    847                 debug_add("Object privilege {$name} has no value, from parent {$parent_guid} it is {$value}.");
    848                 debug_pop();
    849                 continue;
    850             }
    851         }
    852 
     841        if ($GLOBALS['midcom_config']['log_level'] >= MIDCOM_LOG_DEBUG)
     842        {
     843            foreach ($base_privileges as $name => $value)
     844            {
     845                if (!isset($collected_privileges[$name]))
     846                {
     847                    debug_push_class(__CLASS__, __FUNCTION__);
     848                    debug_add("Object privilege {$name} has no value, from parent {$parent_guid} it is {$value}.");
     849                    debug_pop();
     850                    continue;
     851                }
     852            }
     853        }
    853854        $final_privileges = array_merge($base_privileges, $collected_privileges);
    854855
  • branches/ragnaroek/midcom/midcom.core/midcom/services/auth.php

    r26214 r26277  
    984984        if (!is_null($user))
    985985        {
     986            if (is_object($class))
     987            {
     988                $classname = get_class($class);
     989            }
     990            else
     991            {
     992                $classname = $class;
     993            }
     994           
    986995            debug_push_class(__CLASS__, __FUNCTION__);
    987            
    988             if (is_object($class))
    989             {
    990                 $classname = get_class($class);
    991             }
    992             else
    993             {
    994                 $classname = $class;
    995             }
    996            
    997996            debug_add("Querying privilege {$privilege} for user {$user->id} to class {$classname}", MIDCOM_LOG_DEBUG);
    998997            debug_pop();
Note: See TracChangeset for help on using the changeset viewer.