From 58f35d422d6498854533ba5ef0d81f031c4a4b15 Mon Sep 17 00:00:00 2001
From: Marcel Werk <burntime@woltlab.com>
Date: Wed, 13 Jul 2016 16:24:40 +0200
Subject: [PATCH] Improved error pages / messages

---
 .../templates/permissionDenied.tpl            |  7 ++++--
 com.woltlab.wcf/templates/userException.tpl   | 23 +++++++++++++------
 .../exception/IllegalLinkException.class.php  |  5 ++++
 wcfsetup/install/lang/de.xml                  |  5 ++--
 wcfsetup/install/lang/en.xml                  |  6 +++--
 5 files changed, 33 insertions(+), 13 deletions(-)

diff --git a/com.woltlab.wcf/templates/permissionDenied.tpl b/com.woltlab.wcf/templates/permissionDenied.tpl
index 3146e91766..827bd56f28 100644
--- a/com.woltlab.wcf/templates/permissionDenied.tpl
+++ b/com.woltlab.wcf/templates/permissionDenied.tpl
@@ -1,8 +1,11 @@
 {capture assign='pageTitle'}{lang}wcf.global.error.permissionDenied.title{/lang}{/capture}
+{capture assign='contentTitle'}{lang}wcf.global.error.permissionDenied.title{/lang}{/capture}
 
-{include file='header' __disableAds=true __disableContentHeader=true}
+{include file='header' __disableAds=true}
 
-<p class="error">{lang}wcf.global.error.permissionDenied{/lang}</p>
+<div class="section">
+	<p>{lang}wcf.global.error.permissionDenied{/lang}</p>
+</div>
 
 {event name='content'}
 
diff --git a/com.woltlab.wcf/templates/userException.tpl b/com.woltlab.wcf/templates/userException.tpl
index 66b2f430e5..9cbf3959f2 100644
--- a/com.woltlab.wcf/templates/userException.tpl
+++ b/com.woltlab.wcf/templates/userException.tpl
@@ -1,14 +1,23 @@
-{capture assign='pageTitle'}{lang}wcf.global.error.title{/lang}{/capture}
+{if !$title|empty}
+	{capture assign='pageTitle'}{$title}{/capture}
+	{capture assign='contentTitle'}{$title}{/capture}
+{else}
+	{capture assign='pageTitle'}{lang}wcf.global.error.title{/lang}{/capture}
+	{capture assign='contentTitle'}{lang}wcf.global.error.title{/lang}{/capture}
+{/if}
+
+{include file='header' __disableAds=true}
+
+<div class="section">
+	<p id="errorMessage">
+		{@$message}
+	</p>
+</div>
 
-{include file='header' __disableAds=true __disableContentHeader=true}
-	
-<p id="errorMessage" class="error">
-	{@$message}
-</p>
 <script data-relocate="true">
 	//<![CDATA[
 	if (document.referrer) {
-		$('#errorMessage').append('<br><a href="' + document.referrer + '">{lang}wcf.global.error.backward{/lang}</a>'); 
+		$('#errorMessage').append('<br><br><a href="' + document.referrer + '">{lang}wcf.global.error.backward{/lang}</a>'); 
 	}
 	//]]>
 </script>
diff --git a/wcfsetup/install/files/lib/system/exception/IllegalLinkException.class.php b/wcfsetup/install/files/lib/system/exception/IllegalLinkException.class.php
index dfc7bafe76..3824cb45b6 100644
--- a/wcfsetup/install/files/lib/system/exception/IllegalLinkException.class.php
+++ b/wcfsetup/install/files/lib/system/exception/IllegalLinkException.class.php
@@ -23,6 +23,11 @@ class IllegalLinkException extends NamedUserException {
 	 */
 	public function show() {
 		@header('HTTP/1.0 404 Not Found');
+		
+		WCF::getTPL()->assign([
+			'title' => WCF::getLanguage()->get('wcf.global.error.illegalLink.title')
+		]);
+		
 		parent::show();
 	}
 }
diff --git a/wcfsetup/install/lang/de.xml b/wcfsetup/install/lang/de.xml
index 059f9246ef..e33cf030d3 100644
--- a/wcfsetup/install/lang/de.xml
+++ b/wcfsetup/install/lang/de.xml
@@ -2354,7 +2354,7 @@ Fehler sind beispielsweise:
 		<item name="wcf.global.decimalPoint"><![CDATA[,]]></item>
 		<item name="wcf.global.description"><![CDATA[Beschreibung]]></item>
 		<item name="wcf.global.error.permissionDenied"><![CDATA[Der Zutritt zu dieser Seite ist Ihnen leider verwehrt. Sie besitzen nicht die notwendigen Zugriffsrechte, um diese Seite aufrufen zu können.]]></item>
-		<item name="wcf.global.error.permissionDenied.title"><![CDATA[Der Zutritt zu dieser Seite ist Ihnen leider verwehrt.]]></item>
+		<item name="wcf.global.error.permissionDenied.title"><![CDATA[Zugriff verweigert]]></item>
 		<item name="wcf.global.error.timeout"><![CDATA[Keine Antwort vom Server erhalten, Anfrage wurde abgebrochen.]]></item>
 		<item name="wcf.global.error.title"><![CDATA[Fehlermeldung]]></item>
 		<item name="wcf.global.exception.explanation"><![CDATA[<p class="exceptionSubtitle">Was ist passiert?</p>
@@ -2400,7 +2400,8 @@ Fehler sind beispielsweise:
 		<item name="wcf.global.multiSelect"><![CDATA[Mehrfache Markierungen sind durch zusätzliches Drücken der Taste „Strg/Ctrl“ (Windows) oder „Befehl/Command“ (Mac OS) möglich.]]></item>
 		<item name="wcf.global.page.jumpTo"><![CDATA[Gehe zu Seite]]></item>
 		<item name="wcf.global.page.jumpTo.description"><![CDATA[Geben Sie einen Wert zwischen „1“ und „#pages#“ ein.]]></item>
-		<item name="wcf.global.error.illegalLink"><![CDATA[Sie haben einen ungültigen oder nicht mehr gültigen Link aufgerufen.]]></item>
+		<item name="wcf.global.error.illegalLink"><![CDATA[Die von Ihnen angeforderte Seite wurde nicht gefunden. Bitte überprüfen Sie die Adresse oder gehen Sie zurück auf die Startseite.]]></item>
+		<item name="wcf.global.error.illegalLink.title"><![CDATA[Seite nicht gefunden]]></item>
 		<item name="wcf.global.error.backward"><![CDATA[Zurück zur vorherigen Seite]]></item>
 		<item name="wcf.global.noDeclaration"><![CDATA[Keine Angabe]]></item>
 		<item name="wcf.global.defaultValue"><![CDATA[Standard]]></item>
diff --git a/wcfsetup/install/lang/en.xml b/wcfsetup/install/lang/en.xml
index 3eabf5caac..8c643df697 100644
--- a/wcfsetup/install/lang/en.xml
+++ b/wcfsetup/install/lang/en.xml
@@ -2332,7 +2332,8 @@ Errors are:
 		<item name="wcf.global.date"><![CDATA[Date]]></item>
 		<item name="wcf.global.decimalPoint"><![CDATA[.]]></item>
 		<item name="wcf.global.description"><![CDATA[Description]]></item>
-		<item name="wcf.global.error.permissionDenied.title"><![CDATA[Access denied.]]></item>
+		<item name="wcf.global.error.permissionDenied"><![CDATA[Access denied. You’re not authorized to view this page.]]></item>
+		<item name="wcf.global.error.permissionDenied.title"><![CDATA[Access Denied]]></item>
 		<item name="wcf.global.error.timeout"><![CDATA[Did not receive a response from server, request aborted.]]></item>
 		<item name="wcf.global.error.title"><![CDATA[Error Message]]></item>
 		<item name="wcf.global.exception.explanation"><![CDATA[<p class="exceptionSubtitle">What happened?</p>
@@ -2378,7 +2379,8 @@ Errors are:
 		<item name="wcf.global.multiSelect"><![CDATA[Multiple items can be selected by holding the “Ctrl/Shift” key (Windows) or “Command” (Mac OS) key and clicking on the desired items.]]></item>
 		<item name="wcf.global.page.jumpTo"><![CDATA[Go to Page]]></item>
 		<item name="wcf.global.page.jumpTo.description"><![CDATA[Enter a value between “1” and “#pages#”.]]></item>
-		<item name="wcf.global.error.illegalLink"><![CDATA[The link you’re trying to reach is no longer available or invalid.]]></item>
+		<item name="wcf.global.error.illegalLink"><![CDATA[Sorry, but the page you are looking for has not been found. Try checking the URL for errors, then hit the refresh button on your browser.]]></item>
+		<item name="wcf.global.error.illegalLink.title"><![CDATA[Page Not Found]]></item>
 		<item name="wcf.global.error.backward"><![CDATA[Back to previous page.]]></item>
 		<item name="wcf.global.noDeclaration"><![CDATA[Unspecified]]></item>
 		<item name="wcf.global.defaultValue"><![CDATA[Default Value]]></item>
-- 
2.20.1