Changeset 26402


Ignore:
Timestamp:
06/20/10 16:53:23 (7 years ago)
Author:
flack
Message:

remove some redundant code, refs #359

File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/developers/flack/acltuning/midcom.core/midcom/helper/_basicnav.php

    r26401 r26402  
    274274    private function _loadNode($node_id, $up = null)
    275275    {
    276         if (   is_object($node_id)
    277             && $node_id->guid)
    278         {
    279             $topic = $node_id;
    280             $node_id = $this->_nodeid($topic->id, $up);
    281 
    282             // Check if we have a cached version of the node already
    283             if (isset(self::$_nodes[$node_id]))
    284             {
    285                 return MIDCOM_ERROK;
    286             }
    287         }
    288         else
    289         {
    290             // Check if we have a cached version of the node already
    291             if (isset(self::$_nodes[$node_id]))
    292             {
    293                 return MIDCOM_ERROK;
    294             }
    295 
    296             if (!$up)
    297             {
    298                 $up = $this->_up($node_id);
    299             }
    300             $topic_id = (int) $node_id;
    301 
    302             $topic = midcom_db_topic::get_cached($topic_id);
    303             if (   !$topic
    304                 || !$topic->guid)
    305             {
    306                 debug_push_class(__CLASS__, __FUNCTION__);
    307                 debug_add("Could not load Topic #{$topic_id}: " . midcom_application::get_error_string(), MIDCOM_LOG_ERROR);
    308                 debug_pop();
    309                 return MIDCOM_ERRCRIT;
    310             }
     276        // Check if we have a cached version of the node already
     277        if (isset(self::$_nodes[$node_id]))
     278        {
     279            return MIDCOM_ERROK;
     280        }
     281
     282        if (!$up)
     283        {
     284            $up = $this->_up($node_id);
     285        }
     286        $topic_id = (int) $node_id;
     287
     288        $topic = midcom_db_topic::get_cached($topic_id);
     289        if (   !$topic
     290            || !$topic->guid)
     291        {
     292            debug_push_class(__CLASS__, __FUNCTION__);
     293            debug_add("Could not load Topic #{$topic_id}: " . midcom_application::get_error_string(), MIDCOM_LOG_ERROR);
     294            debug_pop();
     295            return MIDCOM_ERRCRIT;
    311296        }
    312297
     
    674659        else
    675660        {
    676             if (!$up || (!isset(self::$_nodes[$up]) && $this->_loadNode($up) !== MIDCOM_ERROK))
     661            if (!$up || $this->_loadNode($up) !== MIDCOM_ERROK)
    677662            {
    678663                $up = $urltopic->up;
     
    10401025        $up = null;
    10411026
    1042         if (!isset(self::$_nodes[$parent_node]))
    1043         {
    1044             if ($this->_loadNode($parent_node) != MIDCOM_ERROK)
    1045             {
    1046                 debug_push_class(__CLASS__, __FUNCTION__);
    1047                 debug_add("Unable to load parent node $parent_node", MIDCOM_LOG_ERROR);
    1048                 debug_pop();
    1049                 return false;
    1050             }
     1027        if ($this->_loadNode($parent_node) !== MIDCOM_ERROK)
     1028        {
     1029            debug_push_class(__CLASS__, __FUNCTION__);
     1030            debug_add("Unable to load parent node $parent_node", MIDCOM_LOG_ERROR);
     1031            debug_pop();
     1032            return false;
    10511033        }
    10521034
     
    10961078            $subnode = midcom_db_topic::get_cached($guid);
    10971079            $subnode_id = $this->_nodeid($subnode->id, $up);
    1098             if (   !isset(self::$_nodes[$subnode_id])
    1099                 && $this->_loadNode($subnode, $up) !== MIDCOM_ERROK)
     1080            if ($this->_loadNode($subnode_id, $up) !== MIDCOM_ERROK)
    11001081            {
    11011082                //debug_add("Node {$subnode->id} could not be loaded, ignoring it", MIDCOM_LOG_INFO);
     
    11591140        static $listed = array();
    11601141
    1161         if (!array_key_exists($parent_node, self::$_nodes))
    1162         {
    1163             if ($this->_loadNode($parent_node) != MIDCOM_ERROK)
    1164             {
    1165                 return false;
    1166             }
     1142        if ($this->_loadNode($parent_node) !== MIDCOM_ERROK)
     1143        {
     1144            return false;
    11671145        }
    11681146
     
    12541232        }
    12551233
     1234        if ($this->_loadNode($node_id) != MIDCOM_ERROK)
     1235        {
     1236            return false;
     1237        }
     1238
    12561239        if (!isset(self::$_nodes[$node_id]))
    12571240        {
    1258             if ($this->_loadNode($node) != MIDCOM_ERROK)
    1259             {
    1260                 return false;
    1261             }
    1262 
    1263             if (!isset(self::$_nodes[$node_id]))
    1264             {
    1265                 return false;
    1266             }
     1241            return false;
    12671242        }
    12681243
     
    14031378    function get_node_uplink($node_id)
    14041379    {
    1405         if (!array_key_exists($node_id, self::$_nodes))
    1406         {
    1407             if ($this->_loadNode($node_id) != MIDCOM_ERROK)
    1408             {
    1409                 return false;
    1410             }
     1380        if ($this->_loadNode($node_id) !== MIDCOM_ERROK)
     1381        {
     1382            return false;
    14111383        }
    14121384
     
    14891461        $node_id = $id_elements[0];
    14901462
    1491         if (   !isset(self::$_nodes[$node_id])
    1492             && $this->_loadNode($node_id) != MIDCOM_ERROK)
     1463        if ($this->_loadNode($node_id) !== MIDCOM_ERROK)
    14931464        {
    14941465            debug_add("Tried to verify the leaf id {$leaf_id}, which should belong to node {$node_id}, but this node cannot be loaded, see debug level log for details.",
Note: See TracChangeset for help on using the changeset viewer.