From 906addf865ccab2c98d5035fc6c1b10d43004dfe Mon Sep 17 00:00:00 2001
From: Fabio Comuni <fabrix.xm@gmail.com>
Date: Mon, 10 Oct 2011 17:01:14 +0200
Subject: [PATCH] common_tabs in notifications.php

---
 mod/notifications.php  | 30 +++++++++++++++++++++---------
 view/notifications.tpl |  8 +-------
 2 files changed, 22 insertions(+), 16 deletions(-)

diff --git a/mod/notifications.php b/mod/notifications.php
index 6d78ea0242..52ce90e149 100644
--- a/mod/notifications.php
+++ b/mod/notifications.php
@@ -62,7 +62,18 @@ function notifications_content(&$a) {
 		notice( t('Permission denied.') . EOL);
 		return;
 	}
-
+	
+	$notif_tpl = get_markup_template('notifications.tpl');
+	
+	$tabs = array(
+		'net' 	=> array('label' => t('Network'), 'url'=>$a->get_baseurl().'/notifications/network', 'sel'=>''),
+		'home' 	=> array('label' => t('Home'), 'url'=>$a->get_baseurl().'/notifications/home', 'sel'=>''),
+		'intros'=> array('label' => t('Introductions'), 'url'=>$a->get_baseurl().'/notifications/intros', 'sel'=>''),
+		'msg'	=> array('label' => t('Messages'), 'url'=>$a->get_baseurl().'/message', 'sel'=>''),
+	);
+	
+	
+	
 	$o = '';
 	
 	if( (($a->argc > 1) && ($a->argv[1] == 'intros')) || (($a->argc == 1))) {
@@ -72,7 +83,7 @@ function notifications_content(&$a) {
 		else
 			$sql_extra = " AND `ignore` = 0 ";
 		
-		$notif_tpl = get_markup_template('notifications.tpl');
+		
 		
 		$notif_content .= '<a href="' . ((strlen($sql_extra)) ? 'notifications/intros/all' : 'notifications/intros' ) . '" id="notifications-show-hide-link" >'
 			. ((strlen($sql_extra)) ? t('Show Ignored Requests') : t('Hide Ignored Requests')) . '</a></div>' . "\r\n";
@@ -164,11 +175,12 @@ function notifications_content(&$a) {
 			}
 		}
 		else
-			info( t('No notifications.') . EOL);
-		
+			$notif_content .= t('<p>No notifications.</p>');
+			
+		$tabs['intros']['sel']='active';
 		$o .= replace_macros($notif_tpl,array(
 			'$notif_content' => $notif_content,
-			'$activetab' => 'intros'
+			'$tabs' => $tabs,
 		));
 		
 		$o .= paginate($a);
@@ -176,7 +188,6 @@ function notifications_content(&$a) {
 				
 	} else if (($a->argc > 1) && ($a->argv[1] == 'network')) {
 		
-		$notif_tpl = get_markup_template('notifications.tpl');
 		
 		$r = q("SELECT `item`.`id`,`item`.`parent`, `item`.`verb`, `item`.`author-name`, 
 				`item`.`author-link`, `item`.`author-avatar`, `item`.`created`, `item`.`object` as `object`, 
@@ -251,14 +262,14 @@ function notifications_content(&$a) {
 			$notif_content = t('Nothing new!');
 		}
 		
+		$tabs['net']['sel']='active';
 		$o .= replace_macros($notif_tpl,array(
 			'$notif_content' => $notif_content,
-			'$activetab' => 'network'
+			'$tabs' => $tabs,
 		));
 		
 	} else if (($a->argc > 1) && ($a->argv[1] == 'home')) {
 		
-		$notif_tpl = get_markup_template('notifications.tpl');
 		
 		$r = q("SELECT `item`.`id`,`item`.`parent`, `item`.`verb`, `item`.`author-name`, 
 				`item`.`author-link`, `item`.`author-avatar`, `item`.`created`, `item`.`object` as `object`, 
@@ -326,9 +337,10 @@ function notifications_content(&$a) {
 			$notif_content = t('Nothing new!');
 		}
 		
+		$tabs['home']['sel'] = 'active';
 		$o .= replace_macros($notif_tpl,array(
 			'$notif_content' => $notif_content,
-			'$activetab' => 'home'
+			'$tabs' => $tabs,
 		));
 	}
 
diff --git a/view/notifications.tpl b/view/notifications.tpl
index 24f78f37ef..487f8b480c 100644
--- a/view/notifications.tpl
+++ b/view/notifications.tpl
@@ -1,13 +1,7 @@
 
 <h1>Notifications</h1>
 
-<p id="notification-listing-desc">
-	<a href="/notifications/network" class="button tabs {{if $activetab==network}}active{{endif}}">Network</a>
-	<a href="/notifications/home" class="button tabs {{if $activetab==home}}active{{endif}}">Home</a>
-	<a href="/notifications/intros" class="button tabs {{if $activetab==intros}}active{{endif}}">Introductions</a>
-	<a href="/message" class="button tabs">Messages</a>
-</p>
-<div class="notification-listing-end"></div>
+{{ inc common_tabs.tpl }}{{ endinc }}
 
 <div class="notif-network-wrapper">
 	$notif_content