Changeset 26403


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

remove some redundant checks, refs #359

File:
1 edited

Legend:

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

    r26402 r26403  
    408408     * @return int            One of the MGD_ERR constants
    409409     */
    410     private function _loadNodeData($node, $up = null)
    411     {
    412         // Load the object.
    413         if (!is_object($node))
    414         {
    415             $topic_id = (int) $node;
    416             $topic = midcom_db_topic::get_cached($topic_id);
    417         }
    418         else
    419         {
    420             $topic = $node;
    421             $topic_id = $topic->id;
    422         }
    423 
    424         if (   !$topic
    425             || !$topic->guid)
    426         {
    427             debug_push_class(__CLASS__, __FUNCTION__);
    428             debug_add("Could not open Topic: " . midcom_application::get_error_string(), MIDCOM_LOG_ERROR);
    429             debug_pop();
    430             return MIDCOM_ERRCRIT;
    431         }
     410    private function _loadNodeData($topic, $up = null)
     411    {
     412        $topic_id = $topic->id;
    432413
    433414        // Load the node data and check visibility.
     
    532513     * @access private
    533514     */
    534     private function _get_node_from_database($node, $up = null)
    535     {
    536         if (   is_object($node)
    537             && $node->guid)
    538         {
    539             $topic = $node;
    540         }
    541         else
    542         {
    543             if (!$up)
    544             {
    545                 $up = $this->_up($node);
    546             }
    547             $node = (int) $node;
    548 
    549             // Load the topic first.
    550             $topic = midcom_db_topic::get_cached($node);
    551             if (   !$topic
    552                 || !mgd_is_guid($topic->guid))
    553             {
    554                 debug_push_class(__CLASS__, __FUNCTION__);
    555                 debug_add("Could not load the topic {$node} through DBA; assuming missing privileges.", MIDCOM_LOG_INFO);
    556                 debug_pop();
    557                 return null;
    558             }
    559         }
    560 
     515    private function _get_node_from_database($topic, $up = null)
     516    {
    561517        $urltopic = $topic;
    562518        $id = $this->_nodeid($urltopic->id, $up);
Note: See TracChangeset for help on using the changeset viewer.