diff --git a/index.php b/index.php index fa6d8b087..34952afcb 100644 --- a/index.php +++ b/index.php @@ -45,6 +45,7 @@ $a->runFrontend( $dice->create(\Friendica\Core\PConfig\Capability\IManagePersonalConfigValues::class), $dice->create(\Friendica\Security\Authentication::class), $dice->create(\Friendica\App\Page::class), + $dice->create(\Friendica\Content\Nav::class), $dice->create(Friendica\Module\Special\HTTPException::class), new \Friendica\Util\HTTPInputData($_SERVER), $start_time diff --git a/src/App.php b/src/App.php index 224a895e2..378ddbc9d 100644 --- a/src/App.php +++ b/src/App.php @@ -25,6 +25,7 @@ use Exception; use Friendica\App\Arguments; use Friendica\App\BaseURL; use Friendica\Capabilities\ICanCreateResponses; +use Friendica\Content\Nav; use Friendica\Core\Config\Factory\Config; use Friendica\Core\Session\Capability\IHandleUserSessions; use Friendica\Database\Definition\DbaDefinition; @@ -579,7 +580,7 @@ class App * @throws HTTPException\InternalServerErrorException * @throws \ImagickException */ - public function runFrontend(App\Router $router, IManagePersonalConfigValues $pconfig, Authentication $auth, App\Page $page, ModuleHTTPException $httpException, HTTPInputData $httpInput, float $start_time) + public function runFrontend(App\Router $router, IManagePersonalConfigValues $pconfig, Authentication $auth, App\Page $page, Nav $nav, ModuleHTTPException $httpException, HTTPInputData $httpInput, float $start_time) { $this->profiler->set($start_time, 'start'); $this->profiler->set(microtime(true), 'classinit'); @@ -718,7 +719,7 @@ class App $response = $module->run($httpException, $input); $this->profiler->set(microtime(true) - $timestamp, 'content'); if ($response->getHeaderLine(ICanCreateResponses::X_HEADER) === ICanCreateResponses::TYPE_HTML) { - $page->run($this, $this->baseURL, $this->args, $this->mode, $response, $this->l10n, $this->profiler, $this->config, $pconfig, $this->session->getLocalUserId()); + $page->run($this, $this->baseURL, $this->args, $this->mode, $response, $this->l10n, $this->profiler, $this->config, $pconfig, $nav, $this->session->getLocalUserId()); } else { $page->exit($response); } diff --git a/src/App/Page.php b/src/App/Page.php index f7adb18d6..3c746ebcc 100644 --- a/src/App/Page.php +++ b/src/App/Page.php @@ -27,18 +27,18 @@ use DOMXPath; use Friendica\App; use Friendica\Content\Nav; use Friendica\Core\Config\Capability\IManageConfigValues; -use Friendica\Core\PConfig\Capability\IManagePersonalConfigValues; use Friendica\Core\Hook; use Friendica\Core\L10n; use Friendica\Core\Logger; +use Friendica\Core\PConfig\Capability\IManagePersonalConfigValues; use Friendica\Core\Renderer; use Friendica\Core\System; use Friendica\Core\Theme; use Friendica\Module\Response; use Friendica\Network\HTTPException; use Friendica\Util\Network; -use Friendica\Util\Strings; use Friendica\Util\Profiler; +use Friendica\Util\Strings; use Psr\Http\Message\ResponseInterface; /** @@ -407,13 +407,16 @@ class Page implements ArrayAccess * @param Mode $mode The current node mode * @param ResponseInterface $response The Response of the module class, including type, content & headers * @param L10n $l10n The l10n language class + * @param Profiler $profiler * @param IManageConfigValues $config The Configuration of this node * @param IManagePersonalConfigValues $pconfig The personal/user configuration - * @param int $localUID The UID of the local user - * - * @throws HTTPException\InternalServerErrorException|HTTPException\ServiceUnavailableException + * @param Nav $nav + * @param int $localUID + * @throws HTTPException\MethodNotAllowedException + * @throws HTTPException\InternalServerErrorException + * @throws HTTPException\ServiceUnavailableException */ - public function run(App $app, BaseURL $baseURL, Arguments $args, Mode $mode, ResponseInterface $response, L10n $l10n, Profiler $profiler, IManageConfigValues $config, IManagePersonalConfigValues $pconfig, int $localUID) + public function run(App $app, BaseURL $baseURL, Arguments $args, Mode $mode, ResponseInterface $response, L10n $l10n, Profiler $profiler, IManageConfigValues $config, IManagePersonalConfigValues $pconfig, Nav $nav, int $localUID) { $moduleName = $args->getModuleName(); @@ -463,7 +466,7 @@ class Page implements ArrayAccess // Add the navigation (menu) template if ($moduleName != 'install' && $moduleName != 'maintenance') { $this->page['htmlhead'] .= Renderer::replaceMacros(Renderer::getMarkupTemplate('nav_head.tpl'), []); - $this->page['nav'] = Nav::build($app); + $this->page['nav'] = $nav->getHtml(); } foreach ($response->getHeaders() as $key => $header) { diff --git a/src/Content/Nav.php b/src/Content/Nav.php index 609e0e331..63eeede89 100644 --- a/src/Content/Nav.php +++ b/src/Content/Nav.php @@ -21,15 +21,21 @@ namespace Friendica\Content; -use Friendica\App; +use Friendica\App\BaseURL; +use Friendica\App\Router; +use Friendica\Core\Config\Capability\IManageConfigValues; use Friendica\Core\Hook; +use Friendica\Core\L10n; use Friendica\Core\Renderer; -use Friendica\Database\DBA; -use Friendica\DI; +use Friendica\Core\Session\Capability\IHandleUserSessions; +use Friendica\Database\Database; use Friendica\Model\Contact; use Friendica\Model\Profile; use Friendica\Model\User; use Friendica\Module\Conversation\Community; +use Friendica\Module\Home; +use Friendica\Module\Security\Login; +use Friendica\Network\HTTPException; class Nav { @@ -53,9 +59,32 @@ class Nav /** * An array of HTML links provided by addons providing a module via the app_menu hook * - * @var array + * @var array|null */ - private static $app_menu = null; + private $appMenu = null; + + /** @var BaseURL */ + private $baseUrl; + /** @var L10n */ + private $l10n; + /** @var IHandleUserSessions */ + private $session; + /** @var Database */ + private $database; + /** @var IManageConfigValues */ + private $config; + /** @var Router */ + private $router; + + public function __construct(BaseURL $baseUrl, L10n $l10n, IHandleUserSessions $session, Database $database, IManageConfigValues $config, Router $router) + { + $this->baseUrl = $baseUrl; + $this->l10n = $l10n; + $this->session = $session; + $this->database = $database; + $this->config = $config; + $this->router = $router; + } /** * Set a menu item in navbar as selected @@ -70,16 +99,17 @@ class Nav /** * Build page header and site navigation bars * - * @param App $a * @return string - * @throws \Friendica\Network\HTTPException\InternalServerErrorException + * @throws HTTPException\InternalServerErrorException + * @throws HTTPException\MethodNotAllowedException + * @throws HTTPException\ServiceUnavailableException */ - public static function build(App $a): string + public function getHtml(): string { // Placeholder div for popup panel $nav = '
'; - $nav_info = self::getInfo($a); + $nav_info = $this->getInfo(); $tpl = Renderer::getMarkupTemplate('nav.tpl'); @@ -87,13 +117,13 @@ class Nav '$sitelocation' => $nav_info['sitelocation'], '$nav' => $nav_info['nav'], '$banner' => $nav_info['banner'], - '$emptynotifications' => DI::l10n()->t('Nothing new here'), + '$emptynotifications' => $this->l10n->t('Nothing new here'), '$userinfo' => $nav_info['userinfo'], '$sel' => self::$selected, - '$apps' => self::getAppMenu(), - '$home' => DI::l10n()->t('Go back'), - '$clear_notifs' => DI::l10n()->t('Clear notifications'), - '$search_hint' => DI::l10n()->t('@name, !forum, #tags, content') + '$apps' => $this->getAppMenu(), + '$home' => $this->l10n->t('Go back'), + '$clear_notifs' => $this->l10n->t('Clear notifications'), + '$search_hint' => $this->l10n->t('@name, !forum, #tags, content') ]); Hook::callAll('page_header', $nav); @@ -105,60 +135,66 @@ class Nav * Returns the addon app menu * * @return array + * @throws HTTPException\InternalServerErrorException */ - public static function getAppMenu(): array + public function getAppMenu(): array { - if (is_null(self::$app_menu)) { - self::populateAppMenu(); + if (is_null($this->appMenu)) { + $this->appMenu = $this->populateAppMenu(); } - return self::$app_menu; + return $this->appMenu; } /** - * Fills the apps static variable with apps that require a menu + * Returns menus for apps that require one * - * @return void + * @return array + * @throws HTTPException\InternalServerErrorException */ - private static function populateAppMenu() + private function populateAppMenu(): array { - self::$app_menu = []; + $appMenu = []; //Don't populate apps_menu if apps are private - $privateapps = DI::config()->get('config', 'private_addons', false); - if (DI::userSession()->getLocalUserId() || !$privateapps) { - $arr = ['app_menu' => self::$app_menu]; + if ( + $this->session->getLocalUserId() + || !$this->config->get('config', 'private_addons', false) + ) { + $arr = ['app_menu' => $appMenu]; Hook::callAll('app_menu', $arr); - self::$app_menu = $arr['app_menu']; + $appMenu = $arr['app_menu']; } + + return $appMenu; } /** * 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) - * @throws \Friendica\Network\HTTPException\InternalServerErrorException + * @throws HTTPException\InternalServerErrorException + * @throws HTTPException\MethodNotAllowedException */ - private static function getInfo(App $a): array + private function getInfo(): array { - $ssl_state = (bool) DI::userSession()->getLocalUserId(); + $ssl_state = (bool) $this->session->getLocalUserId(); /* - * Our network is distributed, and as you visit friends some of the + * Our network is distributed, and as you visit friends some * 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 = !empty($a->getLoggedInUserNickname()) ? $a->getLoggedInUserNickname() . '@' : ''; + $myident = !empty($this->session->getLocalUserNickname()) ? $this->session->getLocalUserNickname() . '@' : ''; - $sitelocation = $myident . substr(DI::baseUrl()->get($ssl_state), strpos(DI::baseUrl()->get($ssl_state), '//') + 2); + $sitelocation = $myident . substr($this->baseUrl->get($ssl_state), strpos($this->baseUrl->get($ssl_state), '//') + 2); $nav = [ 'admin' => null, @@ -182,23 +218,23 @@ class Nav $userinfo = null; // nav links: array of array('href', 'text', 'extra css classes', 'title') - if (DI::userSession()->isAuthenticated()) { - $nav['logout'] = ['logout', DI::l10n()->t('Logout'), '', DI::l10n()->t('End this session')]; + if ($this->session->isAuthenticated()) { + $nav['logout'] = ['logout', $this->l10n->t('Logout'), '', $this->l10n->t('End this session')]; } else { - $nav['login'] = ['login', DI::l10n()->t('Login'), (DI::args()->getModuleName() == 'login' ? 'selected' : ''), DI::l10n()->t('Sign in')]; + $nav['login'] = ['login', $this->l10n->t('Login'), ($this->router->getModuleClass() == Login::class ? 'selected' : ''), $this->l10n->t('Sign in')]; } - if ($a->isLoggedIn()) { + if ($this->session->isAuthenticated()) { // user menu - $nav['usermenu'][] = ['profile/' . $a->getLoggedInUserNickname(), DI::l10n()->t('Status'), '', DI::l10n()->t('Your posts and conversations')]; - $nav['usermenu'][] = ['profile/' . $a->getLoggedInUserNickname() . '/profile', DI::l10n()->t('Profile'), '', DI::l10n()->t('Your profile page')]; - $nav['usermenu'][] = ['profile/' . $a->getLoggedInUserNickname() . '/photos', DI::l10n()->t('Photos'), '', DI::l10n()->t('Your photos')]; - $nav['usermenu'][] = ['profile/' . $a->getLoggedInUserNickname() . '/media', DI::l10n()->t('Media'), '', DI::l10n()->t('Your postings with media')]; - $nav['usermenu'][] = ['calendar/', DI::l10n()->t('Calendar'), '', DI::l10n()->t('Your calendar')]; - $nav['usermenu'][] = ['notes/', DI::l10n()->t('Personal notes'), '', DI::l10n()->t('Your personal notes')]; + $nav['usermenu'][] = ['profile/' . $this->session->getLocalUserNickname(), $this->l10n->t('Conversations'), '', $this->l10n->t('Conversations you started')]; + $nav['usermenu'][] = ['profile/' . $this->session->getLocalUserNickname() . '/profile', $this->l10n->t('Profile'), '', $this->l10n->t('Your profile page')]; + $nav['usermenu'][] = ['profile/' . $this->session->getLocalUserNickname() . '/photos', $this->l10n->t('Photos'), '', $this->l10n->t('Your photos')]; + $nav['usermenu'][] = ['profile/' . $this->session->getLocalUserNickname() . '/media', $this->l10n->t('Media'), '', $this->l10n->t('Your postings with media')]; + $nav['usermenu'][] = ['calendar/', $this->l10n->t('Calendar'), '', $this->l10n->t('Your calendar')]; + $nav['usermenu'][] = ['notes/', $this->l10n->t('Personal notes'), '', $this->l10n->t('Your personal notes')]; // user info - $contact = DBA::selectFirst('contact', ['id', 'url', 'avatar', 'micro', 'name', 'nick', 'baseurl', 'updated'], ['uid' => $a->getLoggedInUserId(), 'self' => true]); + $contact = $this->database->selectFirst('contact', ['id', 'url', 'avatar', 'micro', 'name', 'nick', 'baseurl', 'updated'], ['uid' => $this->session->getLocalUserId(), 'self' => true]); $userinfo = [ 'icon' => Contact::getMicro($contact), 'name' => $contact['name'], @@ -206,103 +242,103 @@ class Nav } // "Home" should also take you home from an authenticated remote profile connection - $homelink = Profile::getMyURL(); - if (! $homelink) { - $homelink = DI::session()->get('visitor_home', ''); + $homelink = $this->session->getMyUrl(); + if (!$homelink) { + $homelink = $this->session->get('visitor_home', ''); } - if (DI::args()->getModuleName() != 'home' && ! DI::userSession()->getLocalUserId()) { - $nav['home'] = [$homelink, DI::l10n()->t('Home'), '', DI::l10n()->t('Home Page')]; + if ($this->router->getModuleClass() != Home::class && !$this->session->getLocalUserId()) { + $nav['home'] = [$homelink, $this->l10n->t('Home'), '', $this->l10n->t('Home Page')]; } - if (intval(DI::config()->get('config', 'register_policy')) === \Friendica\Module\Register::OPEN && !DI::userSession()->isAuthenticated()) { - $nav['register'] = ['register', DI::l10n()->t('Register'), '', DI::l10n()->t('Create an account')]; + if (intval($this->config->get('config', 'register_policy')) === \Friendica\Module\Register::OPEN && !$this->session->isAuthenticated()) { + $nav['register'] = ['register', $this->l10n->t('Register'), '', $this->l10n->t('Create an account')]; } $help_url = 'help'; - if (!DI::config()->get('system', 'hide_help')) { - $nav['help'] = [$help_url, DI::l10n()->t('Help'), '', DI::l10n()->t('Help and documentation')]; + if (!$this->config->get('system', 'hide_help')) { + $nav['help'] = [$help_url, $this->l10n->t('Help'), '', $this->l10n->t('Help and documentation')]; } - if (count(self::getAppMenu()) > 0) { - $nav['apps'] = ['apps', DI::l10n()->t('Apps'), '', DI::l10n()->t('Addon applications, utilities, games')]; + if (count($this->getAppMenu()) > 0) { + $nav['apps'] = ['apps', $this->l10n->t('Apps'), '', $this->l10n->t('Addon applications, utilities, games')]; } - if (DI::userSession()->getLocalUserId() || !DI::config()->get('system', 'local_search')) { - $nav['search'] = ['search', DI::l10n()->t('Search'), '', DI::l10n()->t('Search site content')]; + if ($this->session->getLocalUserId() || !$this->config->get('system', 'local_search')) { + $nav['search'] = ['search', $this->l10n->t('Search'), '', $this->l10n->t('Search site content')]; $nav['searchoption'] = [ - DI::l10n()->t('Full Text'), - DI::l10n()->t('Tags'), - DI::l10n()->t('Contacts') + $this->l10n->t('Full Text'), + $this->l10n->t('Tags'), + $this->l10n->t('Contacts') ]; - if (DI::config()->get('system', 'poco_local_search')) { - $nav['searchoption'][] = DI::l10n()->t('Forums'); + if ($this->config->get('system', 'poco_local_search')) { + $nav['searchoption'][] = $this->l10n->t('Forums'); } } $gdirpath = 'directory'; - if (DI::config()->get('system', 'singleuser') && DI::config()->get('system', 'directory')) { - $gdirpath = Profile::zrl(DI::config()->get('system', 'directory'), true); + if ($this->config->get('system', 'singleuser') && $this->config->get('system', 'directory')) { + $gdirpath = Profile::zrl($this->config->get('system', 'directory'), true); } - if ((DI::userSession()->getLocalUserId() || DI::config()->get('system', 'community_page_style') != Community::DISABLED_VISITOR) && - !(DI::config()->get('system', 'community_page_style') == Community::DISABLED)) { - $nav['community'] = ['community', DI::l10n()->t('Community'), '', DI::l10n()->t('Conversations on this and other servers')]; + if (($this->session->getLocalUserId() || $this->config->get('system', 'community_page_style') != Community::DISABLED_VISITOR) && + !($this->config->get('system', 'community_page_style') == Community::DISABLED)) { + $nav['community'] = ['community', $this->l10n->t('Community'), '', $this->l10n->t('Conversations on this and other servers')]; } - if (DI::userSession()->getLocalUserId()) { - $nav['calendar'] = ['calendar', DI::l10n()->t('Calendar'), '', DI::l10n()->t('Calendar')]; + if ($this->session->getLocalUserId()) { + $nav['calendar'] = ['calendar', $this->l10n->t('Calendar'), '', $this->l10n->t('Calendar')]; } - $nav['directory'] = [$gdirpath, DI::l10n()->t('Directory'), '', DI::l10n()->t('People directory')]; + $nav['directory'] = [$gdirpath, $this->l10n->t('Directory'), '', $this->l10n->t('People directory')]; - $nav['about'] = ['friendica', DI::l10n()->t('Information'), '', DI::l10n()->t('Information about this friendica instance')]; + $nav['about'] = ['friendica', $this->l10n->t('Information'), '', $this->l10n->t('Information about this friendica instance')]; - if (DI::config()->get('system', 'tosdisplay')) { - $nav['tos'] = ['tos', DI::l10n()->t('Terms of Service'), '', DI::l10n()->t('Terms of Service of this Friendica instance')]; + if ($this->config->get('system', 'tosdisplay')) { + $nav['tos'] = ['tos', $this->l10n->t('Terms of Service'), '', $this->l10n->t('Terms of Service of this Friendica instance')]; } - // The following nav links are only show to logged in users - if (DI::userSession()->getLocalUserId() && !empty($a->getLoggedInUserNickname())) { - $nav['network'] = ['network', DI::l10n()->t('Network'), '', DI::l10n()->t('Conversations from your friends')]; + // The following nav links are only show to logged-in users + if ($this->session->getLocalUserNickname()) { + $nav['network'] = ['network', $this->l10n->t('Network'), '', $this->l10n->t('Conversations from your friends')]; - $nav['home'] = ['profile/' . $a->getLoggedInUserNickname(), DI::l10n()->t('Home'), '', DI::l10n()->t('Your posts and conversations')]; + $nav['home'] = ['profile/' . $this->session->getLocalUserNickname(), $this->l10n->t('Home'), '', $this->l10n->t('Your posts and conversations')]; // Don't show notifications for public communities - if (DI::session()->get('page_flags', '') != User::PAGE_FLAGS_COMMUNITY) { - $nav['introductions'] = ['notifications/intros', DI::l10n()->t('Introductions'), '', DI::l10n()->t('Friend Requests')]; - $nav['notifications'] = ['notifications', DI::l10n()->t('Notifications'), '', DI::l10n()->t('Notifications')]; - $nav['notifications']['all'] = ['notifications/system', DI::l10n()->t('See all notifications'), '', '']; - $nav['notifications']['mark'] = ['', DI::l10n()->t('Mark as seen'), '', DI::l10n()->t('Mark all system notifications as seen')]; + if ($this->session->get('page_flags', '') != User::PAGE_FLAGS_COMMUNITY) { + $nav['introductions'] = ['notifications/intros', $this->l10n->t('Introductions'), '', $this->l10n->t('Friend Requests')]; + $nav['notifications'] = ['notifications', $this->l10n->t('Notifications'), '', $this->l10n->t('Notifications')]; + $nav['notifications']['all'] = ['notifications/system', $this->l10n->t('See all notifications'), '', '']; + $nav['notifications']['mark'] = ['', $this->l10n->t('Mark as seen'), '', $this->l10n->t('Mark all system notifications as seen')]; } - $nav['messages'] = ['message', DI::l10n()->t('Messages'), '', DI::l10n()->t('Private mail')]; - $nav['messages']['inbox'] = ['message', DI::l10n()->t('Inbox'), '', DI::l10n()->t('Inbox')]; - $nav['messages']['outbox'] = ['message/sent', DI::l10n()->t('Outbox'), '', DI::l10n()->t('Outbox')]; - $nav['messages']['new'] = ['message/new', DI::l10n()->t('New Message'), '', DI::l10n()->t('New Message')]; + $nav['messages'] = ['message', $this->l10n->t('Messages'), '', $this->l10n->t('Private mail')]; + $nav['messages']['inbox'] = ['message', $this->l10n->t('Inbox'), '', $this->l10n->t('Inbox')]; + $nav['messages']['outbox'] = ['message/sent', $this->l10n->t('Outbox'), '', $this->l10n->t('Outbox')]; + $nav['messages']['new'] = ['message/new', $this->l10n->t('New Message'), '', $this->l10n->t('New Message')]; - if (User::hasIdentities(DI::userSession()->getSubManagedUserId() ?: DI::userSession()->getLocalUserId())) { - $nav['delegation'] = ['delegation', DI::l10n()->t('Accounts'), '', DI::l10n()->t('Manage other pages')]; + if (User::hasIdentities($this->session->getSubManagedUserId() ?: $this->session->getLocalUserId())) { + $nav['delegation'] = ['delegation', $this->l10n->t('Accounts'), '', $this->l10n->t('Manage other pages')]; } - $nav['settings'] = ['settings', DI::l10n()->t('Settings'), '', DI::l10n()->t('Account settings')]; + $nav['settings'] = ['settings', $this->l10n->t('Settings'), '', $this->l10n->t('Account settings')]; - $nav['contacts'] = ['contact', DI::l10n()->t('Contacts'), '', DI::l10n()->t('Manage/edit friends and contacts')]; + $nav['contacts'] = ['contact', $this->l10n->t('Contacts'), '', $this->l10n->t('Manage/edit friends and contacts')]; } // Show the link to the admin configuration page if user is admin - if ($a->isSiteAdmin()) { - $nav['admin'] = ['admin/', DI::l10n()->t('Admin'), '', DI::l10n()->t('Site setup and configuration')]; - $nav['moderation'] = ['moderation/', DI::l10n()->t('Moderation'), '', DI::l10n()->t('Content and user moderation')]; + if ($this->session->isSiteAdmin()) { + $nav['admin'] = ['admin/', $this->l10n->t('Admin'), '', $this->l10n->t('Site setup and configuration')]; + $nav['moderation'] = ['moderation/', $this->l10n->t('Moderation'), '', $this->l10n->t('Content and user moderation')]; } - $nav['navigation'] = ['navigation/', DI::l10n()->t('Navigation'), '', DI::l10n()->t('Site map')]; + $nav['navigation'] = ['navigation/', $this->l10n->t('Navigation'), '', $this->l10n->t('Site map')]; // Provide a banner/logo/whatever - $banner = DI::config()->get('system', 'banner'); + $banner = $this->config->get('system', 'banner'); if (is_null($banner)) { $banner = 'Friendica'; } diff --git a/src/Module/Apps.php b/src/Module/Apps.php index 9f738b2ce..07863e44c 100644 --- a/src/Module/Apps.php +++ b/src/Module/Apps.php @@ -22,13 +22,13 @@ namespace Friendica\Module; use Friendica\App; -use Friendica\App\BaseURL; use Friendica\BaseModule; use Friendica\Content\Nav; use Friendica\Core\Config\Capability\IManageConfigValues; use Friendica\Core\L10n; use Friendica\Core\Renderer; -use Friendica\DI; +use Friendica\Core\Session\Capability\IHandleUserSessions; +use Friendica\Navigation\SystemMessages; use Friendica\Util\Profiler; use Psr\Log\LoggerInterface; @@ -37,22 +37,29 @@ use Psr\Log\LoggerInterface; */ class Apps extends BaseModule { - public function __construct(L10n $l10n, App\BaseURL $baseUrl, App\Arguments $args, LoggerInterface $logger, Profiler $profiler, Response $response, IManageConfigValues $config, array $server, array $parameters = []) + /** @var Nav */ + protected $nav; + /** @var SystemMessages */ + protected $systemMessages; + + public function __construct(SystemMessages $systemMessages, Nav $nav, IHandleUserSessions $session, L10n $l10n, App\BaseURL $baseUrl, App\Arguments $args, LoggerInterface $logger, Profiler $profiler, Response $response, IManageConfigValues $config, array $server, array $parameters = []) { parent::__construct($l10n, $baseUrl, $args, $logger, $profiler, $response, $server, $parameters); + $this->nav = $nav; + $this->systemMessages = $systemMessages; + $privateaddons = $config->get('config', 'private_addons'); - if ($privateaddons === "1" && !DI::userSession()->getLocalUserId()) { + if ($privateaddons === "1" && !$session->getLocalUserId()) { $baseUrl->redirect(); } } protected function content(array $request = []): string { - $apps = Nav::getAppMenu(); - + $apps = $this->nav->getAppMenu(); if (count($apps) == 0) { - DI::sysmsg()->addNotice($this->t('No installed applications.')); + $this->systemMessages->addNotice($this->t('No installed applications.')); } $tpl = Renderer::getMarkupTemplate('apps.tpl'); diff --git a/src/Module/BaseProfile.php b/src/Module/BaseProfile.php index 28b824adb..b7584d9db 100644 --- a/src/Module/BaseProfile.php +++ b/src/Module/BaseProfile.php @@ -54,10 +54,10 @@ class BaseProfile extends BaseModule 'accesskey' => 'r', ], [ - 'label' => DI::l10n()->t('Status'), - 'url' => $baseProfileUrl . '/status', + 'label' => DI::l10n()->t('Conversations'), + 'url' => $baseProfileUrl . '/conversations', 'sel' => $current == 'status' ? 'active' : '', - 'title' => DI::l10n()->t('Status Messages and Posts'), + 'title' => DI::l10n()->t('Conversations started'), 'id' => 'status-tab', 'accesskey' => 'm', ], diff --git a/src/Module/Contact.php b/src/Module/Contact.php index 21b1793af..bc2f2f402 100644 --- a/src/Module/Contact.php +++ b/src/Module/Contact.php @@ -451,7 +451,7 @@ class Contact extends BaseModule /** * List of pages for the Contact TabBar * - * Available Pages are 'Status', 'Profile', 'Contacts' and 'Common Friends' + * Available Pages are 'Conversations', 'Profile', 'Contacts' and 'Common Friends' * * @param array $contact The contact array * @param int $active_tab 1 if tab should be marked as active @@ -473,7 +473,15 @@ class Contact extends BaseModule // tabs $tabs = [ [ - 'label' => DI::l10n()->t('Status'), + 'label' => DI::l10n()->t('Profile'), + 'url' => 'contact/' . $cid, + 'sel' => (($active_tab == self::TAB_PROFILE) ? 'active' : ''), + 'title' => DI::l10n()->t('Profile Details'), + 'id' => 'profile-tab', + 'accesskey' => 'o', + ], + [ + 'label' => DI::l10n()->t('Conversations'), 'url' => 'contact/' . $pcid . '/conversations', 'sel' => (($active_tab == self::TAB_CONVERSATIONS) ? 'active' : ''), 'title' => DI::l10n()->t('Conversations started by this contact'), @@ -484,7 +492,7 @@ class Contact extends BaseModule 'label' => DI::l10n()->t('Posts and Comments'), 'url' => 'contact/' . $pcid . '/posts', 'sel' => (($active_tab == self::TAB_POSTS) ? 'active' : ''), - 'title' => DI::l10n()->t('Status Messages and Posts'), + 'title' => DI::l10n()->t('Individual Posts and Replies'), 'id' => 'posts-tab', 'accesskey' => 'p', ], @@ -496,14 +504,6 @@ class Contact extends BaseModule 'id' => 'media-tab', 'accesskey' => 'd', ], - [ - 'label' => DI::l10n()->t('Profile'), - 'url' => 'contact/' . $cid, - 'sel' => (($active_tab == self::TAB_PROFILE) ? 'active' : ''), - 'title' => DI::l10n()->t('Profile Details'), - 'id' => 'profile-tab', - 'accesskey' => 'o', - ], ['label' => DI::l10n()->t('Contacts'), 'url' => 'contact/' . $pcid . '/contacts', 'sel' => (($active_tab == self::TAB_CONTACTS) ? 'active' : ''), diff --git a/src/Module/Contact/Follow.php b/src/Module/Contact/Follow.php index 5e648deac..57e9ff634 100644 --- a/src/Module/Contact/Follow.php +++ b/src/Module/Contact/Follow.php @@ -188,7 +188,7 @@ class Follow extends BaseModule $this->page['aside'] = VCard::getHTML($contact); $output .= Renderer::replaceMacros(Renderer::getMarkupTemplate('section_title.tpl'), - ['$title' => $this->t('Status Messages and Posts')] + ['$title' => $this->t('Posts and Replies')] ); // Show last public posts diff --git a/src/Module/Contact/Unfollow.php b/src/Module/Contact/Unfollow.php index 36db68818..9bbc34eee 100644 --- a/src/Module/Contact/Unfollow.php +++ b/src/Module/Contact/Unfollow.php @@ -135,7 +135,7 @@ class Unfollow extends \Friendica\BaseModule $this->page['aside'] = Widget\VCard::getHTML(Contact::getByURL($contact['url'], false)); - $o .= Renderer::replaceMacros(Renderer::getMarkupTemplate('section_title.tpl'), ['$title' => $this->t('Status Messages and Posts')]); + $o .= Renderer::replaceMacros(Renderer::getMarkupTemplate('section_title.tpl'), ['$title' => $this->t('Posts and Replies')]); // Show last public posts $o .= Contact::getPostsFromUrl($contact['url']); diff --git a/src/Module/Post/Tag/Add.php b/src/Module/Post/Tag/Add.php index aea18aa64..8c4858794 100644 --- a/src/Module/Post/Tag/Add.php +++ b/src/Module/Post/Tag/Add.php @@ -120,7 +120,7 @@ EOT; $tagger_link = '[url=' . $contact['url'] . ']' . $contact['name'] . '[/url]'; $aauthor_link = '[url=' . $item['author-link'] . ']' . $item['author-name'] . '[/url]'; - $post_link = '[url=' . $item['plink'] . ']' . ($item['resource-id'] ? $this->t('photo') : $this->t('status')) . '[/url]'; + $post_link = '[url=' . $item['plink'] . ']' . ($item['resource-id'] ? $this->t('photo') : $this->t('post')) . '[/url]'; $term_link = '#[url=' . $tagid . ']' . $term . '[/url]'; $post = [ diff --git a/src/Module/Profile/Status.php b/src/Module/Profile/Conversations.php similarity index 97% rename from src/Module/Profile/Status.php rename to src/Module/Profile/Conversations.php index 38c9a6902..322cdc80e 100644 --- a/src/Module/Profile/Status.php +++ b/src/Module/Profile/Conversations.php @@ -51,7 +51,7 @@ use Friendica\Util\Profiler; use Friendica\Util\Strings; use Psr\Log\LoggerInterface; -class Status extends BaseProfile +class Conversations extends BaseProfile { /** @var App */ private $app; @@ -147,8 +147,8 @@ class Status extends BaseProfile $commpage = $profile['page-flags'] == User::PAGE_FLAGS_COMMUNITY; $commvisitor = $commpage && $remote_contact; - $this->page['aside'] .= Widget::postedByYear($this->baseUrl . '/profile/' . $profile['nickname'] . '/status', $profile['profile_uid'] ?? 0, true); - $this->page['aside'] .= Widget::categories($profile['uid'], $this->baseUrl . '/profile/' . $profile['nickname'] . '/status', $category); + $this->page['aside'] .= Widget::postedByYear($this->baseUrl . '/profile/' . $profile['nickname'] . '/conversations', $profile['profile_uid'] ?? 0, true); + $this->page['aside'] .= Widget::categories($profile['uid'], $this->baseUrl . '/profile/' . $profile['nickname'] . '/conversations', $category); $this->page['aside'] .= Widget::tagCloud($profile['uid']); if (Security::canWriteToUserWall($profile['uid'])) { diff --git a/src/Module/Profile/Index.php b/src/Module/Profile/Index.php index 149d16512..7a009739e 100644 --- a/src/Module/Profile/Index.php +++ b/src/Module/Profile/Index.php @@ -42,7 +42,7 @@ use Psr\Log\LoggerInterface; * ActivityPub endpoint, but it should show statuses to web users. * * Both these view have dedicated sub-paths, - * respectively https://domain.tld/profile/nickname/profile and https://domain.tld/profile/nickname/status + * respectively https://domain.tld/profile/nickname/profile and https://domain.tld/profile/nickname/conversations */ class Index extends BaseModule { @@ -90,6 +90,6 @@ class Index extends BaseModule protected function content(array $request = []): string { - return (new Status($this->mode, $this->pConfig, $this->conversation, $this->session, $this->config, $this->dateTimeFormat, $this->page, $this->app, $this->l10n, $this->baseUrl, $this->args, $this->logger, $this->profiler, $this->response, $this->server, $this->parameters))->content(); + return (new Conversations($this->mode, $this->pConfig, $this->conversation, $this->session, $this->config, $this->dateTimeFormat, $this->page, $this->app, $this->l10n, $this->baseUrl, $this->args, $this->logger, $this->profiler, $this->response, $this->server, $this->parameters))->content(); } } diff --git a/static/routes.config.php b/static/routes.config.php index bc10fdac0..2219ea454 100644 --- a/static/routes.config.php +++ b/static/routes.config.php @@ -31,17 +31,17 @@ use Friendica\App\Router as R; use Friendica\Module; $profileRoutes = [ - '' => [Module\Profile\Index::class, [R::GET]], - '/contacts/common' => [Module\Profile\Common::class, [R::GET]], - '/contacts[/{type}]' => [Module\Profile\Contacts::class, [R::GET]], - '/media' => [Module\Profile\Media::class, [R::GET]], - '/photos' => [Module\Profile\Photos::class, [R::GET, R::POST]], - '/profile' => [Module\Profile\Profile::class, [R::GET]], - '/remote_follow' => [Module\Profile\RemoteFollow::class, [R::GET, R::POST]], - '/restricted' => [Module\Profile\Restricted::class, [R::GET ]], - '/schedule' => [Module\Profile\Schedule::class, [R::GET, R::POST]], - '/status[/{category}[/{date1}[/{date2}]]]' => [Module\Profile\Status::class, [R::GET]], - '/unkmail' => [Module\Profile\UnkMail::class, [R::GET, R::POST]], + '' => [Module\Profile\Index::class, [R::GET]], + '/contacts/common' => [Module\Profile\Common::class, [R::GET]], + '/contacts[/{type}]' => [Module\Profile\Contacts::class, [R::GET]], + '/media' => [Module\Profile\Media::class, [R::GET]], + '/photos' => [Module\Profile\Photos::class, [R::GET, R::POST]], + '/profile' => [Module\Profile\Profile::class, [R::GET]], + '/remote_follow' => [Module\Profile\RemoteFollow::class, [R::GET, R::POST]], + '/restricted' => [Module\Profile\Restricted::class, [R::GET ]], + '/schedule' => [Module\Profile\Schedule::class, [R::GET, R::POST]], + '/conversations[/{category}[/{date1}[/{date2}]]]' => [Module\Profile\Conversations::class, [R::GET]], + '/unkmail' => [Module\Profile\UnkMail::class, [R::GET, R::POST]], ]; $apiRoutes = [ diff --git a/view/lang/C/messages.po b/view/lang/C/messages.po index 133f757ee..e0591e499 100644 --- a/view/lang/C/messages.po +++ b/view/lang/C/messages.po @@ -8,7 +8,7 @@ msgid "" msgstr "" "Project-Id-Version: 2023.03-dev\n" "Report-Msgid-Bugs-To: \n" -"POT-Creation-Date: 2023-01-09 17:29+0000\n" +"POT-Creation-Date: 2023-01-10 19:06-0500\n" "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n" "Last-Translator: FULL NAME