Removed global $page

This commit is contained in:
Igor Scheller 2019-11-26 20:12:43 +01:00
parent a2f4426058
commit 0e386d2c7f
4 changed files with 14 additions and 12 deletions

View File

@ -18,9 +18,7 @@ function admin_questions_title()
*/ */
function admin_new_questions() function admin_new_questions()
{ {
global $page; if (current_page() != 'admin_questions') {
if ($page != 'admin_questions') {
if (auth()->can('admin_questions')) { if (auth()->can('admin_questions')) {
$new_messages = count(DB::select('SELECT `QID` FROM `Questions` WHERE `AID` IS NULL')); $new_messages = count(DB::select('SELECT `QID` FROM `Questions` WHERE `AID` IS NULL'));

View File

@ -87,8 +87,6 @@ function news_text(News $news): string
*/ */
function display_news(News $news): string function display_news(News $news): string
{ {
global $page;
$html = ''; $html = '';
$html .= '<div class="panel' . ($news->is_meeting ? ' panel-info' : ' panel-default') . '">'; $html .= '<div class="panel' . ($news->is_meeting ? ' panel-info' : ' panel-default') . '">';
$html .= '<div class="panel-heading">'; $html .= '<div class="panel-heading">';
@ -109,7 +107,7 @@ function display_news(News $news): string
$html .= '<span class="glyphicon glyphicon-time"></span> ' . $news->created_at->format('Y-m-d H:i') . '&emsp;'; $html .= '<span class="glyphicon glyphicon-time"></span> ' . $news->created_at->format('Y-m-d H:i') . '&emsp;';
$html .= User_Nick_render($news->user); $html .= User_Nick_render($news->user);
if ($page != 'news_comments') { if (current_page() != 'news_comments') {
$html .= '&emsp;<a href="' . page_link_to('news_comments', ['nid' => $news->id]) . '">' $html .= '&emsp;<a href="' . page_link_to('news_comments', ['nid' => $news->id]) . '">'
. '<span class="glyphicon glyphicon-comment"></span> ' . '<span class="glyphicon glyphicon-comment"></span> '
. __('Comments') . ' &raquo;</a> ' . __('Comments') . ' &raquo;</a> '

View File

@ -40,13 +40,22 @@ function header_render_hints()
return $hints_renderer->render(); return $hints_renderer->render();
} }
/**
* Returns the path of the current path with underscores instead of hyphens
*
* @return string
*/
function current_page()
{
return request()->query->get('p') ?: str_replace('-', '_', request()->path());
}
/** /**
* @return array * @return array
*/ */
function make_user_submenu() function make_user_submenu()
{ {
global $page; $page = current_page();
$user_submenu = make_language_select(); $user_submenu = make_language_select();
if (auth()->can('user_settings') || auth()->can('logout')) { if (auth()->can('user_settings') || auth()->can('logout')) {
@ -79,8 +88,7 @@ function make_user_submenu()
*/ */
function make_navigation() function make_navigation()
{ {
global $page; $page = current_page();
$menu = []; $menu = [];
$pages = [ $pages = [
'news' => __('News'), 'news' => __('News'),

View File

@ -58,8 +58,6 @@ class LegacyMiddleware implements MiddlewareInterface
ServerRequestInterface $request, ServerRequestInterface $request,
RequestHandlerInterface $handler RequestHandlerInterface $handler
): ResponseInterface { ): ResponseInterface {
global $page;
/** @var Request $appRequest */ /** @var Request $appRequest */
$appRequest = $this->container->get('request'); $appRequest = $this->container->get('request');
$page = $appRequest->query->get('p'); $page = $appRequest->query->get('p');