From bd54358aa6bfe90bfbc8aed7baf3a8110d6e45ba Mon Sep 17 00:00:00 2001
From: marijus <mario@mariovavti.com>
Date: Sat, 20 Sep 2014 16:31:56 +0200
Subject: show bookmarks menu only if we have any bookmarks already

---
 include/conversation.php |  4 +++-
 include/menu.php         | 10 ++++++++++
 include/nav.php          |  9 +++++++--
 3 files changed, 20 insertions(+), 3 deletions(-)

(limited to 'include')

diff --git a/include/conversation.php b/include/conversation.php
index 05ae72d61..8f1767482 100644
--- a/include/conversation.php
+++ b/include/conversation.php
@@ -1556,7 +1556,9 @@ function profile_tabs($a, $is_owner=False, $nickname=Null){
 		);
 	}
 
-	if($is_owner) {
+	require_once('include/menu.php');
+	$has_bookmarks = menu_list_count(local_user(),'',MENU_BOOKMARK) + menu_list_count(local_user(),'',MENU_SYSTEM|MENU_BOOKMARK);
+	if($is_owner && $has_bookmarks) {
 		$tabs[] = array(
 			'label' => t('Bookmarks'),
 			'url'	=> $a->get_baseurl() . '/bookmarks',
diff --git a/include/menu.php b/include/menu.php
index 4b0a11f10..8997d2e39 100644
--- a/include/menu.php
+++ b/include/menu.php
@@ -124,7 +124,17 @@ function menu_list($channel_id, $name = '', $flags = 0) {
 	return $r;
 }
 
+function menu_list_count($channel_id, $name = '', $flags = 0) {
 
+	$sel_options = '';
+	$sel_options .= (($name) ? " and menu_name = '" . protect_sprintf(dbesc($name)) . "' " : '');
+	$sel_options .= (($flags) ? " and menu_flags = " . intval($flags) . " " : '');
+
+	$r = q("select count(*) as total from menu where menu_channel_id = %d $sel_options",
+		intval($channel_id)
+	);
+	return $r[0]['total'];
+}
 
 function menu_edit($arr) {
 
diff --git a/include/nav.php b/include/nav.php
index 799faf5ce..c6bc4d293 100644
--- a/include/nav.php
+++ b/include/nav.php
@@ -110,9 +110,14 @@ EOT;
 			$nav['usermenu'][] = Array('chat/' . $channel['channel_address'],t('Chat'),"",t('Your chatrooms'));
 		}
 
-		$nav['usermenu'][] = Array('bookmarks', t('Bookmarks'), "", t('Your bookmarks'));
+		require_once('include/menu.php');
+		$has_bookmarks = menu_list_count(local_user(),'',MENU_BOOKMARK) + menu_list_count(local_user(),'',MENU_SYSTEM|MENU_BOOKMARK);
+		if($has_bookmarks) {
+			$nav['usermenu'][] = Array('bookmarks', t('Bookmarks'), "", t('Your bookmarks'));
+		}
+
 		if(feature_enabled($channel['channel_id'],'webpages'))
-			$nav['usermenu'][] = Array('webpages/' . $channel['channel_address'],t('Webpages'),"",t('Your webpages'));	
+			$nav['usermenu'][] = Array('webpages/' . $channel['channel_address'],t('Webpages'),"",t('Your webpages'));
 	}
 	else {
 		if(! get_account_id()) 
-- 
cgit v1.2.3


From 6c806f6f48fc7efe5fda380f2f9edb9a7dc09cab Mon Sep 17 00:00:00 2001
From: marijus <mario@mariovavti.com>
Date: Sat, 20 Sep 2014 17:28:42 +0200
Subject: aalso change the way we check for number of chatrooms

---
 include/chat.php         | 11 +++++++++++
 include/conversation.php |  4 ++--
 include/nav.php          |  4 ++--
 3 files changed, 15 insertions(+), 4 deletions(-)

(limited to 'include')

diff --git a/include/chat.php b/include/chat.php
index b8fb185df..5c3d0c9d9 100644
--- a/include/chat.php
+++ b/include/chat.php
@@ -189,6 +189,17 @@ function chatroom_list($uid) {
 	return $r;
 }
 
+function chatroom_list_count($uid) {
+	require_once('include/security.php');
+	$sql_extra = permissions_sql($uid);
+
+	$r = q("select count(*) as total from chatroom where cr_uid = %d $sql_extra",
+		intval($uid)
+	);
+
+	return $r[0]['total'];
+}
+
 /**
  * create a chat message via API.
  * It is the caller's responsibility to enter the room.
diff --git a/include/conversation.php b/include/conversation.php
index 8f1767482..6e56782dd 100644
--- a/include/conversation.php
+++ b/include/conversation.php
@@ -1545,8 +1545,8 @@ function profile_tabs($a, $is_owner=False, $nickname=Null){
 	}
 
 	require_once('include/chat.php');
-	$chats = chatroom_list($uid);
-	if (count($chats)) {
+	$has_chats = chatroom_list_count($uid);
+	if (count($has_chats)) {
 		$tabs[] = array(
 			'label' => t('Chatrooms'),
 			'url'	=> $a->get_baseurl() . '/chat/' . $nickname,
diff --git a/include/nav.php b/include/nav.php
index c6bc4d293..714f4b63d 100644
--- a/include/nav.php
+++ b/include/nav.php
@@ -105,8 +105,8 @@ EOT;
 		$nav['usermenu'][] = Array('cloud/' . $channel['channel_address'],t('Files'),"",t('Your files'));
 
 		require_once('include/chat.php');
-		$chats = chatroom_list(local_user());
-		if (count($chats)) {
+		$has_chats = chatroom_list_count(local_user());
+		if($has_chats) {
 			$nav['usermenu'][] = Array('chat/' . $channel['channel_address'],t('Chat'),"",t('Your chatrooms'));
 		}
 
-- 
cgit v1.2.3


From 69a10e8ca737a5848dd888879b32a95fa7267ad5 Mon Sep 17 00:00:00 2001
From: marijus <mario@mariovavti.com>
Date: Sat, 20 Sep 2014 17:30:11 +0200
Subject: oups...

---
 include/conversation.php | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'include')

diff --git a/include/conversation.php b/include/conversation.php
index 6e56782dd..be5267207 100644
--- a/include/conversation.php
+++ b/include/conversation.php
@@ -1546,7 +1546,7 @@ function profile_tabs($a, $is_owner=False, $nickname=Null){
 
 	require_once('include/chat.php');
 	$has_chats = chatroom_list_count($uid);
-	if (count($has_chats)) {
+	if ($has_chats) {
 		$tabs[] = array(
 			'label' => t('Chatrooms'),
 			'url'	=> $a->get_baseurl() . '/chat/' . $nickname,
-- 
cgit v1.2.3