feature(core): adds handling of 400 and 403 error codes

Improves handling of 400 and 403 HTTP error codes
Elgg · Apr 11, 2015 · 243ca40 · 243ca40
1 parent 0769708
commit 243ca40
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 0 deletions.
diff --git a/engine/lib/pagehandler.php b/engine/lib/pagehandler.php
@@ -293,6 +293,8 @@ function elgg_error_page_handler($hook, $type, $result, $params) {
 function _elgg_page_handler_init() {
 	elgg_register_page_handler('', 'elgg_front_page_handler');
 	// Registered at 600 so that plugins can register at the default 500 and get to run first
+	elgg_register_plugin_hook_handler('forward', '400', 'elgg_error_page_handler', 600);
+	elgg_register_plugin_hook_handler('forward', '403', 'elgg_error_page_handler', 600);
 	elgg_register_plugin_hook_handler('forward', '404', 'elgg_error_page_handler', 600);
 }
 

diff --git a/languages/en.php b/languages/en.php
@@ -112,6 +112,10 @@
 
 	'error:default:title' => 'Oops...',
 	'error:default:content' => 'Oops... something went wrong.',
+	'error:400:title' => 'Bad request',
+	'error:400:content' => 'Sorry. The request is invalid or incomplete.',
+	'error:403:title' => 'Forbidden',
+	'error:403:content' => 'Sorry. You are not allowed to access the requested page.',
 	'error:404:title' => 'Page not found',
 	'error:404:content' => 'Sorry. We could not find the page that you requested.',
 

diff --git a/views/default/errors/400.php b/views/default/errors/400.php
@@ -0,0 +1,6 @@
+<?php
+/**
+ * Bad request error
+ */
+
+echo elgg_echo('error:400:content');
diff --git a/views/default/errors/403.php b/views/default/errors/403.php
@@ -0,0 +1,6 @@
+<?php
+/**
+ * Forbidden error
+ */
+
+echo elgg_echo('error:403:content');