Allowing plugin authors to override the default tabs (filters) displayed on the conte...

git-svn-id: http://code.elgg.org/elgg/trunk@6542 36083f99-b078-4883-b0ff-0f9b5a30f544

diff --git a/views/default/page_elements/content_header.php b/views/default/page_elements/content_header.php
index 825d6a1edc4e00fdec191cf81e3953fb9632c61c..79e1ce279d0e35a0b70a57daa80fd7056914852e 100644 (file)
--- a/views/default/page_elements/content_header.php
+++ b/views/default/page_elements/content_header.php
@@ -25,58 +25,48 @@ $filter_context = $vars['context'];
 // get the object type
 $type = $vars['type'];
 
-// tons of empty strings.
-$mine_selected = $all_selected = $friend_selected =
-$new_button = $action_buttons = $filter_content = '';
+// create an empty string to start with
+$new_button = '';
 
-$all_title = elgg_echo('all');
-$mine_title = elgg_echo('mine');
-$friend_title = elgg_echo('friends');
+// generate a list of default tabs
+$default_tabs = array(
+       'all' => array(
+               'title' => elgg_echo('all'),
+               'url' => (isset($vars['all_link'])) ? $vars['all_link'] : "{$vars['url']}mod/$type/all.php",
+               'selected' => ($filter_context == 'everyone'),
+       ),
+       'mine' => array(
+               'title' => elgg_echo('mine'),
+               'url' => (isset($vars['mine_link'])) ? $vars['mine_link'] : "{$vars['url']}pg/$type/$username",
+               'selected' => ($filter_context == 'mine'),
+       ),
+       'friend' => array(
+               'title' => elgg_echo('friends'),
+               'url' => (isset($vars['friend_link'])) ? $vars['friend_link'] : "{$vars['url']}pg/$type/$username/friends",
+               'selected' => ($filter_context == 'friends'),
+       ),
+);
 
-if (!($page_owner instanceof ElggGroup)){
-       if($filter_context == 'everyone') {
-               $all_selected = "class = 'selected'";
-       }
-       if($filter_context == 'mine') {
-               $mine_selected = "class = 'selected'";
-       }
-       if($filter_context == 'friends') {
-               $friend_selected = "class = 'selected'";
-       }
-}
-
-// allow plugins to override default page handlers
-// @todo switch this over to proper page handling style
-$all_link = (isset($vars['all_link'])) ? $vars['all_link'] : "{$vars['url']}mod/$type/all.php";
-$mine_link = (isset($vars['mine_link'])) ? $vars['mine_link'] : "{$vars['url']}pg/$type/$username";
-$friend_link = (isset($vars['friend_link'])) ? $vars['friend_link'] : "{$vars['url']}pg/$type/$username/friends";
-$new_link = (isset($vars['new_link'])) ? $vars['new_link'] : "{$CONFIG->wwwroot}pg/$type/$username/new";
+// determine if using default or overwritten tabs
+$tabs = (isset($vars['tabs'])) ? $vars['tabs'] : $default_tabs;
+$tab_list = elgg_view('navigation/tabs', array('tabs' => $tabs));
 
 $title = elgg_echo($type);
 $title = '<div class="content_header_title">' . elgg_view_title($title) . '</div>';
 
-$tabs = <<<EOT
-       <div class="elgg_horizontal_tabbed_nav margin_top">
-               <ul>
-                       <li $all_selected><a href="$all_link">$all_title</a></li>
-                       <li $mine_selected><a href="$mine_link">$mine_title</a></li>
-                       <li $friend_selected><a href="$friend_link">$friend_title</a></li>
-               </ul>
-       </div>
-EOT;
-
-// must be logged in to see the filter menu and any action buttons
+// must be logged in to see any action buttons
 if (isloggedin()) {
        // only show the new button when not on the add form.
        // hide the tabs when on the add form.
-       // @todo remove the hard coded reference to the videolist plugin
        if ($filter_context == 'action') {
-               $tabs = '';
+               $tab_list = '';
        } else {
+               // @todo remove the hard coded reference to the videolist plugin
                if(get_context() == "videolist"){
                        $video_link = $CONFIG->wwwroot . "pg/videolist/browse/$username/";
-                       $new_button .= "<a href=\"{$video_link}\" class='action_button'>" . elgg_echo('videolist:browsemenu') . '</a>';
+                       $new_button = "<a href=\"{$video_link}\" class='action_button'>" . elgg_echo('videolist:browsemenu') . '</a>';
                }else{
+                       $new_link = (isset($vars['new_link'])) ? $vars['new_link'] : "{$CONFIG->wwwroot}pg/$type/$username/new";
                        $new_button = "<a href=\"{$new_link}\" class='action_button'>" . elgg_echo($type . ':new') . '</a>';
                }
                $new_button = "<div class='content_header_options'>$new_button</div>";
@@ -85,21 +75,14 @@ if (isloggedin()) {
        // also hide the tabs if in a group context (ie, listing groups) or
        // when viewing tools belonging to a group
        if (get_context() == 'groups' || $page_owner instanceof ElggGroup) {
-               $tabs = '';
+               $tab_list = '';
        }
-} else {
-       // only show logged out users the all tab
-       $page_filter = <<<EOT
-               <div class="elgg_horizontal_tabbed_nav margin_top">
-                       <ul>
-                               <li $all_selected><a href="$all_link">$all_title</a></li>
-                       </ul>
-               </div>
-EOT;
 }
-?>
+
+echo <<<HTML
 <div id="content_header" class="clearfloat">
-       <?php echo $title . $new_button; ?>
+       $title $new_button
 </div>
-<?php echo $tabs; ?>
+HTML;
 
+echo $tab_list;