From c66c6223b785d43749e62d7d5eaf110aeb9e9e26 Mon Sep 17 00:00:00 2001 From: Adam Magness Date: Mon, 15 Jan 2018 14:19:55 -0500 Subject: [PATCH 1/8] Create Nav class create nav class and function from include/nav.php --- src/Content/Nav.php | 257 ++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 257 insertions(+) create mode 100644 src/Content/Nav.php diff --git a/src/Content/Nav.php b/src/Content/Nav.php new file mode 100644 index 0000000000..20ae440b1c --- /dev/null +++ b/src/Content/Nav.php @@ -0,0 +1,257 @@ +page, 'nav'))) { + $a->page['nav'] = ''; + } + + $a->page['htmlhead'] .= replace_macros(get_markup_template('nav_head.tpl'), array()); + + /* + * Placeholder div for popup panel + */ + + $a->page['nav'] .= '' ; + + $nav_info = nav_info($a); + + /* + * Build the page + */ + + $tpl = get_markup_template('nav.tpl'); + + $a->page['nav'] .= replace_macros($tpl, array( + '$baseurl' => System::baseUrl(), + '$sitelocation' => $nav_info['sitelocation'], + '$nav' => $nav_info['nav'], + '$banner' => $nav_info['banner'], + '$emptynotifications' => t('Nothing new here'), + '$userinfo' => $nav_info['userinfo'], + '$sel' => $a->nav_sel, + '$apps' => $a->apps, + '$clear_notifs' => t('Clear notifications'), + '$search_hint' => t('@name, !forum, #tags, content') + )); + + call_hooks('page_header', $a->page['nav']); + } + + /** + * @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); + + /* + * Our network is distributed, and as you visit friends some of the + * sites look exactly the same - it isn't always easy to know where you are. + * Display the current site location as a navigation aid. + */ + + $myident = ((is_array($a->user) && isset($a->user['nickname'])) ? $a->user['nickname'] . '@' : ''); + + $sitelocation = $myident . substr(System::baseUrl($ssl_state), strpos(System::baseUrl($ssl_state), '//') + 2); + + // nav links: array of array('href', 'text', 'extra css classes', 'title') + $nav = array(); + + // Display login or logout + $nav['usermenu'] = array(); + $userinfo = null; + + if (local_user()) { + $nav['logout'] = array('logout', t('Logout'), '', t('End this session')); + + // user menu + $nav['usermenu'][] = array('profile/' . $a->user['nickname'], t('Status'), '', t('Your posts and conversations')); + $nav['usermenu'][] = array('profile/' . $a->user['nickname'] . '?tab=profile', t('Profile'), '', t('Your profile page')); + $nav['usermenu'][] = array('photos/' . $a->user['nickname'], t('Photos'), '', t('Your photos')); + $nav['usermenu'][] = array('videos/' . $a->user['nickname'], t('Videos'), '', t('Your videos')); + $nav['usermenu'][] = array('events/', t('Events'), '', t('Your events')); + $nav['usermenu'][] = array('notes/', t('Personal notes'), '', t('Your personal notes')); + + // user info + $contact = dba::selectFirst('contact', ['micro'], ['uid' => $a->user['uid'], 'self' => true]); + $userinfo = array( + 'icon' => (DBM::is_result($contact) ? $a->remove_baseurl($contact['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 + $homelink = Profile::getMyURL(); + if (! $homelink) { + $homelink = ((x($_SESSION, 'visitor_home')) ? $_SESSION['visitor_home'] : ''); + } + + if (($a->module != 'home') && (! (local_user()))) { + $nav['home'] = array($homelink, t('Home'), '', t('Home Page')); + } + + if (($a->config['register_policy'] == REGISTER_OPEN) && (! local_user()) && (! remote_user())) { + $nav['register'] = array('register', t('Register'), '', t('Create an account')); + } + + $help_url = 'help'; + + if (!Config::get('system', 'hide_help')) { + $nav['help'] = array($help_url, t('Help'), '', t('Help and documentation')); + } + + if (count($a->apps) > 0) { + $nav['apps'] = array('apps', t('Apps'), '', t('Addon applications, utilities, games')); + } + + if (local_user() || !Config::get('system', 'local_search')) { + $nav['search'] = array('search', t('Search'), '', t('Search site content')); + + $nav['searchoption'] = array( + t('Full Text'), + t('Tags'), + t('Contacts')); + + if (Config::get('system', 'poco_local_search')) { + $nav['searchoption'][] = t('Forums'); + } + } + + $gdirpath = 'directory'; + + if (strlen(Config::get('system', 'singleuser'))) { + $gdir = Config::get('system', 'directory'); + if (strlen($gdir)) { + $gdirpath = Profile::zrl($gdir, true); + } + } + + if (local_user() || Config::get('system', 'community_page_style') != CP_NO_COMMUNITY_PAGE) { + $nav['community'] = array('community', t('Community'), '', t('Conversations on this and other servers')); + } + + if (local_user()) { + $nav['events'] = array('events', t('Events'), '', t('Events and Calendar')); + } + + $nav['directory'] = array($gdirpath, t('Directory'), '', t('People directory')); + + $nav['about'] = array('friendica', t('Information'), '', t('Information about this friendica instance')); + + // The following nav links are only show to logged in users + if (local_user()) { + $nav['network'] = array('network', t('Network'), '', t('Conversations from your friends')); + $nav['net_reset'] = array('network/0?f=&order=comment&nets=all', t('Network Reset'), '', t('Load Network page with no filters')); + + $nav['home'] = array('profile/' . $a->user['nickname'], t('Home'), '', t('Your posts and conversations')); + + if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE, PAGE_PRVGROUP))) { + // only show friend requests for normal pages. Other page types have automatic friendship. + if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_PRVGROUP))) { + $nav['introductions'] = array('notifications/intros', t('Introductions'), '', t('Friend Requests')); + } + if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE))) { + $nav['notifications'] = array('notifications', t('Notifications'), '', t('Notifications')); + $nav['notifications']['all'] = array('notifications/system', t('See all notifications'), '', ''); + $nav['notifications']['mark'] = array('', t('Mark as seen'), '', t('Mark all system notifications seen')); + } + } + + $nav['messages'] = array('message', t('Messages'), '', t('Private mail')); + $nav['messages']['inbox'] = array('message', t('Inbox'), '', t('Inbox')); + $nav['messages']['outbox'] = array('message/sent', t('Outbox'), '', t('Outbox')); + $nav['messages']['new'] = array('message/new', t('New Message'), '', t('New Message')); + + if (is_array($a->identities) && count($a->identities) > 1) { + $nav['manage'] = array('manage', t('Manage'), '', t('Manage other pages')); + } + + $nav['delegations'] = array('delegate', t('Delegations'), '', t('Delegate Page Management')); + + $nav['settings'] = array('settings', t('Settings'), '', t('Account settings')); + + if (Feature::isEnabled(local_user(), 'multi_profiles')) { + $nav['profiles'] = array('profiles', t('Profiles'), '', t('Manage/Edit Profiles')); + } + + $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')); + } + + $nav['navigation'] = array('navigation/', t('Navigation'), '', t('Site map')); + + // Provide a banner/logo/whatever + $banner = Config::get('system', 'banner'); + if (is_null($banner)) { + $banner = 'logoFriendica'; + } + + call_hooks('nav_info', $nav); + + return array( + 'sitelocation' => $sitelocation, + 'nav' => $nav, + 'banner' => $banner, + 'userinfo' => $userinfo, + ); + } + + /** + * Set a menu item in navbar as selected + * + */ + function nav_set_selected($item) + { + $a = get_app(); + $a->nav_sel = array( + 'global' => null, + 'community' => null, + 'network' => null, + 'home' => null, + 'profiles' => null, + 'introductions' => null, + 'notifications' => null, + 'messages' => null, + 'directory' => null, + 'settings' => null, + 'contacts' => null, + 'manage' => null, + 'events' => null, + 'register' => null, + ); + $a->nav_sel[$item] = 'selected'; + } +} From 36d891bcadd3cc49a352289d4a95da82cfff4ca7 Mon Sep 17 00:00:00 2001 From: Adam Magness Date: Mon, 15 Jan 2018 14:51:56 -0500 Subject: [PATCH 2/8] Update functions and calls Update function names and scope. update calls. --- boot.php | 1 - index.php | 5 +- mod/cal.php | 5 +- mod/community.php | 7 +- mod/contacts.php | 3 +- mod/directory.php | 3 +- mod/events.php | 5 +- mod/help.php | 7 +- mod/message.php | 3 +- mod/network.php | 5 +- mod/notes.php | 3 +- mod/notifications.php | 7 +- mod/photos.php | 3 +- mod/profile.php | 3 +- mod/profiles.php | 3 +- mod/search.php | 3 +- mod/settings.php | 3 +- mod/videos.php | 3 +- mod/viewcontacts.php | 3 +- src/Content/Nav.php | 144 ++++++++++++++++++++++-------------------- 20 files changed, 122 insertions(+), 97 deletions(-) diff --git a/boot.php b/boot.php index 2fe73bc812..b529131bb6 100644 --- a/boot.php +++ b/boot.php @@ -36,7 +36,6 @@ require_once 'include/plugin.php'; require_once 'include/text.php'; require_once 'include/datetime.php'; require_once 'include/pgettext.php'; -require_once 'include/nav.php'; define('FRIENDICA_PLATFORM', 'Friendica'); define('FRIENDICA_CODENAME', 'Asparagus'); diff --git a/index.php b/index.php index 74a88eb7eb..e62d4e362a 100644 --- a/index.php +++ b/index.php @@ -10,6 +10,7 @@ use Friendica\App; use Friendica\BaseObject; +use Friendica\Content\Nav; use Friendica\Core\System; use Friendica\Core\Config; use Friendica\Core\Worker; @@ -192,7 +193,7 @@ if ($install && $a->module!="view") { check_plugins($a); } -nav_set_selected('nothing'); +Nav::setSelected('nothing'); //Don't populate apps_menu if apps are private $privateapps = Config::get('config', 'private_addons'); @@ -426,7 +427,7 @@ call_hooks('page_end', $a->page['content']); * Add the navigation (menu) template */ if ($a->module != 'install' && $a->module != 'maintenance') { - nav($a); + Nav::build($a); } /* diff --git a/mod/cal.php b/mod/cal.php index 31ab5976b5..6a8c466db1 100644 --- a/mod/cal.php +++ b/mod/cal.php @@ -7,6 +7,7 @@ */ use Friendica\App; use Friendica\Content\Feature; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Core\System; use Friendica\Database\DBM; @@ -27,7 +28,7 @@ function cal_init(App $a) return; } - nav_set_selected('events'); + Nav::setSelected('events'); if ($a->argc > 1) { $nick = $a->argv[1]; @@ -74,7 +75,7 @@ function cal_init(App $a) function cal_content(App $a) { - nav_set_selected('events'); + Nav::setSelected('events'); // get the translation strings for the callendar $i18n = get_event_strings(); diff --git a/mod/community.php b/mod/community.php index 4b755e18ce..64a86f07ed 100644 --- a/mod/community.php +++ b/mod/community.php @@ -1,6 +1,9 @@ $tabs)); - nav_set_selected('community'); + Nav::setSelected('community'); // We need the editor here to be able to reshare an item. if (local_user()) { diff --git a/mod/contacts.php b/mod/contacts.php index 7089cf05a5..afa56b87f5 100644 --- a/mod/contacts.php +++ b/mod/contacts.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\ContactSelector; +use Friendica\Content\Nav; use Friendica\Content\Widget; use Friendica\Core\System; use Friendica\Core\Worker; @@ -369,7 +370,7 @@ function contacts_content(App $a) { $sort_type = 0; $o = ''; - nav_set_selected('contacts'); + Nav::setSelected('contacts'); if (!local_user()) { notice(t('Permission denied.') . EOL); diff --git a/mod/directory.php b/mod/directory.php index a711f72314..02390f080a 100644 --- a/mod/directory.php +++ b/mod/directory.php @@ -3,6 +3,7 @@ * @file mod/directory.php */ use Friendica\App; +use Friendica\Content\Nav; use Friendica\Content\Widget; use Friendica\Core\Config; use Friendica\Database\DBM; @@ -37,7 +38,7 @@ function directory_content(App $a) { } $o = ''; - nav_set_selected('directory'); + Nav::setSelected('directory'); if(x($a->data,'search')) $search = notags(trim($a->data['search'])); diff --git a/mod/events.php b/mod/events.php index b64b6a1340..79db31d207 100644 --- a/mod/events.php +++ b/mod/events.php @@ -4,6 +4,7 @@ * @brief The events module */ use Friendica\App; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Core\System; use Friendica\Core\Worker; @@ -212,9 +213,9 @@ function events_content(App $a) { } if ($a->theme_events_in_profile) { - nav_set_selected('home'); + Nav::setSelected('home'); } else { - nav_set_selected('events'); + Nav::setSelected('events'); } // get the translation strings for the callendar diff --git a/mod/help.php b/mod/help.php index 65221165fd..399b7768be 100644 --- a/mod/help.php +++ b/mod/help.php @@ -1,6 +1,9 @@ true, @@ -575,7 +576,7 @@ function networkThreadedView(App $a, $update = 0) { } } - nav_set_selected('network'); + Nav::setSelected('network'); $content = ""; diff --git a/mod/notes.php b/mod/notes.php index dbf84337ef..8637265fc7 100644 --- a/mod/notes.php +++ b/mod/notes.php @@ -3,6 +3,7 @@ * @file mod/notes.php */ use Friendica\App; +use Friendica\Content\Nav; use Friendica\Database\DBM; use Friendica\Model\Profile; @@ -16,7 +17,7 @@ function notes_init(App $a) { $which = $a->user['nickname']; - nav_set_selected('home'); + Nav::setSelected('home'); //Profile::load($a, $which, $profile); diff --git a/mod/notifications.php b/mod/notifications.php index e2de81e6c7..100170010d 100644 --- a/mod/notifications.php +++ b/mod/notifications.php @@ -5,6 +5,7 @@ */ use Friendica\App; use Friendica\Content\ContactSelector; +use Friendica\Content\Nav; use Friendica\Core\NotificationsManager; use Friendica\Core\System; use Friendica\Database\DBM; @@ -77,7 +78,7 @@ function notifications_content(App $a) { $page = (x($_REQUEST,'page') ? $_REQUEST['page'] : 1); $show = (x($_REQUEST,'show') ? $_REQUEST['show'] : 0); - nav_set_selected('notifications'); + Nav::setSelected('notifications'); $json = (($a->argc > 1 && $a->argv[$a->argc - 1] === 'json') ? true : false); @@ -93,8 +94,8 @@ function notifications_content(App $a) { $startrec = ($page * $perpage) - $perpage; // Get introductions - if( (($a->argc > 1) && ($a->argv[1] == 'intros')) || (($a->argc == 1))) { - nav_set_selected('introductions'); + if ((($a->argc > 1) && ($a->argv[1] == 'intros')) || (($a->argc == 1))) { + Nav::setSelected('introductions'); $notif_header = t('Notifications'); $all = (($a->argc > 2) && ($a->argv[2] == 'all')); diff --git a/mod/photos.php b/mod/photos.php index ac1f06f8e8..a8345a74e3 100644 --- a/mod/photos.php +++ b/mod/photos.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\Feature; +use Friendica\Content\Nav; use Friendica\Core\System; use Friendica\Core\Config; use Friendica\Core\Worker; @@ -33,7 +34,7 @@ function photos_init(App $a) { return; } - nav_set_selected('home'); + Nav::setSelected('home'); if ($a->argc > 1) { $nick = $a->argv[1]; diff --git a/mod/profile.php b/mod/profile.php index 77f8e069d3..765abc9c45 100644 --- a/mod/profile.php +++ b/mod/profile.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\Widget; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Core\PConfig; use Friendica\Core\System; @@ -125,7 +126,7 @@ function profile_content(App $a, $update = 0) // Ensure we've got a profile owner if updating. $a->profile['profile_uid'] = $update; } elseif ($a->profile['profile_uid'] == local_user()) { - nav_set_selected('home'); + Nav::setSelected('home'); } $contact = null; diff --git a/mod/profiles.php b/mod/profiles.php index 5fcd2d8f17..f653414403 100644 --- a/mod/profiles.php +++ b/mod/profiles.php @@ -5,6 +5,7 @@ use Friendica\App; use Friendica\Content\ContactSelector; use Friendica\Content\Feature; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Core\PConfig; use Friendica\Core\System; @@ -16,7 +17,7 @@ use Friendica\Network\Probe; function profiles_init(App $a) { - nav_set_selected('profiles'); + Nav::setSelected('profiles'); if (! local_user()) { return; diff --git a/mod/search.php b/mod/search.php index c17b11c5d9..fe33340dfb 100644 --- a/mod/search.php +++ b/mod/search.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\Feature; +use Friendica\Content\Nav; use Friendica\Core\Cache; use Friendica\Core\Config; use Friendica\Database\DBM; @@ -132,7 +133,7 @@ function search_content(App $a) { Cache::set("remote_search:".$remote, json_encode(array("time" => time(), "accesses" => 1)), CACHE_HOUR); } - nav_set_selected('search'); + Nav::setSelected('search'); if (x($a->data,'search')) $search = notags(trim($a->data['search'])); diff --git a/mod/settings.php b/mod/settings.php index 8fd890804f..951da86209 100644 --- a/mod/settings.php +++ b/mod/settings.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\Feature; +use Friendica\Content\Nav; use Friendica\Core\System; use Friendica\Core\Worker; use Friendica\Core\Config; @@ -654,7 +655,7 @@ function settings_post(App $a) function settings_content(App $a) { $o = ''; - nav_set_selected('settings'); + Nav::setSelected('settings'); if (!local_user()) { //notice(t('Permission denied.') . EOL); diff --git a/mod/videos.php b/mod/videos.php index 1a9f57a396..153bca1877 100644 --- a/mod/videos.php +++ b/mod/videos.php @@ -3,6 +3,7 @@ * @file mod/videos.php */ use Friendica\App; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Core\System; use Friendica\Core\Worker; @@ -26,7 +27,7 @@ function videos_init(App $a) { return; } - nav_set_selected('home'); + Nav::setSelected('home'); $o = ''; diff --git a/mod/viewcontacts.php b/mod/viewcontacts.php index 64dd24e16c..17913a5082 100644 --- a/mod/viewcontacts.php +++ b/mod/viewcontacts.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\ContactSelector; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Database\DBM; use Friendica\Model\Contact; @@ -15,7 +16,7 @@ function viewcontacts_init(App $a) { return; } - nav_set_selected('home'); + Nav::setSelected('home'); if($a->argc > 1) { $nick = $a->argv[1]; diff --git a/src/Content/Nav.php b/src/Content/Nav.php index 20ae440b1c..8445d827f5 100644 --- a/src/Content/Nav.php +++ b/src/Content/Nav.php @@ -11,21 +11,22 @@ use Friendica\Core\System; use Friendica\Database\DBM; use Friendica\Model\Profile; +require_once 'boot.php'; +require_once 'include/pgettext.php'; +require_once 'include/text.php'; + class Nav { - function nav(App $a) + /** + * Build page header and site navigation bars + */ + public static function build(App $a) { - /* - * - * Build page header and site navigation bars - * - */ - if (!(x($a->page, 'nav'))) { $a->page['nav'] = ''; } - $a->page['htmlhead'] .= replace_macros(get_markup_template('nav_head.tpl'), array()); + $a->page['htmlhead'] .= replace_macros(get_markup_template('nav_head.tpl'), []); /* * Placeholder div for popup panel @@ -33,7 +34,7 @@ class Nav $a->page['nav'] .= '' ; - $nav_info = nav_info($a); + $nav_info = self::getInfo($a); /* * Build the page @@ -41,25 +42,28 @@ class Nav $tpl = get_markup_template('nav.tpl'); - $a->page['nav'] .= replace_macros($tpl, array( - '$baseurl' => System::baseUrl(), - '$sitelocation' => $nav_info['sitelocation'], - '$nav' => $nav_info['nav'], - '$banner' => $nav_info['banner'], - '$emptynotifications' => t('Nothing new here'), - '$userinfo' => $nav_info['userinfo'], - '$sel' => $a->nav_sel, - '$apps' => $a->apps, - '$clear_notifs' => t('Clear notifications'), - '$search_hint' => t('@name, !forum, #tags, content') - )); + $a->page['nav'] .= replace_macros($tpl, + [ + '$baseurl' => System::baseUrl(), + '$sitelocation' => $nav_info['sitelocation'], + '$nav' => $nav_info['nav'], + '$banner' => $nav_info['banner'], + '$emptynotifications' => t('Nothing new here'), + '$userinfo' => $nav_info['userinfo'], + '$sel' => $a->nav_sel, + '$apps' => $a->apps, + '$clear_notifs' => t('Clear notifications'), + '$search_hint' => t('@name, !forum, #tags, content') + ] + ); call_hooks('page_header', $a->page['nav']); } /** - * @brief Prepares a list of navigation links + * Prepares a list of navigation links * + * @brief Prepares a list of navigation links * @param App $a * @return array Navigation links * string 'sitelocation' => The webbie (username@site.com) @@ -67,7 +71,7 @@ class Nav * string 'banner' => Formatted html link with banner image * array 'userinfo' => Array of user information (name, icon) */ - function nav_info(App $a) + private static function getInfo(App $a) { $ssl_state = ((local_user()) ? true : false); @@ -82,31 +86,31 @@ class Nav $sitelocation = $myident . substr(System::baseUrl($ssl_state), strpos(System::baseUrl($ssl_state), '//') + 2); // nav links: array of array('href', 'text', 'extra css classes', 'title') - $nav = array(); + $nav = []; // Display login or logout - $nav['usermenu'] = array(); + $nav['usermenu'] = []; $userinfo = null; if (local_user()) { - $nav['logout'] = array('logout', t('Logout'), '', t('End this session')); + $nav['logout'] = ['logout', t('Logout'), '', t('End this session')]; // user menu - $nav['usermenu'][] = array('profile/' . $a->user['nickname'], t('Status'), '', t('Your posts and conversations')); - $nav['usermenu'][] = array('profile/' . $a->user['nickname'] . '?tab=profile', t('Profile'), '', t('Your profile page')); - $nav['usermenu'][] = array('photos/' . $a->user['nickname'], t('Photos'), '', t('Your photos')); - $nav['usermenu'][] = array('videos/' . $a->user['nickname'], t('Videos'), '', t('Your videos')); - $nav['usermenu'][] = array('events/', t('Events'), '', t('Your events')); - $nav['usermenu'][] = array('notes/', t('Personal notes'), '', t('Your personal notes')); + $nav['usermenu'][] = ['profile/' . $a->user['nickname'], t('Status'), '', t('Your posts and conversations')]; + $nav['usermenu'][] = ['profile/' . $a->user['nickname'] . '?tab=profile', t('Profile'), '', t('Your profile page')]; + $nav['usermenu'][] = ['photos/' . $a->user['nickname'], t('Photos'), '', t('Your photos')]; + $nav['usermenu'][] = ['videos/' . $a->user['nickname'], t('Videos'), '', t('Your videos')]; + $nav['usermenu'][] = ['events/', t('Events'), '', t('Your events')]; + $nav['usermenu'][] = ['notes/', t('Personal notes'), '', t('Your personal notes')]; // user info $contact = dba::selectFirst('contact', ['micro'], ['uid' => $a->user['uid'], 'self' => true]); - $userinfo = array( + $userinfo = [ 'icon' => (DBM::is_result($contact) ? $a->remove_baseurl($contact['micro']) : 'images/person-48.jpg'), 'name' => $a->user['username'], - ); + ]; } else { - $nav['login'] = array('login', t('Login'), ($a->module == 'login' ? 'selected' : ''), t('Sign in')); + $nav['login'] = ['login', t('Login'), ($a->module == 'login' ? 'selected' : ''), t('Sign in')]; } // "Home" should also take you home from an authenticated remote profile connection @@ -116,30 +120,31 @@ class Nav } if (($a->module != 'home') && (! (local_user()))) { - $nav['home'] = array($homelink, t('Home'), '', t('Home Page')); + $nav['home'] = [$homelink, t('Home'), '', t('Home Page')]; } if (($a->config['register_policy'] == REGISTER_OPEN) && (! local_user()) && (! remote_user())) { - $nav['register'] = array('register', t('Register'), '', t('Create an account')); + $nav['register'] = ['register', t('Register'), '', t('Create an account')]; } $help_url = 'help'; if (!Config::get('system', 'hide_help')) { - $nav['help'] = array($help_url, t('Help'), '', t('Help and documentation')); + $nav['help'] = [$help_url, t('Help'), '', t('Help and documentation')]; } if (count($a->apps) > 0) { - $nav['apps'] = array('apps', t('Apps'), '', t('Addon applications, utilities, games')); + $nav['apps'] = ['apps', t('Apps'), '', t('Addon applications, utilities, games')]; } if (local_user() || !Config::get('system', 'local_search')) { - $nav['search'] = array('search', t('Search'), '', t('Search site content')); + $nav['search'] = ['search', t('Search'), '', t('Search site content')]; - $nav['searchoption'] = array( + $nav['searchoption'] = [ t('Full Text'), t('Tags'), - t('Contacts')); + t('Contacts') + ]; if (Config::get('system', 'poco_local_search')) { $nav['searchoption'][] = t('Forums'); @@ -156,23 +161,23 @@ class Nav } if (local_user() || Config::get('system', 'community_page_style') != CP_NO_COMMUNITY_PAGE) { - $nav['community'] = array('community', t('Community'), '', t('Conversations on this and other servers')); + $nav['community'] = ['community', t('Community'), '', t('Conversations on this and other servers')]; } if (local_user()) { - $nav['events'] = array('events', t('Events'), '', t('Events and Calendar')); + $nav['events'] = ['events', t('Events'), '', t('Events and Calendar')]; } - $nav['directory'] = array($gdirpath, t('Directory'), '', t('People directory')); + $nav['directory'] = [$gdirpath, t('Directory'), '', t('People directory')]; - $nav['about'] = array('friendica', t('Information'), '', t('Information about this friendica instance')); + $nav['about'] = ['friendica', t('Information'), '', t('Information about this friendica instance')]; // The following nav links are only show to logged in users if (local_user()) { - $nav['network'] = array('network', t('Network'), '', t('Conversations from your friends')); - $nav['net_reset'] = array('network/0?f=&order=comment&nets=all', t('Network Reset'), '', t('Load Network page with no filters')); + $nav['network'] = ['network', t('Network'), '', t('Conversations from your friends')]; + $nav['net_reset'] = ['network/0?f=&order=comment&nets=all', t('Network Reset'), '', t('Load Network page with no filters')]; - $nav['home'] = array('profile/' . $a->user['nickname'], t('Home'), '', t('Your posts and conversations')); + $nav['home'] = ['profile/' . $a->user['nickname'], t('Home'), '', t('Your posts and conversations')]; if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE, PAGE_PRVGROUP))) { // only show friend requests for normal pages. Other page types have automatic friendship. @@ -180,38 +185,38 @@ class Nav $nav['introductions'] = array('notifications/intros', t('Introductions'), '', t('Friend Requests')); } if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE))) { - $nav['notifications'] = array('notifications', t('Notifications'), '', t('Notifications')); - $nav['notifications']['all'] = array('notifications/system', t('See all notifications'), '', ''); - $nav['notifications']['mark'] = array('', t('Mark as seen'), '', t('Mark all system notifications seen')); + $nav['notifications'] = ['notifications', t('Notifications'), '', t('Notifications')]; + $nav['notifications']['all'] = ['notifications/system', t('See all notifications'), '', '']; + $nav['notifications']['mark'] = ['', t('Mark as seen'), '', t('Mark all system notifications seen')]; } } - $nav['messages'] = array('message', t('Messages'), '', t('Private mail')); - $nav['messages']['inbox'] = array('message', t('Inbox'), '', t('Inbox')); - $nav['messages']['outbox'] = array('message/sent', t('Outbox'), '', t('Outbox')); - $nav['messages']['new'] = array('message/new', t('New Message'), '', t('New Message')); + $nav['messages'] = ['message', t('Messages'), '', t('Private mail')]; + $nav['messages']['inbox'] = ['message', t('Inbox'), '', t('Inbox')]; + $nav['messages']['outbox'] = ['message/sent', t('Outbox'), '', t('Outbox')]; + $nav['messages']['new'] = ['message/new', t('New Message'), '', t('New Message')]; if (is_array($a->identities) && count($a->identities) > 1) { - $nav['manage'] = array('manage', t('Manage'), '', t('Manage other pages')); + $nav['manage'] = ['manage', t('Manage'), '', t('Manage other pages')]; } - $nav['delegations'] = array('delegate', t('Delegations'), '', t('Delegate Page Management')); + $nav['delegations'] = ['delegate', t('Delegations'), '', t('Delegate Page Management')]; - $nav['settings'] = array('settings', t('Settings'), '', t('Account settings')); + $nav['settings'] = ['settings', t('Settings'), '', t('Account settings')]; if (Feature::isEnabled(local_user(), 'multi_profiles')) { - $nav['profiles'] = array('profiles', t('Profiles'), '', t('Manage/Edit Profiles')); + $nav['profiles'] = ['profiles', t('Profiles'), '', t('Manage/Edit Profiles')]; } - $nav['contacts'] = array('contacts', t('Contacts'), '', t('Manage/edit friends and contacts')); + $nav['contacts'] = ['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')); + $nav['admin'] = ['admin/', t('Admin'), '', t('Site setup and configuration')]; } - $nav['navigation'] = array('navigation/', t('Navigation'), '', t('Site map')); + $nav['navigation'] = ['navigation/', t('Navigation'), '', t('Site map')]; // Provide a banner/logo/whatever $banner = Config::get('system', 'banner'); @@ -221,22 +226,21 @@ class Nav call_hooks('nav_info', $nav); - return array( + return [ 'sitelocation' => $sitelocation, 'nav' => $nav, 'banner' => $banner, 'userinfo' => $userinfo, - ); + ]; } /** * Set a menu item in navbar as selected - * */ - function nav_set_selected($item) + public static function setSelected($item) { $a = get_app(); - $a->nav_sel = array( + $a->nav_sel = [ 'global' => null, 'community' => null, 'network' => null, @@ -250,8 +254,8 @@ class Nav 'contacts' => null, 'manage' => null, 'events' => null, - 'register' => null, - ); + 'register' => null + ]; $a->nav_sel[$item] = 'selected'; } } From d01a31fa4f0bdc84247dd7518a24ef778e039369 Mon Sep 17 00:00:00 2001 From: Adam Magness Date: Mon, 15 Jan 2018 17:08:36 -0500 Subject: [PATCH 3/8] use dba add use dba --- src/Content/Nav.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/Content/Nav.php b/src/Content/Nav.php index 8445d827f5..a576423bfd 100644 --- a/src/Content/Nav.php +++ b/src/Content/Nav.php @@ -10,8 +10,10 @@ use Friendica\Core\Config; use Friendica\Core\System; use Friendica\Database\DBM; use Friendica\Model\Profile; +use dba; require_once 'boot.php'; +require_once 'dba.php'; require_once 'include/pgettext.php'; require_once 'include/text.php'; From ac317060fb3ccacd0ce1e3bf1f3688f1079d599b Mon Sep 17 00:00:00 2001 From: Adam Magness Date: Mon, 15 Jan 2018 14:19:55 -0500 Subject: [PATCH 4/8] Create Nav class create nav class and function from include/nav.php --- src/Content/Nav.php | 257 ++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 257 insertions(+) create mode 100644 src/Content/Nav.php diff --git a/src/Content/Nav.php b/src/Content/Nav.php new file mode 100644 index 0000000000..20ae440b1c --- /dev/null +++ b/src/Content/Nav.php @@ -0,0 +1,257 @@ +page, 'nav'))) { + $a->page['nav'] = ''; + } + + $a->page['htmlhead'] .= replace_macros(get_markup_template('nav_head.tpl'), array()); + + /* + * Placeholder div for popup panel + */ + + $a->page['nav'] .= '' ; + + $nav_info = nav_info($a); + + /* + * Build the page + */ + + $tpl = get_markup_template('nav.tpl'); + + $a->page['nav'] .= replace_macros($tpl, array( + '$baseurl' => System::baseUrl(), + '$sitelocation' => $nav_info['sitelocation'], + '$nav' => $nav_info['nav'], + '$banner' => $nav_info['banner'], + '$emptynotifications' => t('Nothing new here'), + '$userinfo' => $nav_info['userinfo'], + '$sel' => $a->nav_sel, + '$apps' => $a->apps, + '$clear_notifs' => t('Clear notifications'), + '$search_hint' => t('@name, !forum, #tags, content') + )); + + call_hooks('page_header', $a->page['nav']); + } + + /** + * @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); + + /* + * Our network is distributed, and as you visit friends some of the + * sites look exactly the same - it isn't always easy to know where you are. + * Display the current site location as a navigation aid. + */ + + $myident = ((is_array($a->user) && isset($a->user['nickname'])) ? $a->user['nickname'] . '@' : ''); + + $sitelocation = $myident . substr(System::baseUrl($ssl_state), strpos(System::baseUrl($ssl_state), '//') + 2); + + // nav links: array of array('href', 'text', 'extra css classes', 'title') + $nav = array(); + + // Display login or logout + $nav['usermenu'] = array(); + $userinfo = null; + + if (local_user()) { + $nav['logout'] = array('logout', t('Logout'), '', t('End this session')); + + // user menu + $nav['usermenu'][] = array('profile/' . $a->user['nickname'], t('Status'), '', t('Your posts and conversations')); + $nav['usermenu'][] = array('profile/' . $a->user['nickname'] . '?tab=profile', t('Profile'), '', t('Your profile page')); + $nav['usermenu'][] = array('photos/' . $a->user['nickname'], t('Photos'), '', t('Your photos')); + $nav['usermenu'][] = array('videos/' . $a->user['nickname'], t('Videos'), '', t('Your videos')); + $nav['usermenu'][] = array('events/', t('Events'), '', t('Your events')); + $nav['usermenu'][] = array('notes/', t('Personal notes'), '', t('Your personal notes')); + + // user info + $contact = dba::selectFirst('contact', ['micro'], ['uid' => $a->user['uid'], 'self' => true]); + $userinfo = array( + 'icon' => (DBM::is_result($contact) ? $a->remove_baseurl($contact['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 + $homelink = Profile::getMyURL(); + if (! $homelink) { + $homelink = ((x($_SESSION, 'visitor_home')) ? $_SESSION['visitor_home'] : ''); + } + + if (($a->module != 'home') && (! (local_user()))) { + $nav['home'] = array($homelink, t('Home'), '', t('Home Page')); + } + + if (($a->config['register_policy'] == REGISTER_OPEN) && (! local_user()) && (! remote_user())) { + $nav['register'] = array('register', t('Register'), '', t('Create an account')); + } + + $help_url = 'help'; + + if (!Config::get('system', 'hide_help')) { + $nav['help'] = array($help_url, t('Help'), '', t('Help and documentation')); + } + + if (count($a->apps) > 0) { + $nav['apps'] = array('apps', t('Apps'), '', t('Addon applications, utilities, games')); + } + + if (local_user() || !Config::get('system', 'local_search')) { + $nav['search'] = array('search', t('Search'), '', t('Search site content')); + + $nav['searchoption'] = array( + t('Full Text'), + t('Tags'), + t('Contacts')); + + if (Config::get('system', 'poco_local_search')) { + $nav['searchoption'][] = t('Forums'); + } + } + + $gdirpath = 'directory'; + + if (strlen(Config::get('system', 'singleuser'))) { + $gdir = Config::get('system', 'directory'); + if (strlen($gdir)) { + $gdirpath = Profile::zrl($gdir, true); + } + } + + if (local_user() || Config::get('system', 'community_page_style') != CP_NO_COMMUNITY_PAGE) { + $nav['community'] = array('community', t('Community'), '', t('Conversations on this and other servers')); + } + + if (local_user()) { + $nav['events'] = array('events', t('Events'), '', t('Events and Calendar')); + } + + $nav['directory'] = array($gdirpath, t('Directory'), '', t('People directory')); + + $nav['about'] = array('friendica', t('Information'), '', t('Information about this friendica instance')); + + // The following nav links are only show to logged in users + if (local_user()) { + $nav['network'] = array('network', t('Network'), '', t('Conversations from your friends')); + $nav['net_reset'] = array('network/0?f=&order=comment&nets=all', t('Network Reset'), '', t('Load Network page with no filters')); + + $nav['home'] = array('profile/' . $a->user['nickname'], t('Home'), '', t('Your posts and conversations')); + + if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE, PAGE_PRVGROUP))) { + // only show friend requests for normal pages. Other page types have automatic friendship. + if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_PRVGROUP))) { + $nav['introductions'] = array('notifications/intros', t('Introductions'), '', t('Friend Requests')); + } + if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE))) { + $nav['notifications'] = array('notifications', t('Notifications'), '', t('Notifications')); + $nav['notifications']['all'] = array('notifications/system', t('See all notifications'), '', ''); + $nav['notifications']['mark'] = array('', t('Mark as seen'), '', t('Mark all system notifications seen')); + } + } + + $nav['messages'] = array('message', t('Messages'), '', t('Private mail')); + $nav['messages']['inbox'] = array('message', t('Inbox'), '', t('Inbox')); + $nav['messages']['outbox'] = array('message/sent', t('Outbox'), '', t('Outbox')); + $nav['messages']['new'] = array('message/new', t('New Message'), '', t('New Message')); + + if (is_array($a->identities) && count($a->identities) > 1) { + $nav['manage'] = array('manage', t('Manage'), '', t('Manage other pages')); + } + + $nav['delegations'] = array('delegate', t('Delegations'), '', t('Delegate Page Management')); + + $nav['settings'] = array('settings', t('Settings'), '', t('Account settings')); + + if (Feature::isEnabled(local_user(), 'multi_profiles')) { + $nav['profiles'] = array('profiles', t('Profiles'), '', t('Manage/Edit Profiles')); + } + + $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')); + } + + $nav['navigation'] = array('navigation/', t('Navigation'), '', t('Site map')); + + // Provide a banner/logo/whatever + $banner = Config::get('system', 'banner'); + if (is_null($banner)) { + $banner = 'logoFriendica'; + } + + call_hooks('nav_info', $nav); + + return array( + 'sitelocation' => $sitelocation, + 'nav' => $nav, + 'banner' => $banner, + 'userinfo' => $userinfo, + ); + } + + /** + * Set a menu item in navbar as selected + * + */ + function nav_set_selected($item) + { + $a = get_app(); + $a->nav_sel = array( + 'global' => null, + 'community' => null, + 'network' => null, + 'home' => null, + 'profiles' => null, + 'introductions' => null, + 'notifications' => null, + 'messages' => null, + 'directory' => null, + 'settings' => null, + 'contacts' => null, + 'manage' => null, + 'events' => null, + 'register' => null, + ); + $a->nav_sel[$item] = 'selected'; + } +} From 0f546f7f0f8b41bd7510bd3644419c002746ce34 Mon Sep 17 00:00:00 2001 From: Adam Magness Date: Mon, 15 Jan 2018 14:51:56 -0500 Subject: [PATCH 5/8] Update functions and calls Update function names and scope. update calls. --- boot.php | 1 - index.php | 5 +- mod/cal.php | 5 +- mod/community.php | 7 +- mod/contacts.php | 3 +- mod/directory.php | 3 +- mod/events.php | 5 +- mod/help.php | 7 +- mod/message.php | 3 +- mod/network.php | 5 +- mod/notes.php | 3 +- mod/notifications.php | 7 +- mod/photos.php | 3 +- mod/profile.php | 3 +- mod/profiles.php | 3 +- mod/search.php | 3 +- mod/settings.php | 3 +- mod/videos.php | 3 +- mod/viewcontacts.php | 3 +- src/Content/Nav.php | 144 ++++++++++++++++++++++-------------------- 20 files changed, 122 insertions(+), 97 deletions(-) diff --git a/boot.php b/boot.php index 4bfa5c0985..0692e84757 100644 --- a/boot.php +++ b/boot.php @@ -36,7 +36,6 @@ require_once 'include/plugin.php'; require_once 'include/text.php'; require_once 'include/datetime.php'; require_once 'include/pgettext.php'; -require_once 'include/nav.php'; define('FRIENDICA_PLATFORM', 'Friendica'); define('FRIENDICA_CODENAME', 'Asparagus'); diff --git a/index.php b/index.php index bfd5cda64a..dcb25cc3a3 100644 --- a/index.php +++ b/index.php @@ -10,6 +10,7 @@ use Friendica\App; use Friendica\BaseObject; +use Friendica\Content\Nav; use Friendica\Core\System; use Friendica\Core\Config; use Friendica\Core\Worker; @@ -192,7 +193,7 @@ if ($install && $a->module!="view") { check_plugins($a); } -nav_set_selected('nothing'); +Nav::setSelected('nothing'); //Don't populate apps_menu if apps are private $privateapps = Config::get('config', 'private_addons'); @@ -426,7 +427,7 @@ call_hooks('page_end', $a->page['content']); * Add the navigation (menu) template */ if ($a->module != 'install' && $a->module != 'maintenance') { - nav($a); + Nav::build($a); } /* diff --git a/mod/cal.php b/mod/cal.php index de00feadd2..8f5c1b14c6 100644 --- a/mod/cal.php +++ b/mod/cal.php @@ -7,6 +7,7 @@ */ use Friendica\App; use Friendica\Content\Feature; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Core\System; use Friendica\Database\DBM; @@ -27,7 +28,7 @@ function cal_init(App $a) return; } - nav_set_selected('events'); + Nav::setSelected('events'); if ($a->argc > 1) { $nick = $a->argv[1]; @@ -74,7 +75,7 @@ function cal_init(App $a) function cal_content(App $a) { - nav_set_selected('events'); + Nav::setSelected('events'); // get the translation strings for the callendar $i18n = get_event_strings(); diff --git a/mod/community.php b/mod/community.php index cfd2087d97..5fdd6de28d 100644 --- a/mod/community.php +++ b/mod/community.php @@ -1,6 +1,9 @@ $tabs]); - nav_set_selected('community'); + Nav::setSelected('community'); // We need the editor here to be able to reshare an item. if (local_user()) { diff --git a/mod/contacts.php b/mod/contacts.php index ae0edb3cce..682d427c6d 100644 --- a/mod/contacts.php +++ b/mod/contacts.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\ContactSelector; +use Friendica\Content\Nav; use Friendica\Content\Widget; use Friendica\Core\System; use Friendica\Core\Worker; @@ -369,7 +370,7 @@ function contacts_content(App $a) { $sort_type = 0; $o = ''; - nav_set_selected('contacts'); + Nav::setSelected('contacts'); if (!local_user()) { notice(t('Permission denied.') . EOL); diff --git a/mod/directory.php b/mod/directory.php index 59a3195b1b..6398127cf5 100644 --- a/mod/directory.php +++ b/mod/directory.php @@ -3,6 +3,7 @@ * @file mod/directory.php */ use Friendica\App; +use Friendica\Content\Nav; use Friendica\Content\Widget; use Friendica\Core\Config; use Friendica\Database\DBM; @@ -37,7 +38,7 @@ function directory_content(App $a) { } $o = ''; - nav_set_selected('directory'); + Nav::setSelected('directory'); if(x($a->data,'search')) $search = notags(trim($a->data['search'])); diff --git a/mod/events.php b/mod/events.php index 791499515d..1c8c92a31b 100644 --- a/mod/events.php +++ b/mod/events.php @@ -4,6 +4,7 @@ * @brief The events module */ use Friendica\App; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Core\System; use Friendica\Core\Worker; @@ -212,9 +213,9 @@ function events_content(App $a) { } if ($a->theme_events_in_profile) { - nav_set_selected('home'); + Nav::setSelected('home'); } else { - nav_set_selected('events'); + Nav::setSelected('events'); } // get the translation strings for the callendar diff --git a/mod/help.php b/mod/help.php index ab5223a397..b9c2bae3a0 100644 --- a/mod/help.php +++ b/mod/help.php @@ -1,6 +1,9 @@ true, @@ -575,7 +576,7 @@ function networkThreadedView(App $a, $update = 0) { } } - nav_set_selected('network'); + Nav::setSelected('network'); $content = ""; diff --git a/mod/notes.php b/mod/notes.php index a98577e546..c02f0a759e 100644 --- a/mod/notes.php +++ b/mod/notes.php @@ -3,6 +3,7 @@ * @file mod/notes.php */ use Friendica\App; +use Friendica\Content\Nav; use Friendica\Database\DBM; use Friendica\Model\Profile; @@ -16,7 +17,7 @@ function notes_init(App $a) { $which = $a->user['nickname']; - nav_set_selected('home'); + Nav::setSelected('home'); //Profile::load($a, $which, $profile); diff --git a/mod/notifications.php b/mod/notifications.php index 4d549b46db..f44007ce31 100644 --- a/mod/notifications.php +++ b/mod/notifications.php @@ -5,6 +5,7 @@ */ use Friendica\App; use Friendica\Content\ContactSelector; +use Friendica\Content\Nav; use Friendica\Core\NotificationsManager; use Friendica\Core\System; use Friendica\Database\DBM; @@ -77,7 +78,7 @@ function notifications_content(App $a) { $page = (x($_REQUEST,'page') ? $_REQUEST['page'] : 1); $show = (x($_REQUEST,'show') ? $_REQUEST['show'] : 0); - nav_set_selected('notifications'); + Nav::setSelected('notifications'); $json = (($a->argc > 1 && $a->argv[$a->argc - 1] === 'json') ? true : false); @@ -93,8 +94,8 @@ function notifications_content(App $a) { $startrec = ($page * $perpage) - $perpage; // Get introductions - if( (($a->argc > 1) && ($a->argv[1] == 'intros')) || (($a->argc == 1))) { - nav_set_selected('introductions'); + if ((($a->argc > 1) && ($a->argv[1] == 'intros')) || (($a->argc == 1))) { + Nav::setSelected('introductions'); $notif_header = t('Notifications'); $all = (($a->argc > 2) && ($a->argv[2] == 'all')); diff --git a/mod/photos.php b/mod/photos.php index 5714b0e866..f399c3ae80 100644 --- a/mod/photos.php +++ b/mod/photos.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\Feature; +use Friendica\Content\Nav; use Friendica\Core\System; use Friendica\Core\Config; use Friendica\Core\Worker; @@ -33,7 +34,7 @@ function photos_init(App $a) { return; } - nav_set_selected('home'); + Nav::setSelected('home'); if ($a->argc > 1) { $nick = $a->argv[1]; diff --git a/mod/profile.php b/mod/profile.php index 1982605127..4b1d4d2a0a 100644 --- a/mod/profile.php +++ b/mod/profile.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\Widget; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Core\PConfig; use Friendica\Core\System; @@ -125,7 +126,7 @@ function profile_content(App $a, $update = 0) // Ensure we've got a profile owner if updating. $a->profile['profile_uid'] = $update; } elseif ($a->profile['profile_uid'] == local_user()) { - nav_set_selected('home'); + Nav::setSelected('home'); } $contact = null; diff --git a/mod/profiles.php b/mod/profiles.php index 93571db9cd..f8d77e27d8 100644 --- a/mod/profiles.php +++ b/mod/profiles.php @@ -5,6 +5,7 @@ use Friendica\App; use Friendica\Content\ContactSelector; use Friendica\Content\Feature; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Core\PConfig; use Friendica\Core\System; @@ -16,7 +17,7 @@ use Friendica\Network\Probe; function profiles_init(App $a) { - nav_set_selected('profiles'); + Nav::setSelected('profiles'); if (! local_user()) { return; diff --git a/mod/search.php b/mod/search.php index 08b21d1e44..60995e98a3 100644 --- a/mod/search.php +++ b/mod/search.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\Feature; +use Friendica\Content\Nav; use Friendica\Core\Cache; use Friendica\Core\Config; use Friendica\Database\DBM; @@ -132,7 +133,7 @@ function search_content(App $a) { Cache::set("remote_search:".$remote, json_encode(["time" => time(), "accesses" => 1]), CACHE_HOUR); } - nav_set_selected('search'); + Nav::setSelected('search'); if (x($a->data,'search')) $search = notags(trim($a->data['search'])); diff --git a/mod/settings.php b/mod/settings.php index 06ac0794b8..572857a48b 100644 --- a/mod/settings.php +++ b/mod/settings.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\Feature; +use Friendica\Content\Nav; use Friendica\Core\System; use Friendica\Core\Worker; use Friendica\Core\Config; @@ -654,7 +655,7 @@ function settings_post(App $a) function settings_content(App $a) { $o = ''; - nav_set_selected('settings'); + Nav::setSelected('settings'); if (!local_user()) { //notice(t('Permission denied.') . EOL); diff --git a/mod/videos.php b/mod/videos.php index 6dfbb74ed3..239d86168c 100644 --- a/mod/videos.php +++ b/mod/videos.php @@ -3,6 +3,7 @@ * @file mod/videos.php */ use Friendica\App; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Core\System; use Friendica\Core\Worker; @@ -26,7 +27,7 @@ function videos_init(App $a) { return; } - nav_set_selected('home'); + Nav::setSelected('home'); $o = ''; diff --git a/mod/viewcontacts.php b/mod/viewcontacts.php index a918c1ae57..22ab555b3b 100644 --- a/mod/viewcontacts.php +++ b/mod/viewcontacts.php @@ -4,6 +4,7 @@ */ use Friendica\App; use Friendica\Content\ContactSelector; +use Friendica\Content\Nav; use Friendica\Core\Config; use Friendica\Database\DBM; use Friendica\Model\Contact; @@ -15,7 +16,7 @@ function viewcontacts_init(App $a) { return; } - nav_set_selected('home'); + Nav::setSelected('home'); if($a->argc > 1) { $nick = $a->argv[1]; diff --git a/src/Content/Nav.php b/src/Content/Nav.php index 20ae440b1c..8445d827f5 100644 --- a/src/Content/Nav.php +++ b/src/Content/Nav.php @@ -11,21 +11,22 @@ use Friendica\Core\System; use Friendica\Database\DBM; use Friendica\Model\Profile; +require_once 'boot.php'; +require_once 'include/pgettext.php'; +require_once 'include/text.php'; + class Nav { - function nav(App $a) + /** + * Build page header and site navigation bars + */ + public static function build(App $a) { - /* - * - * Build page header and site navigation bars - * - */ - if (!(x($a->page, 'nav'))) { $a->page['nav'] = ''; } - $a->page['htmlhead'] .= replace_macros(get_markup_template('nav_head.tpl'), array()); + $a->page['htmlhead'] .= replace_macros(get_markup_template('nav_head.tpl'), []); /* * Placeholder div for popup panel @@ -33,7 +34,7 @@ class Nav $a->page['nav'] .= '' ; - $nav_info = nav_info($a); + $nav_info = self::getInfo($a); /* * Build the page @@ -41,25 +42,28 @@ class Nav $tpl = get_markup_template('nav.tpl'); - $a->page['nav'] .= replace_macros($tpl, array( - '$baseurl' => System::baseUrl(), - '$sitelocation' => $nav_info['sitelocation'], - '$nav' => $nav_info['nav'], - '$banner' => $nav_info['banner'], - '$emptynotifications' => t('Nothing new here'), - '$userinfo' => $nav_info['userinfo'], - '$sel' => $a->nav_sel, - '$apps' => $a->apps, - '$clear_notifs' => t('Clear notifications'), - '$search_hint' => t('@name, !forum, #tags, content') - )); + $a->page['nav'] .= replace_macros($tpl, + [ + '$baseurl' => System::baseUrl(), + '$sitelocation' => $nav_info['sitelocation'], + '$nav' => $nav_info['nav'], + '$banner' => $nav_info['banner'], + '$emptynotifications' => t('Nothing new here'), + '$userinfo' => $nav_info['userinfo'], + '$sel' => $a->nav_sel, + '$apps' => $a->apps, + '$clear_notifs' => t('Clear notifications'), + '$search_hint' => t('@name, !forum, #tags, content') + ] + ); call_hooks('page_header', $a->page['nav']); } /** - * @brief Prepares a list of navigation links + * Prepares a list of navigation links * + * @brief Prepares a list of navigation links * @param App $a * @return array Navigation links * string 'sitelocation' => The webbie (username@site.com) @@ -67,7 +71,7 @@ class Nav * string 'banner' => Formatted html link with banner image * array 'userinfo' => Array of user information (name, icon) */ - function nav_info(App $a) + private static function getInfo(App $a) { $ssl_state = ((local_user()) ? true : false); @@ -82,31 +86,31 @@ class Nav $sitelocation = $myident . substr(System::baseUrl($ssl_state), strpos(System::baseUrl($ssl_state), '//') + 2); // nav links: array of array('href', 'text', 'extra css classes', 'title') - $nav = array(); + $nav = []; // Display login or logout - $nav['usermenu'] = array(); + $nav['usermenu'] = []; $userinfo = null; if (local_user()) { - $nav['logout'] = array('logout', t('Logout'), '', t('End this session')); + $nav['logout'] = ['logout', t('Logout'), '', t('End this session')]; // user menu - $nav['usermenu'][] = array('profile/' . $a->user['nickname'], t('Status'), '', t('Your posts and conversations')); - $nav['usermenu'][] = array('profile/' . $a->user['nickname'] . '?tab=profile', t('Profile'), '', t('Your profile page')); - $nav['usermenu'][] = array('photos/' . $a->user['nickname'], t('Photos'), '', t('Your photos')); - $nav['usermenu'][] = array('videos/' . $a->user['nickname'], t('Videos'), '', t('Your videos')); - $nav['usermenu'][] = array('events/', t('Events'), '', t('Your events')); - $nav['usermenu'][] = array('notes/', t('Personal notes'), '', t('Your personal notes')); + $nav['usermenu'][] = ['profile/' . $a->user['nickname'], t('Status'), '', t('Your posts and conversations')]; + $nav['usermenu'][] = ['profile/' . $a->user['nickname'] . '?tab=profile', t('Profile'), '', t('Your profile page')]; + $nav['usermenu'][] = ['photos/' . $a->user['nickname'], t('Photos'), '', t('Your photos')]; + $nav['usermenu'][] = ['videos/' . $a->user['nickname'], t('Videos'), '', t('Your videos')]; + $nav['usermenu'][] = ['events/', t('Events'), '', t('Your events')]; + $nav['usermenu'][] = ['notes/', t('Personal notes'), '', t('Your personal notes')]; // user info $contact = dba::selectFirst('contact', ['micro'], ['uid' => $a->user['uid'], 'self' => true]); - $userinfo = array( + $userinfo = [ 'icon' => (DBM::is_result($contact) ? $a->remove_baseurl($contact['micro']) : 'images/person-48.jpg'), 'name' => $a->user['username'], - ); + ]; } else { - $nav['login'] = array('login', t('Login'), ($a->module == 'login' ? 'selected' : ''), t('Sign in')); + $nav['login'] = ['login', t('Login'), ($a->module == 'login' ? 'selected' : ''), t('Sign in')]; } // "Home" should also take you home from an authenticated remote profile connection @@ -116,30 +120,31 @@ class Nav } if (($a->module != 'home') && (! (local_user()))) { - $nav['home'] = array($homelink, t('Home'), '', t('Home Page')); + $nav['home'] = [$homelink, t('Home'), '', t('Home Page')]; } if (($a->config['register_policy'] == REGISTER_OPEN) && (! local_user()) && (! remote_user())) { - $nav['register'] = array('register', t('Register'), '', t('Create an account')); + $nav['register'] = ['register', t('Register'), '', t('Create an account')]; } $help_url = 'help'; if (!Config::get('system', 'hide_help')) { - $nav['help'] = array($help_url, t('Help'), '', t('Help and documentation')); + $nav['help'] = [$help_url, t('Help'), '', t('Help and documentation')]; } if (count($a->apps) > 0) { - $nav['apps'] = array('apps', t('Apps'), '', t('Addon applications, utilities, games')); + $nav['apps'] = ['apps', t('Apps'), '', t('Addon applications, utilities, games')]; } if (local_user() || !Config::get('system', 'local_search')) { - $nav['search'] = array('search', t('Search'), '', t('Search site content')); + $nav['search'] = ['search', t('Search'), '', t('Search site content')]; - $nav['searchoption'] = array( + $nav['searchoption'] = [ t('Full Text'), t('Tags'), - t('Contacts')); + t('Contacts') + ]; if (Config::get('system', 'poco_local_search')) { $nav['searchoption'][] = t('Forums'); @@ -156,23 +161,23 @@ class Nav } if (local_user() || Config::get('system', 'community_page_style') != CP_NO_COMMUNITY_PAGE) { - $nav['community'] = array('community', t('Community'), '', t('Conversations on this and other servers')); + $nav['community'] = ['community', t('Community'), '', t('Conversations on this and other servers')]; } if (local_user()) { - $nav['events'] = array('events', t('Events'), '', t('Events and Calendar')); + $nav['events'] = ['events', t('Events'), '', t('Events and Calendar')]; } - $nav['directory'] = array($gdirpath, t('Directory'), '', t('People directory')); + $nav['directory'] = [$gdirpath, t('Directory'), '', t('People directory')]; - $nav['about'] = array('friendica', t('Information'), '', t('Information about this friendica instance')); + $nav['about'] = ['friendica', t('Information'), '', t('Information about this friendica instance')]; // The following nav links are only show to logged in users if (local_user()) { - $nav['network'] = array('network', t('Network'), '', t('Conversations from your friends')); - $nav['net_reset'] = array('network/0?f=&order=comment&nets=all', t('Network Reset'), '', t('Load Network page with no filters')); + $nav['network'] = ['network', t('Network'), '', t('Conversations from your friends')]; + $nav['net_reset'] = ['network/0?f=&order=comment&nets=all', t('Network Reset'), '', t('Load Network page with no filters')]; - $nav['home'] = array('profile/' . $a->user['nickname'], t('Home'), '', t('Your posts and conversations')); + $nav['home'] = ['profile/' . $a->user['nickname'], t('Home'), '', t('Your posts and conversations')]; if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE, PAGE_PRVGROUP))) { // only show friend requests for normal pages. Other page types have automatic friendship. @@ -180,38 +185,38 @@ class Nav $nav['introductions'] = array('notifications/intros', t('Introductions'), '', t('Friend Requests')); } if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE))) { - $nav['notifications'] = array('notifications', t('Notifications'), '', t('Notifications')); - $nav['notifications']['all'] = array('notifications/system', t('See all notifications'), '', ''); - $nav['notifications']['mark'] = array('', t('Mark as seen'), '', t('Mark all system notifications seen')); + $nav['notifications'] = ['notifications', t('Notifications'), '', t('Notifications')]; + $nav['notifications']['all'] = ['notifications/system', t('See all notifications'), '', '']; + $nav['notifications']['mark'] = ['', t('Mark as seen'), '', t('Mark all system notifications seen')]; } } - $nav['messages'] = array('message', t('Messages'), '', t('Private mail')); - $nav['messages']['inbox'] = array('message', t('Inbox'), '', t('Inbox')); - $nav['messages']['outbox'] = array('message/sent', t('Outbox'), '', t('Outbox')); - $nav['messages']['new'] = array('message/new', t('New Message'), '', t('New Message')); + $nav['messages'] = ['message', t('Messages'), '', t('Private mail')]; + $nav['messages']['inbox'] = ['message', t('Inbox'), '', t('Inbox')]; + $nav['messages']['outbox'] = ['message/sent', t('Outbox'), '', t('Outbox')]; + $nav['messages']['new'] = ['message/new', t('New Message'), '', t('New Message')]; if (is_array($a->identities) && count($a->identities) > 1) { - $nav['manage'] = array('manage', t('Manage'), '', t('Manage other pages')); + $nav['manage'] = ['manage', t('Manage'), '', t('Manage other pages')]; } - $nav['delegations'] = array('delegate', t('Delegations'), '', t('Delegate Page Management')); + $nav['delegations'] = ['delegate', t('Delegations'), '', t('Delegate Page Management')]; - $nav['settings'] = array('settings', t('Settings'), '', t('Account settings')); + $nav['settings'] = ['settings', t('Settings'), '', t('Account settings')]; if (Feature::isEnabled(local_user(), 'multi_profiles')) { - $nav['profiles'] = array('profiles', t('Profiles'), '', t('Manage/Edit Profiles')); + $nav['profiles'] = ['profiles', t('Profiles'), '', t('Manage/Edit Profiles')]; } - $nav['contacts'] = array('contacts', t('Contacts'), '', t('Manage/edit friends and contacts')); + $nav['contacts'] = ['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')); + $nav['admin'] = ['admin/', t('Admin'), '', t('Site setup and configuration')]; } - $nav['navigation'] = array('navigation/', t('Navigation'), '', t('Site map')); + $nav['navigation'] = ['navigation/', t('Navigation'), '', t('Site map')]; // Provide a banner/logo/whatever $banner = Config::get('system', 'banner'); @@ -221,22 +226,21 @@ class Nav call_hooks('nav_info', $nav); - return array( + return [ 'sitelocation' => $sitelocation, 'nav' => $nav, 'banner' => $banner, 'userinfo' => $userinfo, - ); + ]; } /** * Set a menu item in navbar as selected - * */ - function nav_set_selected($item) + public static function setSelected($item) { $a = get_app(); - $a->nav_sel = array( + $a->nav_sel = [ 'global' => null, 'community' => null, 'network' => null, @@ -250,8 +254,8 @@ class Nav 'contacts' => null, 'manage' => null, 'events' => null, - 'register' => null, - ); + 'register' => null + ]; $a->nav_sel[$item] = 'selected'; } } From bd63a36269e8298b4335b9ac66abd558ab046be2 Mon Sep 17 00:00:00 2001 From: Adam Magness Date: Mon, 15 Jan 2018 17:08:36 -0500 Subject: [PATCH 6/8] use dba add use dba --- src/Content/Nav.php | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/Content/Nav.php b/src/Content/Nav.php index 8445d827f5..a576423bfd 100644 --- a/src/Content/Nav.php +++ b/src/Content/Nav.php @@ -10,8 +10,10 @@ use Friendica\Core\Config; use Friendica\Core\System; use Friendica\Database\DBM; use Friendica\Model\Profile; +use dba; require_once 'boot.php'; +require_once 'dba.php'; require_once 'include/pgettext.php'; require_once 'include/text.php'; From bb08f87bbacb0436f774341f91dd15be34f12229 Mon Sep 17 00:00:00 2001 From: Adam Magness Date: Mon, 15 Jan 2018 17:12:39 -0500 Subject: [PATCH 7/8] Remove old file remove include/nav.php --- include/nav.php | 250 -------------------------------------------- src/Content/Nav.php | 6 +- 2 files changed, 2 insertions(+), 254 deletions(-) delete mode 100644 include/nav.php diff --git a/include/nav.php b/include/nav.php deleted file mode 100644 index 3cee38b342..0000000000 --- a/include/nav.php +++ /dev/null @@ -1,250 +0,0 @@ -page,'nav'))) - $a->page['nav'] = ''; - - $a->page['htmlhead'] .= replace_macros(get_markup_template('nav_head.tpl'), []); - - /* - * Placeholder div for popup panel - */ - - $a->page['nav'] .= '' ; - - $nav_info = nav_info($a); - - /* - * Build the page - */ - - $tpl = get_markup_template('nav.tpl'); - - $a->page['nav'] .= replace_macros($tpl, [ - '$baseurl' => System::baseUrl(), - '$sitelocation' => $nav_info['sitelocation'], - '$nav' => $nav_info['nav'], - '$banner' => $nav_info['banner'], - '$emptynotifications' => t('Nothing new here'), - '$userinfo' => $nav_info['userinfo'], - '$sel' => $a->nav_sel, - '$apps' => $a->apps, - '$clear_notifs' => t('Clear notifications'), - '$search_hint' => t('@name, !forum, #tags, content') - ]); - - call_hooks('page_header', $a->page['nav']); -} - -/** - * @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); - - /* - * Our network is distributed, and as you visit friends some of the - * sites look exactly the same - it isn't always easy to know where you are. - * Display the current site location as a navigation aid. - */ - - $myident = ((is_array($a->user) && isset($a->user['nickname'])) ? $a->user['nickname'] . '@' : ''); - - $sitelocation = $myident . substr(System::baseUrl($ssl_state), strpos(System::baseUrl($ssl_state), '//') + 2 ); - - // nav links: array of array('href', 'text', 'extra css classes', 'title') - $nav = []; - - // Display login or logout - $nav['usermenu'] = []; - $userinfo = null; - - if (local_user()) { - $nav['logout'] = ['logout', t('Logout'), '', t('End this session')]; - - // user menu - $nav['usermenu'][] = ['profile/' . $a->user['nickname'], t('Status'), '', t('Your posts and conversations')]; - $nav['usermenu'][] = ['profile/' . $a->user['nickname'] . '?tab=profile', t('Profile'), '', t('Your profile page')]; - $nav['usermenu'][] = ['photos/' . $a->user['nickname'], t('Photos'), '', t('Your photos')]; - $nav['usermenu'][] = ['videos/' . $a->user['nickname'], t('Videos'), '', t('Your videos')]; - $nav['usermenu'][] = ['events/', t('Events'), '', t('Your events')]; - $nav['usermenu'][] = ['notes/', t('Personal notes'), '', t('Your personal notes')]; - - // user info - $contact = dba::selectFirst('contact', ['micro'], ['uid' => $a->user['uid'], 'self' => true]); - $userinfo = [ - 'icon' => (DBM::is_result($contact) ? $a->remove_baseurl($contact['micro']) : 'images/person-48.jpg'), - 'name' => $a->user['username'], - ]; - } else { - $nav['login'] = ['login', t('Login'), ($a->module == 'login' ? 'selected' : ''), t('Sign in')]; - } - - // "Home" should also take you home from an authenticated remote profile connection - $homelink = Profile::getMyURL(); - if (! $homelink) { - $homelink = ((x($_SESSION, 'visitor_home')) ? $_SESSION['visitor_home'] : ''); - } - - if (($a->module != 'home') && (! (local_user()))) { - $nav['home'] = [$homelink, t('Home'), '', t('Home Page')]; - } - - if (($a->config['register_policy'] == REGISTER_OPEN) && (! local_user()) && (! remote_user())) { - $nav['register'] = ['register', t('Register'), '', t('Create an account')]; - } - - $help_url = 'help'; - - if (!Config::get('system', 'hide_help')) { - $nav['help'] = [$help_url, t('Help'), '', t('Help and documentation')]; - } - - if (count($a->apps) > 0) { - $nav['apps'] = ['apps', t('Apps'), '', t('Addon applications, utilities, games')]; - } - - if (local_user() || !Config::get('system', 'local_search')) { - $nav['search'] = ['search', t('Search'), '', t('Search site content')]; - - $nav['searchoption'] = [ - t('Full Text'), - t('Tags'), - t('Contacts')]; - - if (Config::get('system', 'poco_local_search')) { - $nav['searchoption'][] = t('Forums'); - } - } - - $gdirpath = 'directory'; - - if (strlen(Config::get('system', 'singleuser'))) { - $gdir = Config::get('system', 'directory'); - if (strlen($gdir)) { - $gdirpath = Profile::zrl($gdir, true); - } - } - - if (local_user() || Config::get('system', 'community_page_style') != CP_NO_COMMUNITY_PAGE) { - $nav['community'] = ['community', t('Community'), '', t('Conversations on this and other servers')]; - } - - if (local_user()) { - $nav['events'] = ['events', t('Events'), '', t('Events and Calendar')]; - } - - $nav['directory'] = [$gdirpath, t('Directory'), '', t('People directory')]; - - $nav['about'] = ['friendica', t('Information'), '', t('Information about this friendica instance')]; - - // The following nav links are only show to logged in users - if (local_user()) { - $nav['network'] = ['network', t('Network'), '', t('Conversations from your friends')]; - $nav['net_reset'] = ['network/0?f=&order=comment&nets=all', t('Network Reset'), '', t('Load Network page with no filters')]; - - $nav['home'] = ['profile/' . $a->user['nickname'], t('Home'), '', t('Your posts and conversations')]; - - if (in_array($_SESSION['page_flags'], [PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE, PAGE_PRVGROUP])) { - // only show friend requests for normal pages. Other page types have automatic friendship. - if (in_array($_SESSION['page_flags'], [PAGE_NORMAL, PAGE_SOAPBOX, PAGE_PRVGROUP])) { - $nav['introductions'] = ['notifications/intros', t('Introductions'), '', t('Friend Requests')]; - } - if (in_array($_SESSION['page_flags'], [PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE])) { - $nav['notifications'] = ['notifications', t('Notifications'), '', t('Notifications')]; - $nav['notifications']['all'] = ['notifications/system', t('See all notifications'), '', '']; - $nav['notifications']['mark'] = ['', t('Mark as seen'), '', t('Mark all system notifications seen')]; - } - } - - $nav['messages'] = ['message', t('Messages'), '', t('Private mail')]; - $nav['messages']['inbox'] = ['message', t('Inbox'), '', t('Inbox')]; - $nav['messages']['outbox'] = ['message/sent', t('Outbox'), '', t('Outbox')]; - $nav['messages']['new'] = ['message/new', t('New Message'), '', t('New Message')]; - - if (is_array($a->identities) && count($a->identities) > 1) { - $nav['manage'] = ['manage', t('Manage'), '', t('Manage other pages')]; - } - - $nav['delegations'] = ['delegate', t('Delegations'), '', t('Delegate Page Management')]; - - $nav['settings'] = ['settings', t('Settings'), '', t('Account settings')]; - - if (Feature::isEnabled(local_user(), 'multi_profiles')) { - $nav['profiles'] = ['profiles', t('Profiles'), '', t('Manage/Edit Profiles')]; - } - - $nav['contacts'] = ['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'] = ['admin/', t('Admin'), '', t('Site setup and configuration')]; - } - - $nav['navigation'] = ['navigation/', t('Navigation'), '', t('Site map')]; - - // Provide a banner/logo/whatever - $banner = Config::get('system', 'banner'); - if (is_null($banner)) { - $banner = 'logoFriendica'; - } - - call_hooks('nav_info', $nav); - - return [ - 'sitelocation' => $sitelocation, - 'nav' => $nav, - 'banner' => $banner, - 'userinfo' => $userinfo, - ]; -} - -/** - * Set a menu item in navbar as selected - * - */ -function nav_set_selected($item){ - $a = get_app(); - $a->nav_sel = [ - 'global' => null, - 'community' => null, - 'network' => null, - 'home' => null, - 'profiles' => null, - 'introductions' => null, - 'notifications' => null, - 'messages' => null, - 'directory' => null, - 'settings' => null, - 'contacts' => null, - 'manage' => null, - 'events' => null, - 'register' => null, - ]; - $a->nav_sel[$item] = 'selected'; -} diff --git a/src/Content/Nav.php b/src/Content/Nav.php index a576423bfd..0d05d03510 100644 --- a/src/Content/Nav.php +++ b/src/Content/Nav.php @@ -44,8 +44,7 @@ class Nav $tpl = get_markup_template('nav.tpl'); - $a->page['nav'] .= replace_macros($tpl, - [ + $a->page['nav'] .= replace_macros($tpl, [ '$baseurl' => System::baseUrl(), '$sitelocation' => $nav_info['sitelocation'], '$nav' => $nav_info['nav'], @@ -56,8 +55,7 @@ class Nav '$apps' => $a->apps, '$clear_notifs' => t('Clear notifications'), '$search_hint' => t('@name, !forum, #tags, content') - ] - ); + ]); call_hooks('page_header', $a->page['nav']); } From 246e673659c3efba4fb3f38cd86dc7da5053d70e Mon Sep 17 00:00:00 2001 From: Adam Magness Date: Mon, 15 Jan 2018 17:42:12 -0500 Subject: [PATCH 8/8] Review updates --- src/Content/Nav.php | 36 ++++++++++++++++++------------------ 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/src/Content/Nav.php b/src/Content/Nav.php index 0d05d03510..4bbc7900d4 100644 --- a/src/Content/Nav.php +++ b/src/Content/Nav.php @@ -45,16 +45,16 @@ class Nav $tpl = get_markup_template('nav.tpl'); $a->page['nav'] .= replace_macros($tpl, [ - '$baseurl' => System::baseUrl(), - '$sitelocation' => $nav_info['sitelocation'], - '$nav' => $nav_info['nav'], - '$banner' => $nav_info['banner'], - '$emptynotifications' => t('Nothing new here'), - '$userinfo' => $nav_info['userinfo'], - '$sel' => $a->nav_sel, - '$apps' => $a->apps, - '$clear_notifs' => t('Clear notifications'), - '$search_hint' => t('@name, !forum, #tags, content') + '$baseurl' => System::baseUrl(), + '$sitelocation' => $nav_info['sitelocation'], + '$nav' => $nav_info['nav'], + '$banner' => $nav_info['banner'], + '$emptynotifications' => t('Nothing new here'), + '$userinfo' => $nav_info['userinfo'], + '$sel' => $a->nav_sel, + '$apps' => $a->apps, + '$clear_notifs' => t('Clear notifications'), + '$search_hint' => t('@name, !forum, #tags, content') ]); call_hooks('page_header', $a->page['nav']); @@ -141,10 +141,10 @@ class Nav $nav['search'] = ['search', t('Search'), '', t('Search site content')]; $nav['searchoption'] = [ - t('Full Text'), - t('Tags'), - t('Contacts') - ]; + t('Full Text'), + t('Tags'), + t('Contacts') + ]; if (Config::get('system', 'poco_local_search')) { $nav['searchoption'][] = t('Forums'); @@ -179,12 +179,12 @@ class Nav $nav['home'] = ['profile/' . $a->user['nickname'], t('Home'), '', t('Your posts and conversations')]; - if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE, PAGE_PRVGROUP))) { + if (in_array($_SESSION['page_flags'], [PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE, PAGE_PRVGROUP])) { // only show friend requests for normal pages. Other page types have automatic friendship. - if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_PRVGROUP))) { - $nav['introductions'] = array('notifications/intros', t('Introductions'), '', t('Friend Requests')); + if (in_array($_SESSION['page_flags'], [PAGE_NORMAL, PAGE_SOAPBOX, PAGE_PRVGROUP])) { + $nav['introductions'] = ['notifications/intros', t('Introductions'), '', t('Friend Requests')]; } - if (in_array($_SESSION['page_flags'], array(PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE))) { + if (in_array($_SESSION['page_flags'], [PAGE_NORMAL, PAGE_SOAPBOX, PAGE_FREELOVE])) { $nav['notifications'] = ['notifications', t('Notifications'), '', t('Notifications')]; $nav['notifications']['all'] = ['notifications/system', t('See all notifications'), '', '']; $nav['notifications']['mark'] = ['', t('Mark as seen'), '', t('Mark all system notifications seen')];