From ed38c6a76767a893a319f7bfd9229b0dad6b08db Mon Sep 17 00:00:00 2001
From: Mischa POSLAWSKY <perl@shiar.org>
Date: Tue, 4 Feb 2020 20:52:00 +0100
Subject: [PATCH] page: article method to find handler code

---
 article.inc.php | 24 +++++++++++++++++++++
 page.php        | 57 ++++++++++++++++++++-----------------------------
 2 files changed, 47 insertions(+), 34 deletions(-)

diff --git a/article.inc.php b/article.inc.php
index da18368..d8ddf77 100644
--- a/article.inc.php
+++ b/article.inc.php
@@ -50,6 +50,30 @@ class ArchiveArticle
 		return $this->$col = $this->$col();  # run method and cache
 	}
 
+	function handler()
+	{
+		$path = $this->link;
+		$this->path = '';
+		$this->restricted = FALSE;
+		while (TRUE) {
+			if (file_exists("$path/.private")) {
+				$this->restricted = $path;
+			}
+
+			if (file_exists("$path/index.php")) {
+				return $path;
+			}
+
+			$up = strrpos($path, '/');
+			$this->path = substr($path, $up) . $this->path;
+			$path = substr($path, 0, $up);
+			if ($up === FALSE) {
+				break;
+			}
+		}
+		return;
+	}
+
 	function safetitle()
 	{
 		return trim($this->meta['og:title'] ?? strip_tags($this->title));
diff --git a/page.php b/page.php
index e23b3e2..f4d6057 100644
--- a/page.php
+++ b/page.php
@@ -122,38 +122,14 @@ $User = NULL;
 include_once 'auth.inc.php';
 $Edit = isset($_GET['edit']);
 
-# distinguish subpage Args from topmost Page script
+# setup requested page
 
 $Args = '';
 $Page = preg_replace('/\?.*/', '', @$_SERVER['PATH_INFO'] ?: $_SERVER['REQUEST_URI']);
 $Page = urldecode(trim($Page, '/')) ?: 'index';
-while (TRUE) {
-	if (file_exists("$Page/.private")) {
-		# access restriction
-		if (empty($User)) {
-			http_response_code(303);
-			$target = urlencode($_SERVER['REQUEST_URI']);
-			header("Location: /login?goto=$target");
-			exit;
-		}
-		$PageAccess = $Page;
-	}
 
-	if (file_exists("$Page/index.php")) {
-		break;
-	}
-
-	$up = strrpos($Page, '/');
-	$Args = substr($Page, $up) . $Args;
-	$Page = substr($Page, 0, $up);
-	if ($up === FALSE) {
-		break;
-	}
-}
-
-$staticpage = NULL;
-if (file_exists("$Page$Args.html")) {
-	$staticpage = "$Page$Args.html";
+$staticpage = "$Page.html";
+if (file_exists($staticpage)) {
 	if (is_link($staticpage)) {
 		$target = preg_replace('/\.html$/', '', readlink($staticpage));
 		header("HTTP/1.1 302 Shorthand");
@@ -161,18 +137,28 @@ if (file_exists("$Page$Args.html")) {
 		exit;
 	}
 }
-elseif (file_exists("$Page$Args/index.html")) {
-	$staticpage = "$Page$Args/index.html";
+elseif (file_exists("$Page/index.html")) {
+	$staticpage = "$Page/index.html";
 }
-elseif ($User and $User->admin("edit $Page$Args")) {
-	$staticpage = (file_exists("$Page/template.inc.html") ? "$Page/template.inc.html" : 'template.inc.html');
-}
-
-# prepare page contents
 
 require_once('article.inc.php');
 $Article = new ArchiveArticle($staticpage);
 
+$Page = $Article->handler;
+$Args = $Article->path;
+
+if ($PageAccess = $Article->restricted) {
+	# access restriction
+	if (empty($User)) {
+		http_response_code(303);
+		$target = urlencode($Article->link);
+		header("Location: /login?goto=$target");
+		exit;
+	}
+}
+
+# prepare page contents
+
 ob_start(); # page body
 $Place = [
 	'user'  => $User ? $User->login : '',
@@ -191,6 +177,9 @@ if (isset($Article->raw)) {
 	}
 	$Article->raw = '<div class="static">'."\n\n".$Article->raw."</div>\n\n";
 }
+elseif (!$Article->raw and $User and $User->admin("edit {$Article->link}")) {
+	$Article->raw(file_exists("$Page/template.inc.html") ? "$Page/template.inc.html" : 'template.inc.html');
+}
 
 # output dynamic and/or static html
 
-- 
2.30.0