page: delay loading user code until after page

[minimedit.git] / page.php

diff --git a/page.php b/page.php
index 5eefadb5c8870e13ad8eeb045dce849f1ff258c3..a4440cb2ff7447e9b1f49f2eb7b4e167f7a9102b 100644 (file)
--- a/page.php
+++ b/page.php
@@ -2,58 +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';
-       $Page->place['debug'] = htmlspecialchars($error);
-       print $Page->render();
-}
-
-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);
-
-# user login and control
-
-include_once 'auth.inc.php'; // sets global $User
-$Edit = isset($_GET['edit']);
+include_once 'error.inc.php';
 
 # setup requested page
 
@@ -76,6 +28,10 @@ elseif (file_exists("$request/index.html")) {
 require_once('article.inc.php');
 $Page = new ArchiveArticle($staticpage);
 
+# user login and control
+
+include_once 'auth.inc.php'; // sets global $User
+
 if ($Page->restricted) {
        # access restriction
        if (!$User->login) {
@@ -95,7 +51,6 @@ header(sprintf('Content-Security-Policy: %s', implode('; ', [
        "frame-ancestors 'none'", # prevent malicious embedding
 ])));
 
-ob_start(); # page body
 $Page->place += [
        'user'  => $User->login ?: '',
        'url'   => htmlspecialchars($_SERVER['REQUEST_URI']),
@@ -110,6 +65,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)) {
@@ -127,15 +83,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;
        }
 }