mirror/php-phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-03-22 00:21:02 +01:00
Code Issues Releases Wiki Activity

Merge pull request #4943 from Elsensee/ticket/14933

Browse Source 
[ticket/14933] Make "route not found" more user-friendly
This commit is contained in:
Máté Bartus 2017-09-10 15:17:54 +02:00
commit 16c4564272
3 changed files with 18 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
phpBB/config/default/container/services_event.yml
View File

@ -9,6 +9,7 @@ services:
arguments:
- '@template'
- '@language'
- '%debug.exceptions%'
tags:
- { name: kernel.event_subscriber }

1
phpBB/language/en/common.php
View File

@ -556,6 +556,7 @@ $lang = array_merge($lang, array(
),
'OPTIONS' => 'Options',
'PAGE_NOT_FOUND' => 'The requested page could not be found.',
'PAGE_OF' => 'Page <strong>%1$d</strong> of <strong>%2$d</strong>',
'PAGE_TITLE_NUMBER' => 'Page %s',
'PASSWORD' => 'Password',

18
phpBB/phpbb/event/kernel_exception_subscriber.php
View File

@ -16,12 +16,20 @@ namespace phpbb\event;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpKernel\Exception\HttpExceptionInterface;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
use Symfony\Component\HttpKernel\KernelEvents;
use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
use Symfony\Component\HttpFoundation\Response;
class kernel_exception_subscriber implements EventSubscriberInterface
{
/**
* Set to true to show full exception messages
*
* @var bool
*/
protected $debug;
/**
* Template object
*
@ -44,9 +52,11 @@ class kernel_exception_subscriber implements EventSubscriberInterface
*
* @param \phpbb\template\template $template Template object
* @param \phpbb\language\language $language Language object
* @param bool $debug Set to true to show full exception messages
*/
public function __construct(\phpbb\template\template $template, \phpbb\language\language $language)
public function __construct(\phpbb\template\template $template, \phpbb\language\language $language, $debug = false)
{
$this->debug = $debug || defined('DEBUG');
$this->template = $template;
$this->language = $language;
$this->type_caster = new \phpbb\request\type_cast_helper();
@ -69,6 +79,10 @@ class kernel_exception_subscriber implements EventSubscriberInterface
{
$message = $this->language->lang_array($message, $exception->get_parameters());
}
else if (!$this->debug && $exception instanceof NotFoundHttpException)
{
$message = $this->language->lang('PAGE_NOT_FOUND');
}
// Show <strong> text in bold
$message = preg_replace('#&lt;(/?strong)&gt;#i', '<$1>', $message);
@ -99,7 +113,7 @@ class kernel_exception_subscriber implements EventSubscriberInterface
$data['message'] = $message;
}
if (defined('DEBUG'))
if ($this->debug)
{
$data['trace'] = $exception->getTrace();
}