More usage of dbm::is_result($r) instead of count($r):

[friendica.git] / include / nav.php

diff --git a/include/nav.php b/include/nav.php
index a55629a5a09f0995c8115fedb4ff7f4fd7e2ae49..2a9f24ca9f5644e351e4ac29fd7ee2a676a4074e 100644 (file)
--- a/include/nav.php
+++ b/include/nav.php
@@ -43,8 +43,17 @@ function nav(&$a) {
        call_hooks('page_header', $a->page['nav']);
 }
 
-
-function nav_info(&$a)
+/**
+ * @brief Prepares a list of navigation links
+ *
+ * @param App $a
+ * @return array Navigation links
+ *     string 'sitelocation' => The webbie (username@site.com)
+ *     array 'nav' => Array of links used in the nav menu
+ *     string 'banner' => Formatted html link with banner image
+ *     array 'userinfo' => Array of user information (name, icon)
+ */
+function nav_info(App $a)
 {
        $ssl_state = ((local_user()) ? true : false);
 
@@ -56,7 +65,7 @@ function nav_info(&$a)
 
        $myident = ((is_array($a->user) && isset($a->user['nickname'])) ? $a->user['nickname'] . '@' : '');
 
-       $sitelocation = $myident . substr($a->get_baseurl($ssl_state),strpos($a->get_baseurl($ssl_state),'//') + 2 );
+       $sitelocation = $myident . substr($a->get_baseurl($ssl_state), strpos($a->get_baseurl($ssl_state), '//') + 2 );
 
        // nav links: array of array('href', 'text', 'extra css classes', 'title')
        $nav = array();
@@ -79,14 +88,14 @@ function nav_info(&$a)
                // user info
                $r = q("SELECT `micro` FROM `contact` WHERE `uid` = %d AND `self` = 1", intval($a->user['uid']));
                $userinfo = array(
-                       'icon' => (count($r) ? $a->remove_baseurl($r[0]['micro']) : 'images/person-48.jpg'),
+                       'icon' => (dbm::is_result($r) ? $a->remove_baseurl($r[0]['micro']) : 'images/person-48.jpg'),
                        'name' => $a->user['username'],
                );
        } else {
                $nav['login'] = array('login', t('Login'), ($a->module == 'login' ? 'selected' : ''), t('Sign in'));
        }
 
-    // "Home" should also take you home from an authenticated remote profile connection
+       // "Home" should also take you home from an authenticated remote profile connection
        $homelink = get_my_url();
        if (! $homelink) {
                $homelink = ((x($_SESSION,'visitor_home')) ? $_SESSION['visitor_home'] : '');
@@ -183,6 +192,7 @@ function nav_info(&$a)
                $nav['contacts'] = array('contacts', t('Contacts'), '', t('Manage/edit friends and contacts'));
        }
 
+       // Show the link to the admin configuration page if user is admin
        if (is_site_admin()) {
                $nav['admin'] = array('admin/', t('Admin'), '', t('Site setup and configuration'));
        }
@@ -192,7 +202,7 @@ function nav_info(&$a)
        // Provide a banner/logo/whatever
        $banner = get_config('system', 'banner');
        if ($banner === false) {
-               $banner .= '<a href="http://friendica.com"><img id="logo-img" src="images/friendica-32.png" alt="logo" /></a><span id="logo-text"><a href="http://friendica.com">Friendica</a></span>';
+               $banner = '<a href="http://friendica.com"><img id="logo-img" src="images/friendica-32.png" alt="logo" /></a><span id="logo-text"><a href="http://friendica.com">Friendica</a></span>';
        }
 
        call_hooks('nav_info', $nav);
@@ -205,7 +215,6 @@ function nav_info(&$a)
        );
 }
 
-
 /**
  * Set a menu item in navbar as selected
  *