/[drupal]/contributions/modules/drigg/drigg_ui/drigg_ui.module

Diff of /contributions/modules/drigg/drigg_ui/drigg_ui.module

Parent Directory | Revision Log | View Revision Graph Revision Graph | View Patch Patch

-revision 1.1.2.14.2.1,
Fri Nov  7 01:00:12 2008 UTC
+revision 1.1.2.14.2.2,
Tue Jun 16 03:46:25 2009 UTC
 Line 1
  <?php
- // $Id$
+ // $Id: drigg_ui.module,v 1.1.2.14.2.1 2008/11/07 01:00:12 mercmobily Exp $
  /**
  * @file
  */
  /**
   * Implementation of hook_help().
   */
  function drigg_ui_help($path, $arg) {
    switch ($path) {
    }
  }
  /**
   * Implementation of hook_menu().
   *
   * This is again "the usual story". However, note that this "menu" is
   * quite tricky, because it needs to pass the right arguments to
   * drigg_ui_node_list(). The function is commented throughout
   */
  function drigg_ui_menu() {
    $items = array();
    $items['admin/settings/drigg/drigg_ui'] = array(
      'title' => 'Drigg UI',
      'description' => 'Settings for the Drigg module',
      'page callback' => 'drupal_get_form',
      'page arguments' => array('drigg_ui_admin_settings'),
      'access callback' => 'user_access',
      'access arguments' => array('administer site configuration'),
      'weight' => -7,
      'type' => MENU_LOCAL_TASK,
    );
    $path = variable_get('drigg_home_name', 'drigg_home');
    // CASE #-1: The home page
    $items[$path] = array(
      'title' => 'Home',
      'page callback' => 'drigg_ui_node_list',
      'page arguments' => array('published', 'newest' ),
      'access callback' => TRUE,
      'type' => MENU_CALLBACK_ITEM,
    );
    // CASE #0: arg(0) is a valid section, and arg(1) is empty
    // E.g. /EndUser
    // Attempt to retrieve the node from the URL
    $tid = drigg_is_section_valid(arg(0));
    $catlist = drigg_section_list();
    if ($catlist) {
      foreach ($catlist['by_safe_name'] as $key => $value) {
        $items[$key] = array(
          'title' => 'View',
          'page callback' => 'drigg_ui_node_list',
          'page arguments' => array('published', 'newest', 0),
          'access callback' => TRUE,
          'type' => MENU_CALLBACK_ITEM,
        );
      }
    }
    //#########   arg(0) not working in DRUPAL-6 ####################
    //$items['%my_special_loader'] = array(                         #
    //  'title' => 'View',                                          #
    //  'page callback' => 'drigg_ui_node_list',                    #
    //  'page arguments' => array('published', 'newest', 0),        #
    //  'access callback' => TRUE,                                  #
    //  'type' => MENU_CALLBACK_ITEM,                               #
    //);                                                            #
    //###############################################################
    $items['published'] = array(
      'title' => 'View',
      'page callback' => 'drigg_ui_node_list',
      'page arguments' => array(0),
      // NOTE: the following line is here because the 'path' is
      // not passed as parameters to the callback - this forces it
      'access callback' => TRUE,
      'type' => MENU_CALLBACK_ITEM,
    );
    $items['upcoming'] = array(
      'title' => 'View',
      'page callback' => 'drigg_ui_node_list',
      'page arguments' => array(0),
      // NOTE: the following line is here because the 'path' is
      // not passed as parameters to the callback - this forces it
      'access callback' => TRUE,
      'type' => MENU_CALLBACK_ITEM,
    );
    $items['archived'] = array(
      'title' => 'View',
      'page callback' => 'drigg_ui_node_list',
      'page arguments' => array(0),
      // NOTE: the following line is here because the 'path' is
      // not passed as parameters to the callback - this forces it
      'access callback' => TRUE,
      'type' => MENU_CALLBACK_ITEM,
    );
    $items['feed'] = array(
      'title' => 'View',
      'page callback' => 'drigg_ui_node_feed',
      'page arguments' => array(1, 2),
      'access callback' => TRUE,
      'type' => MENU_CALLBACK_ITEM,
    );
    return $items;
  }
  //##########    Currently not working with DRUPAL-6  #############
  function my_special_loader_load($my_special_loader) {
    $tid = drigg_is_section_valid($my_special_loader);
    if ($tid) {
      return $my_special_loader;
    }
    else {
      return FALSE;
    }
  }
  //################################################################
  function drigg_ui_init() {
    drupal_add_js(drupal_get_path('module', 'drigg') .'/drigg.js');
  }
  // FIXME: See if this needs to stay. Not sure.
  /*function drigg_ui_nodeapi($node, $op) {
    if ($op == 'rss item' && $node->type == 'drigg') {
      $node->teaser = extra_voting_forms_show_form($node,'n',3) . $node->teaser;
    }
  }*/
  /////////////////////////////////////////////////////////////////
  //           MAIN PATH ROUTING FUNCTIONS - FEED AND PAGE
  /////////////////////////////////////////////////////////////////
  /**
   * Returns an RSS feed (all, or only published) for the specified category.
   *
   * @param $action
   *   It can be "published" (only promoted nodes are shown) or "all" (all
   *   submitted nodes are shown)
   * @param $category
   *   A valid section (in its safe_name format). If it's empty, it's
   *   assumed "all"
   * @return
   *   The feed
  */
  function drigg_ui_node_feed($action, $category) {
    // Set some defaults, so that it's easier to know
    // what to do
    if ($action == '') {
      $action = 'published';
    }
    if ($category == '') {
      $category = 'All';
    }
    // Return errors if the wrong values are passed here
    if ($action != 'all' && $action != 'published') {
      drupal_not_found();
      return;
    }
    if ($category != 'All' && ! drigg_is_section_valid( $category )) {
      drupal_not_found();
      return;
    }
    // Make up the URL string
    $url = "";
    if ($category != 'All') {
      $url = $category;
    }
    $options = array(
      'query' => NULL,
      'fragment' => NULL,
      'absolute' => TRUE,
    );
    $url = url($url, $options);
    // Make up a decent title
    if ($action == 'all') {
      $title = t('Submitted news');
    }
    else {
      $title = t('Published news');
    }
    if ($category != '' && $category != 'All') {
      $title .= ' - '. drigg_section_name_by_safe_name($category);
    }
    $title = variable_get('site_name', 'Drupal') .' - '. $title;
    // Set the $channel variable, which will be used by node_feed
    $channel['link'] = $url;
    $channel['title'] = $title;
    $channel['description'] = '';
    if ($category != 'All') {
      $tl = drigg_section_list();
      $tid = $tl['by_safe_name'][$category];
      $extra_join .= ' LEFT JOIN {term_node} t ON n.nid = t.nid ';
      $where_conditions .=" t.tid = $tid AND ";
    }
    // If the action is 'published', add a WHERE condition so that only
    // promoted stuff is shown
    if ($action == 'published') {
      $where_conditions .= ' n.promote = 1 AND ';
      $order_by_statement = 'ORDER BY dn.promoted_on DESC';
    }
    else {
      $order_by_statement = 'ORDER BY n.created DESC';
    }
    // Actually run the query
    $query = 'SELECT * FROM {node} n LEFT JOIN {drigg_node} dn ON n.nid = dn.dnid '. $extra_join .' WHERE n.type = \'drigg\' AND '. $where_conditions .' n.status = 1 '. $order_by_statement;
    $result = db_query_range(db_rewrite_sql($query,'n','nid'), 0, 20);
          while ($rs = db_fetch_object($result)) {
            $nids[] = $rs->nid;
          }
    // Show the feed baby!
    node_feed($nids, $channel);
  }
  /**
   * The heart of the system. It will show the right nodes according
   * to what the requested URL is. The code is self-documenting (with
   * tons of code)
   * The URL scheme is:
   *
   *    http://www.site.com/Section_1/The_Title_Of_The_story
   *                       /Section_2/Yet_another_title
   *                       /Section_1/
   *                       /Section_2/
   *
   *                       /published
   *                       /published/newest
   *                       /published/newest/Section
   *                       /published/top24h/Section
   *                       /published/top7days/Sectiony
   *                       /published/top30days/Sectiony
   *                       /published/top365days/Section
   *
   *                       /upcoming
   *                       /upcoming/newest
   *                       /upcoming/newest/Section
   *                       /upcoming/oldest/Section
   *                       /upcoming/mostpopular/Section
   *                       /upcoming/leastpopular/Section
   *                       /archived
   *                       /archived/newest
   *                       /archived/newest/Section
   *                       /archived/oldest/Section
   *                       /archived/mostpopular/Section
   *                       /archived/leastpopular/Section
   *
   * @param $action
   *   It can be 'published', 'upcoming', 'archived'
   * @param $filter
   *   This is tricky, because what it can be depends on the
   *   $action.
   *   For 'upcoming' and 'archived', it can be:
   *   'newest', 'oldest', 'mostpopular', 'leastpopular'.
   *   For 'published', it can be:
   *   'newest', 'top24h', 'top7days', 'top30days', 'top356days'.
   *
   * @param $category
   *   A valid section (in its safe_name format).
   *
   * @return
   *
  */
  function drigg_ui_node_list($action, $filter = NULL, $category = NULL) {
    $query_vars = array();
    $where_conditions = '';
    #drupal_set_message("Values: $action,$filter,$category");
    if ($filter == NULL) {
      $filter = 'newest';
    }
    #drupal_set_message("Values (after): $action,$filter,$category");
    // Set the breadcrumb
    $breadcrumb = array(l(t('Home'), NULL), t('Scoops') );
    drupal_set_breadcrumb($breadcrumb);
    // Check that the category exists.
    // Also, store it in $tid whire you're there
    if ($category != NULL) {
      $tid = drigg_is_section_valid( $category );
      if (!$tid) {
        #drupal_set_message('1');
        return drupal_not_found();
      }
    }
    // Check the filter - for action "published"
    if ($action == 'published') {
      if (
        $filter != 'newest' &&
        $filter != 'top24h' &&
        $filter != 'top7days' &&
        $filter != 'top30days' &&
        $filter != 'top365days') {
          #drupal_set_message('2');
          return drupal_not_found();
      }
    // Check the filter - for actions "upcoming" and "archived""
    }
    else {
      if (
        $filter != 'newest' &&
        $filter != 'oldest' &&
        $filter != 'mostpopular' &&
        $filter != 'leastpopular') {
          #drupal_set_message('3');
          return drupal_not_found();
      }
    }
    // Sort out the feed!
    if ($action == 'published') {
      $feed_action = 'published';
    }
    else {
      $feed_action = 'all';
    }
    if ($category == 'All' || $category == '') {
      $feed_category = 'All';
    }
    else {
      $feed_category = $category;
    }
    $options = array(
      'query' => NULL,
      'fragment' => NULL,
      'absolute' => TRUE,
    );
    $feed_url = url("feed/$feed_action/$feed_category", $options);
    drupal_add_feed($feed_url, variable_get('site_name', 'Drupal') .' '. t('RSS'));
    // Assigning a decent title to this page
    if ($action == 'published') {
      $page_title = t('Published scoops');
    }
    if ($action == 'upcoming') {
      $page_title = t('Upcoming scoops');
    }
    if ($action == 'archived') {
      $page_title = t('Archived scoops');
    }
    if ($category != '') {
      $page_title .= " - ". drigg_section_name_by_safe_name($category);
    }
    // Set the title
    drupal_set_title($page_title);
    // Initialising the basic query variables
    $extra_join = '';
    $where_conditions = '';
    $query_vars = array();
    $order_by = '';
    // This bit is true for every single case... so, here it is
    // Populate the tid bit
    if ($tid) {
      /*
      $extra_join .= ' LEFT JOIN {term_node} t ON n.nid = t.nid ';
      $where_conditions.=" t.tid=%d AND ";
      $query_vars[]=$tid;
      */
      $tid_list = listize_from_array(drigg_ui_get_all_offspring_tids($tid)); //string
      //now $tid_list = "12,13,14,16, ";
      $tid_list .= $tid;
      $extra_join .= ' LEFT JOIN {term_node} t ON n.nid = t.nid ';
      $where_conditions .= " t.tid IN ($tid_list) AND ";
      $query_vars[] = $tid;
    }
    /////////////////////////////////////
    // DEAL WITH THE "published" OPTION
    /////////////////////////////////////
    if ($action == 'published') {
      if ($filter == 'newest') {
        $order_by = 'ORDER BY n.sticky DESC, dn2.promoted_on DESC';
      }
      if ( $filter == 'top24h' ||
          $filter == 'top7days' ||
          $filter == 'top30days' ||
          $filter == 'top365days' ) {
        // Order by is different
        $order_by = 'ORDER BY vc.value DESC, dn2.promoted_on DESC';
        // Join something extra
        $extra_join .= ' LEFT JOIN {votingapi_cache} vc ON n.nid = vc.content_id ';
        // Work out the timeframe
        $where_bit['top24h'] = 60*60*24;
        $where_bit['top7days'] = 60*60*24*7;
        $where_bit['top30days'] = 60*60*24*30;
        $where_bit['top365days'] = 60*60*24*365;
        $where_conditions .= drigg_votingapi_query('vc') ." AND n.created > UNIX_TIMESTAMP(CURRENT_TIMESTAMP()) - ". $where_bit[$filter] ." AND ";
      }
      $query = 'SELECT * FROM {node} n '. $extra_join .' WHERE n.type = \'drigg\' AND '. $where_conditions .' n.status = 1 and n.promote = 1 '. $order_by;
    }
    ////////////////////////////////////////////////
    // DEAL WITH THE "upcoming/archived" OPTION
    ////////////////////////////////////////////////
    if ($action == 'upcoming' || $action == 'archived') {
      // Sort out the upcoming/archived stuff first. Only the operator changes
      if ($action == 'upcoming') {
        $operator = '>';
      }
      else {
        $operator = '<';
      }
      $where_conditions .= " n.created  $operator UNIX_TIMESTAMP(CURRENT_TIMESTAMP()) - 60*60*". variable_get('drigg_hours_in_upcoming', '48') ." AND ";
      // CASE #1: It's either "newest" or "oldest"
      if ($filter == 'newest' || $filter == 'oldest') {
        $order_by = 'ORDER BY n.created';
        if ($filter == 'newest') {
          $order_by .=  ' DESC ';
        }
      }
      // CASE #2: It's either "mostpopular" or "leastpopular"
      if ($filter == 'mostpopular' || $filter == 'leastpopular') {
        // Order by first...
        if ($filter == 'mostpopular') {
          $extra_bit = ' DESC ';
        }
        $order_by = "ORDER BY vc.value $extra_bit";
        // Join something extra
        $extra_join .= ' LEFT JOIN {votingapi_cache} vc ON n.nid = vc.content_id ';
        $where_conditions .= drigg_votingapi_query('vc') ." AND ";
      }
      // Make up the query
      $query = 'SELECT * FROM {node} n '. $extra_join .' WHERE n.type = \'drigg\' AND '. $where_conditions .' n.status = 1 and n.promote = 0 '. $order_by;
    }
    #drupal_set_message($query);
+    global $is_front;
-   $output = '';
+    if ($is_front) {
+      $output = "<div class='drigg_published'";
-   // How many items to be shown in each page?
+          }
-   if (variable_get('drigg_hide_archived_menu', FALSE) && $action == 'archived') {
+    else {
-     $items_per_page = 150;
+      $output = "<div class='drigg_$action'";
-   }
+          }
-   else {
-     $items_per_page = variable_get('drigg_items_in_main_page', 20);
+   // How many items to be shown in each page?
-   }
+   if (variable_get('drigg_hide_archived_menu', FALSE) && $action == 'archived') {
+     $items_per_page = 150;
-   // Scan through the results
+   }
-   $result = pager_query(db_rewrite_sql($query), $items_per_page, 0, NULL, $query_vars);
+   else {
+     $items_per_page = variable_get('drigg_items_in_main_page', 20);
-   while ($node = db_fetch_object($result)) {
+   }
-     $something_there = TRUE;
-     if ($action == 'archived' && variable_get('drigg_hide_archived_menu', FALSE)) {
+   // Scan through the results
-       $output .= '<p> '. l($node->title, 'node/'. $node->nid) .'</p>';
+   $result = pager_query(db_rewrite_sql($query), $items_per_page, 0, NULL, $query_vars);
-     }
-     else {
+   while ($node = db_fetch_object($result)) {
-       $output .= node_view(node_load($node->nid), TRUE);
+     $something_there = TRUE;
-     }
+     if ($action == 'archived' && variable_get('drigg_hide_archived_menu', FALSE)) {
-   }
+       $output .= '<p> '. l($node->title, 'node/'. $node->nid) .'</p>';
-   if (!$something_there) {
+     }
-     drupal_set_message(t(variable_get('drigg_no_items_to_display', 'No scoop available!')));
+     else {
-   }
+       $output .= node_view(node_load($node->nid), TRUE);
+     }
-   $output .= theme('pager', NULL, variable_get('drigg_items_in_main_page', 20), 0);
+   }
+   if (!$something_there) {
-   return $output;
+     drupal_set_message(t(variable_get('drigg_no_items_to_display', 'No scoop available!')));
- }
+   }
- /**
+   $output .= theme('pager', NULL, variable_get('drigg_items_in_main_page', 20), 0);
-  * The standard "settings" function
+   $output .= "</div>";
-  *
- */
+   return $output;
- function drigg_ui_admin_settings() {
+ }
-   $form['drigg_home_name'] = array(
-     '#type' => 'textfield',
+ /**
-     '#title' => t('The path to the Drigg home page'),
+  * The standard "settings" function
-     '#size' => 20,
+  *
-     '#maxlength' => 20,
+ */
-     '#default_value' => variable_get('drigg_home_name', 'drigg_home'),
+ function drigg_ui_admin_settings() {
-   );
+   $form['drigg_home_name'] = array(
+     '#type' => 'textfield',
-   $form['drigg_hide_archived_menu'] = array(
+     '#title' => t('The path to the Drigg home page'),
-     '#type' => 'checkbox',
+     '#size' => 20,
-     '#title' => t('Hide the "archived" menu entry'),
+     '#maxlength' => 20,
-     '#default_value' => variable_get('drigg_hide_archived_menu', FALSE),
+     '#default_value' => variable_get('drigg_home_name', 'drigg_home'),
-     '#required' => FALSE,
+   );
-   );
+   $form['drigg_hide_archived_menu'] = array(
-   $form['drigg_items_in_main_page'] = array(
+     '#type' => 'checkbox',
-     '#type' => 'textfield',
+     '#title' => t('Hide the "archived" menu entry'),
-     '#title' => t('How many items should be shown in each page?'),
+     '#default_value' => variable_get('drigg_hide_archived_menu', FALSE),
-     '#size' => 5,
+     '#required' => FALSE,
-     '#maxlength' => 5,
+   );
-     '#required' => TRUE,
-     '#default_value' => variable_get('drigg_items_in_main_page', 20),
+   $form['drigg_items_in_main_page'] = array(
-    );
+     '#type' => 'textfield',
+     '#title' => t('How many items should be shown in each page?'),
-    $form['rss_icons'] = array(
+     '#size' => 5,
-     '#type' => 'fieldset',
+     '#maxlength' => 5,
-     '#title' => t('Look of the "feed" icons next to the category names in the category box'),
+     '#required' => TRUE,
-     '#collapsible' => FALSE,
+     '#default_value' => variable_get('drigg_items_in_main_page', 20),
-     '#collapsed' => FALSE,
+    );
-   );
+    $form['rss_icons'] = array(
-   $form['rss_icons']['drigg_look_rss_cat_all'] = array(
+     '#type' => 'fieldset',
-     '#type' => 'textfield',
+     '#title' => t('Look of the "feed" icons next to the category names in the category box'),
-     '#title' => t('Feed string for feeds pointing to all articles'),
+     '#collapsible' => FALSE,
-     '#size' => 25,
+     '#collapsed' => FALSE,
-     '#maxlength' => 255,
+   );
-     '#required' => FALSE,
-     '#default_value' => variable_get('drigg_look_rss_cat_all', ''),
+   $form['rss_icons']['drigg_look_rss_cat_all'] = array(
-   );
+     '#type' => 'textfield',
+     '#title' => t('Feed string for feeds pointing to all articles'),
-   $form['rss_icons']['drigg_look_rss_cat_pro'] = array(
+     '#size' => 25,
-     '#type' => 'textfield',
+     '#maxlength' => 255,
-     '#title' => t('Feed string for feeds pointing to promoted articles'),
+     '#required' => FALSE,
-     '#size' => 25,
+     '#default_value' => variable_get('drigg_look_rss_cat_all', ''),
-     '#maxlength' => 255,
+   );
-     '#required' => FALSE,
-     '#default_value' => variable_get('drigg_look_rss_cat_pro', ''),
+   $form['rss_icons']['drigg_look_rss_cat_pro'] = array(
-   );
+     '#type' => 'textfield',
+     '#title' => t('Feed string for feeds pointing to promoted articles'),
-   $form['drigg_no_items_to_display']=array(
+     '#size' => 25,
-     '#type' => 'textarea',
+     '#maxlength' => 255,
-     '#title' => t('Text to display if no scoop is available'),
+     '#required' => FALSE,
-     '#default_value' => variable_get('drigg_no_items_to_display', 'No scoop available!'),
+     '#default_value' => variable_get('drigg_look_rss_cat_pro', ''),
-     '#required' => FALSE,
+   );
-   );
+   $form['drigg_no_items_to_display']=array(
-   return system_settings_form($form);
+     '#type' => 'textarea',
- }
+     '#title' => t('Text to display if no scoop is available'),
+     '#default_value' => variable_get('drigg_no_items_to_display', 'No scoop available!'),
- ##################################################################
+     '#required' => FALSE,
- ####   MENU RENDERING FUNCTIONS - SECTIONS, ORDER, TOP, ARTICLE
+   );
- ##################################################################
+   return system_settings_form($form);
- /**
+ }
-  * Returns the "sections" menu.
-  * This is complicated by the fact that each category needs to be
+ ##################################################################
-  * linked to correctly. The code needs to check whether drigg is
+ ####   MENU RENDERING FUNCTIONS - SECTIONS, ORDER, TOP, ARTICLE
-  * actually handling the home page or not (the links will change).
+ ##################################################################
-  * Also, an "All" entry is added artificially.
-  * The function also has a $print_rss_links flag, in case the user wants
+ /**
-  * rss links (for which the formatting depends on the config string) next
+  * Returns the "sections" menu.
-  * to each category.
+  * This is complicated by the fact that each category needs to be
-  *
+  * linked to correctly. The code needs to check whether drigg is
-  * ALSO, things are complicated by the fact that according to the
+  * actually handling the home page or not (the links will change).
-  * query string, order and action need to be preserved. This means that
+  * Also, an "All" entry is added artificially.
-  * from /upcoming/newest/Business  a link to the "Community" category
+  * The function also has a $print_rss_links flag, in case the user wants
-  * will be /upcoming/newest/Community
+  * rss links (for which the formatting depends on the config string) next
-  *
+  * to each category.
-  * @param $print_rss_links
+  *
-  *   If TRUE, it will print the rss links
+  * ALSO, things are complicated by the fact that according to the
-  * @param $depth
+  * query string, order and action need to be preserved. This means that
-  *   Only choose the sections that are '$depth' level deep
+  * from /upcoming/newest/Business  a link to the "Community" category
-  *   counting from root, which has term_id of 0.
+  * will be /upcoming/newest/Community
-  *   For example, if we have
+  *
-  *   root -> technology -> CMS -> Drupal -> Drigg
+  * @param $print_rss_links
-  *                             -> Joomla
+  *   If TRUE, it will print the rss links
-  *        -> sports
+  * @param $depth
-  *   Then drigg_ui_sections(,0) will return the menu including 'technology' + 'sports'
+  *   Only choose the sections that are '$depth' level deep
-  *   drigg_ui_sections(,1) will return 'CMS' if we are viewing content of 'Drupal', 'Joomla' or 'Drigg'
+  *   counting from root, which has term_id of 0.
-  *   and so on....
+  *   For example, if we have
-  * @return
+  *   root -> technology -> CMS -> Drupal -> Drigg
-  *   The menu if there is a menu to display
+  *                             -> Joomla
-  *   Null if there is no sections available.
+  *        -> sports
- */
+  *   Then drigg_ui_sections(,0) will return the menu including 'technology' + 'sports'
- function drigg_ui_sections($print_rss_links = FALSE, $depth = 0) {
+  *   drigg_ui_sections(,1) will return 'CMS' if we are viewing content of 'Drupal', 'Joomla' or 'Drigg'
+  *   and so on....
-   $nr_of_sections = 0; //number of sections available.
+  * @return
-   $output = '';
+  *   The menu if there is a menu to display
+  *   Null if there is no sections available.
-   //section_id (term_id) array of all the ancestors of current 'term'
+ */
-   //being viewed.
+ function drigg_ui_sections($print_rss_links = FALSE, $depth = 0) {
-   $ancestor_arr = drigg_ui_ancestor_section_id_array();
+   $nr_of_sections = 0; //number of sections available.
-   //Now find the parent term_id of the sections we are generating.
+   $output = '';
-   if ($depth == 0) {
-     $parent_tid = 0;
+   //section_id (term_id) array of all the ancestors of current 'term'
-   }
+   //being viewed.
-   else {
+   $ancestor_arr = drigg_ui_ancestor_section_id_array();
-     $idx = count($ancestor_arr) - $depth - 1;
-     if ($idx >= 0)
+   //Now find the parent term_id of the sections we are generating.
-       $parent_tid = $ancestor_arr[$idx];
+   if ($depth == 0) {
-     else
+     $parent_tid = 0;
-       return NULL; //No sections available with this '$depth'
+   }
-   }
+   else {
-   $terms = drigg_section_list();
+     $idx = count($ancestor_arr) - $depth - 1;
+     if ($idx >= 0)
-   // You never know...
+       $parent_tid = $ancestor_arr[$idx];
-   if (count($terms) == 0) {
+     else
-     return NULL;
+       return NULL; //No sections available with this '$depth'
    }
+   $terms = drigg_section_list();
-   // HTML class for selected items
-   $selected_string = ' class="active" ';
+   // You never know...
-   $class_string = ' class="drigg-categories" ';
+   if (count($terms) == 0) {
+     return NULL;
-   // This will make the following code prettier
+   }
-   $p0 = arg(0);
-   $p1 = arg(1);
+   // HTML class for selected items
-   $p2 = arg(2);
+   $selected_string = ' class="active" ';
+   $class_string = ' class="drigg-categories" ';
-   // Little hack to have the "All" menu
+   // This will make the following code prettier
-   // This way, the database doesn't need to get dirty with
+   $p0 = arg(0);
-   // crap and it all "works"
+   $p1 = arg(1);
-   $terms['lookup'][ $parent_tid ]['name'] = t('All');
+   $p2 = arg(2);
-   $terms['lookup'][ $parent_tid ]['safe_name'] = 'All';
-   $terms['lookup'][ $parent_tid ]['description'] = t('All categories');
-   $terms['lookup'][ $parent_tid ]['weight'] = -100;
+   // Little hack to have the "All" menu
-   $terms['lookup'][ $parent_tid ]['tid'] = $tid;
+   // This way, the database doesn't need to get dirty with
-   array_unshift($terms['by_weight'], 0);
+   // crap and it all "works"
-   $terms['by_safe_name']['All'] = 0;
+   $terms['lookup'][ $parent_tid ]['name'] = t('All');
+   $terms['lookup'][ $parent_tid ]['safe_name'] = 'All';
+   $terms['lookup'][ $parent_tid ]['description'] = t('All categories');
-   if ($terms) {
+   $terms['lookup'][ $parent_tid ]['weight'] = -100;
-     foreach ($terms['by_weight'] as $tid ) {
+   $terms['lookup'][ $parent_tid ]['tid'] = $tid;
-       //only checks for terms that are children of $parent_tid
+   array_unshift($terms['by_weight'], 0);
-       if (($terms['lookup'][$tid]['parent_tid'] != $parent_tid ) /*&& ($tid != $parent_tid)*/) {
+   $terms['by_safe_name']['All'] = 0;
-         continue;
-       }
-       $category = $terms['lookup'][$tid]['safe_name'];
+   if ($terms) {
-       $selected = '';
+     foreach ($terms['by_weight'] as $tid ) {
+       //only checks for terms that are children of $parent_tid
-       //if $tid is an ancestor of current term being viewed.
+       if (($terms['lookup'][$tid]['parent_tid'] != $parent_tid ) /*&& ($tid != $parent_tid)*/) {
-       // then it should be highlighted (i.e. 'selected')
+         continue;
-       if ($ancestor_arr && in_array($tid, $ancestor_arr) && $tid != 0) {
+       }
-         $selected = $selected_string;
+       $category = $terms['lookup'][$tid]['safe_name'];
-       }
+       $selected = '';
-       // CASE "DEFAULT"
+       //if $tid is an ancestor of current term being viewed.
-       // This is the default, in case another page is being seen
+       // then it should be highlighted (i.e. 'selected')
-       if ($category == 'All') {
+       if ($ancestor_arr && in_array($tid, $ancestor_arr) && $tid != 0) {
-         if ($parent_tid == 0) { // Only have 'All' for the top categories
+         $selected = $selected_string;
-           $link = drigg_ui_home_url();
+       }
-         }
-       }
+       // CASE "DEFAULT"
-       else {
+       // This is the default, in case another page is being seen
-         $link = "$category";
+       if ($category == 'All') {
-       }
+         if ($parent_tid == 0) { // Only have 'All' for the top categories
+           $link = drigg_ui_home_url();
-       // Case #1: it's just /Category (other than /All). Check that the
+         }
-       // category is correct, and that there's nothing after the category's
+       }
-       // name
+       else {
-       if ( $category != 'All' && drigg_is_section_valid(arg(0)) && arg(1) == '') {
+         $link = "$category";
-         if ($category == arg(0)) {
+       }
-           $selected = $selected_string;
-         }
+       // Case #1: it's just /Category (other than /All). Check that the
-         $link = "$category";
+       // category is correct, and that there's nothing after the category's
-       }
+       // name
+       if ( $category != 'All' && drigg_is_section_valid(arg(0)) && arg(1) == '') {
+         if ($category == arg(0)) {
-       // Case #2: it's just 'node' or 'drigg_home'. In this case,
+           $selected = $selected_string;
-       // the URL is empty, and just show the roots
+         }
-       if ((arg(0) == 'node' || arg(0) == variable_get('drigg_home_name', 'drigg_home')) && arg(1) == '' ) {
+         $link = "$category";
-         if ($category == 'All' ) {
+       }
-           $selected = $selected_string;
-           $link = drigg_ui_home_url();
-         }
+       // Case #2: it's just 'node' or 'drigg_home'. In this case,
-         else {
+       // the URL is empty, and just show the roots
-           $link = "$category";
+       if ((arg(0) == 'node' || arg(0) == variable_get('drigg_home_name', 'drigg_home')) && arg(1) == '' ) {
-         }
+         if ($category == 'All' ) {
-       }
+           $selected = $selected_string;
+           $link = drigg_ui_home_url();
+         }
-       // Case #3: it's viewing an article. In this case,
+         else {
-       // get the article's category
+           $link = "$category";
-       if (arg(0) == 'node' && arg(1) != '' && is_numeric(arg(1))) {
+         }
-         $n = node_load(arg(1));
+       }
-         if ($n->safe_section == $category) {
-           $selected = $selected_string;
-         }
+       // Case #3: it's viewing an article. In this case,
-         if ($category == 'All' ) {
+       // get the article's category
-           $link = drigg_ui_home_url();
+       if (arg(0) == 'node' && arg(1) != '' && is_numeric(arg(1))) {
-         }
+         $n = node_load(arg(1));
-         else {
+         if ($n->safe_section == $category) {
-           $link = "$category";
+           $selected = $selected_string;
          }
-       }
+         if ($category == 'All' ) {
+           $link = drigg_ui_home_url();
-       // Case #4: it's viewing a list. This is the most complex case.
+         }
-       if ($p0 == 'published' || $p0 == 'upcoming' || $p0 == 'archived') {
+         else {
+           $link = "$category";
-         if ($p1 == '') {
+         }
-           $p1 = 'newest';
+       }
-         }
+       // Case #4: it's viewing a list. This is the most complex case.
-         // *** IF ***
+       if ($p0 == 'published' || $p0 == 'upcoming' || $p0 == 'archived') {
-         // If it's looking for the "published" and "newest", then
-         // the link can be (and will be) shortened to /Category
+         if ($p1 == '') {
-         // (with the usual druligg_home trick)
+           $p1 = 'newest';
-         if ($p0 == 'published' && $p1 == 'newest' ) {
+         }
-           if ($category == 'All' ) {
-             $link = drigg_ui_home_url();
+         // *** IF ***
-           }
+         // If it's looking for the "published" and "newest", then
-           else {
+         // the link can be (and will be) shortened to /Category
-             $link = "$category";
+         // (with the usual druligg_home trick)
-           }
+         if ($p0 == 'published' && $p1 == 'newest' ) {
+           if ($category == 'All' ) {
-         // *** OTHERWISE ***
+             $link = drigg_ui_home_url();
-         // Otherwise, the link is just the straight "long" version
+           }
-         }
+           else {
-         else {
+             $link = "$category";
+           }
-           // Last minute change, so that it never tries to link
-           // to upcoming/newest/All
+         // *** OTHERWISE ***
-           if ($category == 'All') {
+         // Otherwise, the link is just the straight "long" version
-             $category = '';
+         }
-           }
+         else {
-           // It's a link!
+           // Last minute change, so that it never tries to link
-           $link = "$p0/$p1/$category";
+           // to upcoming/newest/All
-         }
+           if ($category == 'All') {
-         if ($category == $p2) {
+             $category = '';
-           $selected = $selected_string;
+           }
-         }
-       }
+           // It's a link!
+           $link = "$p0/$p1/$category";
+         }
-       // NOW that link is set, I will add the Drupal's root to it
+         if ($category == $p2) {
-       // in case there is one
+           $selected = $selected_string;
-       //
+         }
-       #$bp = base_path();
+       }
-       #if ($bp != '/') {
-       #  $link = substr($bp, 0, -1) . $link;
-       #}
+       // NOW that link is set, I will add the Drupal's root to it
+       // in case there is one
-       // Work out the links for the RSS feed for that category - if
+       //
-       // present!
+       #$bp = base_path();
-       $rss_links = ' ';
+       #if ($bp != '/') {
-       if ($print_rss_links) {
+       #  $link = substr($bp, 0, -1) . $link;
-         $all_string = variable_get('drigg_look_rss_cat_all', '');
+       #}
-         $cat_string = variable_get('drigg_look_rss_cat_pro', '');
+       // Work out the links for the RSS feed for that category - if
-         if ($all_string || $cat_string) {
+       // present!
-           $rss_links .= '<span class="rss">';
+       $rss_links = ' ';
-         }
+       if ($print_rss_links) {
+         $all_string = variable_get('drigg_look_rss_cat_all', '');
-         $options = array(
+         $cat_string = variable_get('drigg_look_rss_cat_pro', '');
-           'query' => NULL,
-           'fragment' => NULL,
+         if ($all_string || $cat_string) {
-           'absolute' => FALSE,
+           $rss_links .= '<span class="rss">';
-           'html' => TRUE,
+         }
-         );
+         $options = array(
-         if ($all_string) {
+           'query' => NULL,
-           $rss_links .= l($all_string, "feed/all/$category", $options);
+           'fragment' => NULL,
-         }
+           'absolute' => FALSE,
+           'html' => TRUE,
-         if ($cat_string) {
+         );
-           $rss_links .= l($cat_string, "feed/published/$category", $options);
-         }
+         if ($all_string) {
+           $rss_links .= l($all_string, "feed/all/$category", $options);
-         if ($all_string || $cat_string) {
+         }
-           $rss_links .= '</span>';
-         }
+         if ($cat_string) {
-       }
+           $rss_links .= l($cat_string, "feed/published/$category", $options);
+         }
-       // Finally shows the output
-       #$output .= '<li><a '. $selected .' href="'. $link .'">'. $terms['lookup'][$tid]['name'] . $rss_links .'</a></li>';
+         if ($all_string || $cat_string) {
-       $output .= '<li><a '. $selected .' href="'. url($link) .'" title="'. t($terms['lookup'][$tid]['description']) .'"><span>'. t($terms['lookup'][$tid]['name']) .'</span></a>'. $rss_links .'</li>';
+           $rss_links .= '</span>';
-       $nr_of_sections ++;
+         }
-     }
+       }
-   }
-   //return data only if there is some sections available
+       // Finally shows the output
-   if ($nr_of_sections >= 1) {
+       #$output .= '<li><a '. $selected .' href="'. $link .'">'. $terms['lookup'][$tid]['name'] . $rss_links .'</a></li>';
-     return "<ul ". $class_string ." >$output</ul>";
+       $output .= '<li><a '. $selected .' href="'. url($link) .'" title="'. t($terms['lookup'][$tid]['description']) .'"><span>'. t($terms['lookup'][$tid]['name']) .'</span></a>'. $rss_links .'</li>';
-   }
+       $nr_of_sections ++;
-   else {
+     }
-     return NULL;
+   }
-   }
+   //return data only if there is some sections available
- }
+   if ($nr_of_sections >= 1) {
+     return "<ul ". $class_string ." >$output</ul>";
- function drigg_ui_home_url() {
+   }
-   if (variable_get('site_frontpage', '') == variable_get('drigg_home_name', 'drigg_home')) {
+   else {
-     return '';
+     return NULL;
    }
-   else {
+ }
-     return variable_get('drigg_home_name', 'drigg_home');
-   }
+ function drigg_ui_home_url() {
+   if (variable_get('site_frontpage', '') == variable_get('drigg_home_name', 'drigg_home')) {
- }
+     return '';
+   }
- /**
+   else {
-  * Returns the "order" menu.
+     return variable_get('drigg_home_name', 'drigg_home');
-  * This menu depends on its context: it will be different for "published" and
+   }
-  * "upcoming" | "archived". As for the "sections" menu, the current
-  * path (and category) is maintained.
+ }
-  * However, it cannot be done all the time: for example, you cannot go from
-  * upcoming/mostpopular/Business to published/mostpopular/Business because
+ /**
-  * "published" doesn't have "mostpopular" as an sorting criteria!
+  * Returns the "order" menu.
-  * So yes, this is a tricky one too. This is why it's basically impossible
+  * This menu depends on its context: it will be different for "published" and
-  * to write a Digg/Pligg clone without a _very_ specialised module.
+  * "upcoming" | "archived". As for the "sections" menu, the current
-  *
+  * path (and category) is maintained.
-  * @return
+  * However, it cannot be done all the time: for example, you cannot go from
-  *   The actual menu
+  * upcoming/mostpopular/Business to published/mostpopular/Business because
- */
+  * "published" doesn't have "mostpopular" as an sorting criteria!
- function drigg_ui_order_menu() {
+  * So yes, this is a tricky one too. This is why it's basically impossible
+  * to write a Digg/Pligg clone without a _very_ specialised module.
-   // This is here simply to make the extraction of translation strings
+  *
-   // possible (because of that $t(item) )
+  * @return
-   t('newest');
+  *   The actual menu
-   t('top24h');
+ */
-   t('top7days');
+ function drigg_ui_order_menu() {
-   t('top30days');
-   t('top365days');
+   // This is here simply to make the extraction of translation strings
+   // possible (because of that $t(item) )
-   $output = '';
+   t('newest');
-   $selected_string = ' class="active" ';
+   t('top24h');
+   t('top7days');
-   $p0 = arg(0);
+   t('top30days');
-   $p1 = arg(1);
+   t('top365days');
-   $p2 = arg(2);
+   $output = '';
-   // This will make sure that even the home page will work OK
+   $selected_string = ' class="active" ';
-   if ((arg(0)=='node' || arg(0) == variable_get('drigg_home_name', 'drigg_home') ) && arg(1) == '' ) {
-     $p0 = 'published';
+   $p0 = arg(0);
-     $p1 = 'newest';
+   $p1 = arg(1);
-   }
+   $p2 = arg(2);
-   // This will make sure that even /Section works fine
+   // This will make sure that even the home page will work OK
-   if (drigg_is_section_valid( $p0 ) && $p1 == '' ) {
+   if ((arg(0)=='node' || arg(0) == variable_get('drigg_home_name', 'drigg_home') ) && arg(1) == '' ) {
      $p0 = 'published';
      $p1 = 'newest';
-     $p2 = arg(0);
+   }
-   }
+   // This will make sure that even /Section works fine
-   if (( $p0 == 'published' || $p0 == 'upcoming' || $p0  == 'archived') && $p1 == '') {
+   if (drigg_is_section_valid( $p0 ) && $p1 == '' ) {
-     $p1 = 'newest';
+     $p0 = 'published';
-   }
+     $p1 = 'newest';
+     $p2 = arg(0);
+   }
-   // This avoids doing anything if the URL is incorrect
-   if ($p0 != 'published' && $p0 != 'upcoming' && $p0 != 'archived') {
+   if (( $p0 == 'published' || $p0 == 'upcoming' || $p0  == 'archived') && $p1 == '') {
-     return '';
+     $p1 = 'newest';
    }
-   // Set the items in the menu
-   if ($p0 == 'published') {
+   // This avoids doing anything if the URL is incorrect
-     $items = array('newest', 'top24h', 'top7days', 'top30days', 'top365days');
+   if ($p0 != 'published' && $p0 != 'upcoming' && $p0 != 'archived') {
-   }
+     return '';
-   else {
+   }
-     $items = array('newest', 'oldest', 'mostpopular', 'leastpopular');
-   }
+   // Set the items in the menu
+   if ($p0 == 'published') {
-   // For each item, prints it, but with a twist...
+     $items = array('newest', 'top24h', 'top7days', 'top30days', 'top365days');
-   // At this point, things are "normalised". So, no matter what,
+   }
-   // the format is for example published/top24h/Community or
+   else {
-   // published/top24h
+     $items = array('newest', 'oldest', 'mostpopular', 'leastpopular');
-   foreach ($items as $item) {
+   }
-     $selected = '';
+   // For each item, prints it, but with a twist...
-     // Mark it as selected if it's the same as the current one
+   // At this point, things are "normalised". So, no matter what,
-     if ($item == $p1) {
+   // the format is for example published/top24h/Community or
-       $selected = $selected_string;
+   // published/top24h
-     }
+   foreach ($items as $item) {
+     $selected = '';
-     // TWIST: if the item would point to published/newest, then make the
-     // url simply /Category
+     // Mark it as selected if it's the same as the current one
-     if ($p0 == 'published' && $item == 'newest') {
+     if ($item == $p1) {
-       $link = "$p2";
+       $selected = $selected_string;
-       if ($link == '') {
+     }
-         $link = drigg_ui_home_url();
-       }
+     // TWIST: if the item would point to published/newest, then make the
-     }
+     // url simply /Category
-     else {
+     if ($p0 == 'published' && $item == 'newest') {
-       $link = "$p0";
+       $link = "$p2";
-       if ($item != '') {
+       if ($link == '') {
-         $link .= "/$item";
+         $link = drigg_ui_home_url();
        }
-       if ($p2 != '') {
+     }
-         $link .= "/$p2";
+     else {
-       }
+       $link = "$p0";
-     }
+       if ($item != '') {
+         $link .= "/$item";
-     // NOW that link is set, I will add the Drupal's root to it
+       }
-     // in case there is one
+       if ($p2 != '') {
-     //
+         $link .= "/$p2";
-     #$bp = base_path();
+       }
-     #if ($bp != '/') {
+     }
-     #  $link = substr($bp, 0, -1) . $link;
-     #}
+     // NOW that link is set, I will add the Drupal's root to it
+     // in case there is one
-     $output .= '<li '. $selected .'><a '. $selected .' href="'. url($link) .'">'. t($item) .'</a></li>';
+     //
+     #$bp = base_path();
-   }
+     #if ($bp != '/') {
-   return "<ul class=\"drigg-order\">$output</ul>";
+     #  $link = substr($bp, 0, -1) . $link;
+     #}
- }
+     $output .= '<li '. $selected .'><a '. $selected .' href="'. url($link) .'">'. t($item) .'</a></li>';
- /**
+   }
-  * Returns the top menu ("published", "upcoming", "archived")
+   return "<ul class=\"drigg-order\">$output</ul>";
-  * This function ALSO preservs all of the links, whenever possible.
-  *
+ }
-  * @return
-  *   The top menu
- */
+ /**
- function drigg_ui_type_menu() {
+  * Returns the top menu ("published", "upcoming", "archived")
-   $output = '';
+  * This function ALSO preservs all of the links, whenever possible.
-   $selected_string = ' class="active" ';
+  *
+  * @return
-   $p0 = arg(0);
+  *   The top menu
-   $p1 = arg(1);
+ */
-   $p2 = arg(2);
+ function drigg_ui_type_menu() {
+   $output = '';
-   // This will make sure that even the home page will work OK
+   $selected_string = ' class="active" ';
-   if ((arg(0) == 'node' || arg(0)== variable_get('drigg_home_name', 'drigg_home') ) && arg(1) == '' ) {
-     $p0 = 'published';
+   $p0 = arg(0);
-     $p1 = 'newest';
+   $p1 = arg(1);
-   }
+   $p2 = arg(2);
-   // This will make sure that even /Section works fine
+   // This will make sure that even the home page will work OK
-   if (drigg_is_section_valid( $p0 ) && $p1 == '' ) {
+   if ((arg(0) == 'node' || arg(0)== variable_get('drigg_home_name', 'drigg_home') ) && arg(1) == '' ) {
      $p0 = 'published';
      $p1 = 'newest';
-     $p2 = arg(0);
+   }
-   }
+   // This will make sure that even /Section works fine
-   // This avoids doing anything if the URL is incorrect
+   if (drigg_is_section_valid( $p0 ) && $p1 == '' ) {
-   if ($p0 != 'published' && $p0 != 'upcoming' && $p0 != 'archived') {
+     $p0 = 'published';
-     return '';
+     $p1 = 'newest';
-   }
+     $p2 = arg(0);
+   }
-   // Hide the "archived" menu entry, UNLESS you _ARE_ in "archived"
-   if (variable_get('drigg_hide_archived_menu', FALSE) && $p0 != 'archived') {
+   // This avoids doing anything if the URL is incorrect
-     $items = array('published', 'upcoming');
+   if ($p0 != 'published' && $p0 != 'upcoming' && $p0 != 'archived') {
-   }
+     return '';
-   else {
+   }
-     $items = array('published', 'upcoming', 'archived');
-   }
+   // Hide the "archived" menu entry, UNLESS you _ARE_ in "archived"
+   if (variable_get('drigg_hide_archived_menu', FALSE) && $p0 != 'archived') {
-   // For each item, prints it, but with a twist...
+     $items = array('published', 'upcoming');
-   // At this point, things are "normalised". So, no matter what,
+   }
-   // the format is for example published/top24h/Community or
+   else {
-   // published/top24h
+     $items = array('published', 'upcoming', 'archived');
-   foreach ($items as $item ) {
+   }
-     $selected = '';
+   // For each item, prints it, but with a twist...
-     $pp1 = $p1;
+   // At this point, things are "normalised". So, no matter what,
+   // the format is for example published/top24h/Community or
-     // Mark it as selected if it's the same as the current one
+   // published/top24h
-     if ($item == $p0) {
+   foreach ($items as $item ) {
-       $selected = $selected_string;
+     $selected = '';
-     }
+     $pp1 = $p1;
-     // Last minute change: you can't jump from 'published/top24h' to
-     // 'upcoming/24h'!!! So, if things are not right, fix them on the
+     // Mark it as selected if it's the same as the current one
-     // spot. From now on, $pp1 needs to be considered, rather than
+     if ($item == $p0) {
-     // $p1
+       $selected = $selected_string;
-     //
+     }
-     if ($item == 'upcoming' || $item == 'archived') {
-       if ($p1 != 'oldest' && $p1 != 'mostpopular' && $p1 != 'leastpopular') {
+     // Last minute change: you can't jump from 'published/top24h' to
-         $pp1 = 'newest';
+     // 'upcoming/24h'!!! So, if things are not right, fix them on the
-       }
+     // spot. From now on, $pp1 needs to be considered, rather than
-     }
+     // $p1
-     if ($item == 'published') {
+     //
-       if ($p1 != 'top7days' && $p1 != 'top24h' && $p1 != 'top30days' && $p1 != 'top365days') {
+     if ($item == 'upcoming' || $item == 'archived') {
-         $pp1 = 'newest';
+       if ($p1 != 'oldest' && $p1 != 'mostpopular' && $p1 != 'leastpopular') {
-       }
+         $pp1 = 'newest';
-     }
+       }
+     }
-     // If the link is going to be /published/newest, then change it
+     if ($item == 'published') {
-     // last minute to just /
+       if ($p1 != 'top7days' && $p1 != 'top24h' && $p1 != 'top30days' && $p1 != 'top365days') {
-     if ($item == 'published' && $pp1 == 'newest' && $p2 == '') {
+         $pp1 = 'newest';
-       $link = drigg_ui_home_url();
+       }
-     // Otherwise, business as normal
+     }
-     }
-     else {
+     // If the link is going to be /published/newest, then change it
-       $link = "$item";
+     // last minute to just /
-       if ($pp1 != '') {
+     if ($item == 'published' && $pp1 == 'newest' && $p2 == '') {
-         $link .= "/$pp1";
+       $link = drigg_ui_home_url();
-       }
+     // Otherwise, business as normal
-       if ($p2 != '') {
+     }
-         $link .= "/$p2";
+     else {
-       }
+       $link = "$item";
-       #$link="/$item" .  $pp1 != '' ? "/$pp1/$p2" : '';
+       if ($pp1 != '') {
-     }
+         $link .= "/$pp1";
+       }
-     // NOW that link is set, I will add the Drupal's root to it
+       if ($p2 != '') {
-     // in case there is one
+         $link .= "/$p2";
-     //
+       }
-     #$bp = base_path();
+       #$link="/$item" .  $pp1 != '' ? "/$pp1/$p2" : '';
-     #if ($bp != '/') {
+     }
-     #  $link = substr($bp, 0, -1) . $link;
-     #}
+     // NOW that link is set, I will add the Drupal's root to it
+     // in case there is one
-     $output .= "<li $selected><a ". $selected .' href="'. url($link) .'">'. t($item) .'</a></li>';
+     //
+     #$bp = base_path();
-   }
+     #if ($bp != '/') {
-   return "<ul class=\"drigg-viewtype\">$output</ul>";
+     #  $link = substr($bp, 0, -1) . $link;
+     #}
- }
+     $output .= "<li $selected><a ". $selected .' href="'. url($link) .'">'. t($item) .'</a></li>';
- /**
+   }
-  * Returns the article menu (who voted, comments, related links)
+   return "<ul class=\"drigg-viewtype\">$output</ul>";
-  * THis will most likely get called by node-drigg.tpl.php
-  *
+ }
-  * @return
-  *   The menu
-  * TODO: IMPORTANT!!! This needs to check if drigg_users AND drigg_sl are
+ /**
-  *       set, and base the creation of the menu according to the existence
+  * Returns the article menu (who voted, comments, related links)
-  *       of those modules AND the user input (which needs to be added for
+  * THis will most likely get called by node-drigg.tpl.php
-  *       "who voted")
+  *
- */
+  * @return
- function drigg_ui_article_menu() {
+  *   The menu
+  * TODO: IMPORTANT!!! This needs to check if drigg_users AND drigg_sl are
-   if (! (arg(0) == 'node' && is_numeric(arg(1))) ) {
+  *       set, and base the creation of the menu according to the existence
-     return '';
+  *       of those modules AND the user input (which needs to be added for
-   }
+  *       "who voted")
+ */
-   // List of links at the bottom of the article
+ function drigg_ui_article_menu() {
-   $items = array( '' => t('Comments'), );
+   if (! (arg(0) == 'node' && is_numeric(arg(1))) ) {
+     return '';
-   if (module_exists('drigg_users') && variable_get('drigg_users_tab_menu_show', TRUE)) {
+   }
-     $items['who_voted'] = t('Who voted');
-   }
+   // List of links at the bottom of the article
+   $items = array( '' => t('Comments'), );
-   // Only add related links if the admin wants it
-   if (module_exists('drigg_rl') && variable_get('drigg_rl_tab_menu_show', TRUE)) {
-     $items['related_links'] = t('Related links');
+   if (module_exists('drigg_users') && variable_get('drigg_users_tab_menu_show', TRUE)) {
-   };
+     $items['who_voted'] = t('Who voted');
+   }
-  // For each item, prints it
-   foreach ($items as $item => $item_value) {
+   // Only add related links if the admin wants it
-     $active_style = '';
+   if (module_exists('drigg_rl') && variable_get('drigg_rl_tab_menu_show', TRUE)) {