Changeset 26395


Ignore:
Timestamp:
06/19/10 17:16:25 (7 years ago)
Author:
flack
Message:

remove useless counter, refs #359

Location:
branches/developers/flack/acltuning/midcom.core/midcom/helper
Files:
2 edited

Legend:

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

    r26221 r26395  
    555555            $leaves[$id][MIDCOM_NAV_FULLURL] = $fullprefix . $leaves[$id][MIDCOM_NAV_RELATIVEURL];
    556556            $leaves[$id][MIDCOM_NAV_ABSOLUTEURL] = $absoluteprefix . $leaves[$id][MIDCOM_NAV_RELATIVEURL];
    557            
     557
    558558            if (is_null($leaves[$id][MIDCOM_NAV_GUID]))
    559559            {
     
    665665                $leaf[MIDCOM_NAV_GUID] = null;
    666666                $leaf[MIDCOM_NAV_OBJECT] = null;
    667                
     667
    668668                // Get the pseudo leaf score from the topic
    669669                if (($score = $topic->get_parameter('midcom.helper.nav.score', "{$topic->id}-{$id}")))
     
    680680                $leaf[MIDCOM_NAV_OBJECT] = $_MIDCOM->dbfactory->get_object_by_guid($leaf[MIDCOM_NAV_GUID]);
    681681            }
    682            
     682
    683683            if (!isset($leaf[MIDCOM_NAV_SORTABLE]))
    684684            {
     
    687687
    688688            // Now complete the actual leaf information
    689            
     689
    690690            // Score
    691691            if (!isset($leaf[MIDCOM_NAV_SCORE]))
     
    10381038            if (array_key_exists($uptopic->id, $this->_nodes))
    10391039            {
    1040                 // We already saw this node, no point in loading it again 
     1040                // We already saw this node, no point in loading it again
    10411041                $lastgoodnode = $uptopic->id;
    10421042                continue;
     
    11421142            }
    11431143        }
    1144        
     1144
    11451145        if (isset($listed[$parent_node]))
    11461146        {
    11471147            return $listed[$parent_node];
    11481148        }
    1149        
     1149
    11501150        //echo "Called for {$parent_node} {$this->_nodes[$parent_node][MIDCOM_NAV_RELATIVEURL]}<br />\n";
    11511151
     
    11651165
    11661166        $qb->add_order('metadata.score', 'DESC');
    1167         $qb->add_order('metadata.created'); 
     1167        $qb->add_order('metadata.created');
    11681168        $subnodes = $qb->execute();
    11691169
     
    12041204            }
    12051205
    1206             $result[] = $subnode_id;           
    1207         }
    1208        
     1206            $result[] = $subnode_id;
     1207        }
     1208
    12091209        if ($GLOBALS['midcom_config']['i18n_multilang_navigation'])
    12101210        {
     
    12151215                $sortable_result[] = $this->_nodes[$subnode_id];
    12161216            }
    1217            
     1217
    12181218            // Sort via midcom_helper_itemlist
    12191219            if (!uasort($sortable_result, array('midcom_helper_itemlist_score', 'sort_cmp')))
     
    12221222                return $result;
    12231223            }
    1224            
     1224
    12251225            // Set the newly sorted items to the results
    12261226            $result = array();
     
    12861286                $sortable_result[] = $this->_leaves[$leaf_id];
    12871287            }
    1288            
     1288
    12891289            // Sort via midcom_helper_itemlist
    12901290            if (!uasort($sortable_result, array('midcom_helper_itemlist_score', 'sort_cmp')))
     
    12931293                return $result;
    12941294            }
    1295            
     1295
    12961296            // Set the newly sorted items to the results
    12971297            $result = array();
     
    13081308
    13091309    /**
    1310      * This will give you a key-value pair describeing the node with the ID
     1310     * This will give you a key-value pair describing the node with the ID
    13111311     * $node_id. The defined keys are described above in Node data interchange
    13121312     * format. You will get false if the node ID is invalid.
     
    13301330                return false;
    13311331            }
    1332            
     1332
    13331333            if (!isset($this->_nodes[$node_id]))
    13341334            {
     
    15381538    /**
    15391539     * Retrieve the ID of the upper node of the currently displayed node.
    1540      * 
     1540     *
    15411541     * @return mixed    The ID of the node in question.
    15421542     */
  • branches/developers/flack/acltuning/midcom.core/midcom/helper/itemlist/score.php

    r22990 r26395  
    22/**
    33 * @package midcom
    4  * @author The Midgard Project, http://www.midgard-project.org 
     4 * @author The Midgard Project, http://www.midgard-project.org
    55 * @version $Id$
    66 * @copyright The Midgard Project, http://www.midgard-project.org
     
    1010/**
    1111 * Itemlist sorter for navigation items sorted by score (leaves and nodes mixed)
    12  * 
     12 *
    1313 * @package midcom
    1414 */
     
    1616{
    1717    /**
    18      * get_sorted_list  - get a list objects ready for showing. 
    19      * 
     18     * get_sorted_list  - get a list objects ready for showing.
     19     *
    2020     * @access public
    2121     * @return mixed  False on failure or an array of navigation items on success
     
    4242
    4343        $result = array();
    44         $counter = 0;
     44
    4545        if ($nodes_list)
    4646        {
    4747            foreach ($nodes_list as $node_id)
    4848            {
    49                 $result[$counter] = $this->_basicnav->get_node($node_id);
    50                 $counter++;
     49                $result[] = $this->_basicnav->get_node($node_id);
    5150            }
    5251        }
     
    5554            foreach ($leaves_list as $leaf_id)
    5655            {
    57                 $result[$counter] = $this->_basicnav->get_leaf($leaf_id);
    58                 $counter++;
     56                $result[] = $this->_basicnav->get_leaf($leaf_id);
    5957            }
    6058        }
     
    6563        return $result;
    6664    }
    67    
     65
    6866    /**
    6967     * User defined sort comparison method
    70      * 
     68     *
    7169     * @static
    7270     * @access public
     
    7876    {
    7977        // This should also sort out the situation were score is not
    80         // set. 
     78        // set.
    8179        if ($a[MIDCOM_NAV_SCORE] === $b[MIDCOM_NAV_SCORE])
    8280        {
    8381             return strcmp($a[MIDCOM_NAV_NAME], $b[MIDCOM_NAV_NAME]);
    8482        }
    85        
     83
    8684        return (integer) (($a[MIDCOM_NAV_SCORE] < $b[MIDCOM_NAV_SCORE]) ? 1 : -1);
    8785    }
Note: See TracChangeset for help on using the changeset viewer.