Changeset 26283


Ignore:
Timestamp:
06/05/10 17:10:33 (7 years ago)
Author:
flack
Message:

restore old optimization and use better function name, refs #1848

File:
1 edited

Legend:

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

    r26282 r26283  
    726726        {
    727727            // Check whether we need to take this privilege into account
    728             if (!$privilege->_is_privilege_valid($user))
     728            if (!$privilege->_does_privilege_apply($user))
    729729            {
    730730                continue;
     
    844844     * @return boolean Indicating whether the privilege record applies for the user, or not.
    845845     */
    846     private function _is_privilege_valid($user)
    847     {
    848         if (!$this->__privilege)
     846    private function _does_privilege_apply($user)
     847    {
     848        if (!is_array($this->__privilege))
    849849        {
    850850            return false;
     
    856856                && $user == 'EVERYONE'))
    857857        {
    858             if (   $this->assignee != 'EVERYONE'
    859                 && $this->assignee != 'ANONYMOUS')
     858            if (   $this->__privilege['assignee'] != 'EVERYONE'
     859                && $this->__privilege['assignee'] != 'ANONYMOUS')
    860860            {
    861861                return false;
     
    864864        else
    865865        {
    866             if ($this->assignee == 'ANONYMOUS')
     866            if ($this->__privilege['assignee'] == 'ANONYMOUS')
    867867            {
    868868                return false;
    869869            }
    870             if (    strstr($this->assignee, 'user:') !== false
    871                 && $this->assignee != $user->id)
     870            if (    strstr($this->__privilege['assignee'], 'user:') !== false
     871                && $this->__privilege['assignee'] != $user->id)
    872872            {
    873873                return false;
    874874            }
    875             if (strstr($this->assignee, 'group:') !== false)
    876             {
    877                 if (! $user->is_in_group($this->assignee))
     875            if (strstr($this->__privilege['assignee'], 'group:') !== false)
     876            {
     877                if (! $user->is_in_group($this->__privilege['assignee']))
    878878                {
    879879                    return false;
Note: See TracChangeset for help on using the changeset viewer.