page: move custom error handling to include

[minimedit.git] / page.php

diff --git a/page.php b/page.php
index 214f9d284804e89a7d4cb0949e177b22c9db775b..71bc74afaa0a964fe91f9c0124b4ad1b9b65b5e1 100644 (file)
--- a/page.php
+++ b/page.php
@@ -2,52 +2,10 @@
 error_reporting(E_ALL);
 ini_set('display_errors', TRUE);
 
-function abort($body, $status = NULL) {
-       if ($status) header("HTTP/1.1 $status");
-       print "$body\n";
-       exit;
-}
-
-# custom error handling
-
 define('DOCROOT', getcwd());
 set_include_path(implode(PATH_SEPARATOR, [ DOCROOT, __DIR__ ]));
 
-function fail($error)
-{
-       global $User, $Page;
-       http_response_code(500);
-       if (!isset($Page)) {
-               require_once('article.inc.php');
-               $Page = new ArchiveArticle(NULL);
-               $Page->title = 'Fout';
-       }
-       include_once 'page.inc.php';
-       ob_start();
-       require '500.inc.html';
-       print $Page->render(['debug' => htmlspecialchars($error)]);
-}
-
-set_exception_handler('fail');
-
-define('E_FATAL', E_ERROR | E_CORE_ERROR | E_COMPILE_ERROR | E_USER_ERROR);
-
-set_error_handler(function ($level, $error, $file, $line) {
-       if ($level & E_FATAL) {
-               fail($error);
-               return;
-       }
-       return FALSE;
-});
-
-register_shutdown_function(function () {
-       # display failure page for fatal exceptions
-       $error = error_get_last();
-       if (!($error['type'] & E_FATAL)) return;
-       fail("Fatal: $error[message] in $error[file]:$error[line]");
-});
-
-error_reporting(error_reporting() & ~E_FATAL);
+include_once 'error.inc.php';
 
 # user login and control
 
@@ -94,8 +52,7 @@ header(sprintf('Content-Security-Policy: %s', implode('; ', [
        "frame-ancestors 'none'", # prevent malicious embedding
 ])));
 
-ob_start(); # page body
-$Place = [
+$Page->place += [
        'user'  => $User->login ?: '',
        'url'   => htmlspecialchars($_SERVER['REQUEST_URI']),
 ];
@@ -109,6 +66,7 @@ if (!isset($Page->raw) and $User->admin("edit {$Page->link}")) {
        $Page->raw($template);
        $Page->meta['article:published_time'] = date('Y-m-d h:i:s O');
        $Page->meta['article:author'] = '/' . $User->dir;
+       $Page->body = NULL;
 }
 
 if (isset($Page->raw)) {
@@ -126,15 +84,18 @@ if (isset($Page->raw)) {
 
 # output dynamic and/or static html
 
-if (!$Page->handler or require("./{$Page->handler}/index.php")) {
-       # static contents
-       if (isset($Page->raw)) {
-               print $Page->raw;
-       }
-       else {
+ob_start();
+if ($Page->handler and !require("./{$Page->handler}/index.php")) {
+       # replace contents by code output on false return
+       $Page->raw = ob_get_clean();
+}
+else {
+       # keep article contents
+       if (!isset($Page->body)) {
                # no resulting output
                http_response_code(404);
                @require '404.inc.html';
+               $Page->raw = ob_get_clean() . $Page->raw;
        }
 }