From b4e872adfb3c7b67447826a5b162f89590e14c80 Mon Sep 17 00:00:00 2001
From: Mischa POSLAWSKY <perl@shiar.org>
Date: Fri, 15 Jan 2021 10:24:40 +0100
Subject: [PATCH] login/commits: git show of hash parameter

Link overview to further details, currently unformatted.
---
 login/commits/index.php  | 28 ++++++++++++++++++++++++++++
 widget/login/commits.php | 15 ++++++++++-----
 2 files changed, 38 insertions(+), 5 deletions(-)
 create mode 100644 login/commits/index.php

diff --git a/login/commits/index.php b/login/commits/index.php
new file mode 100644
index 0000000..c53bf39
--- /dev/null
+++ b/login/commits/index.php
@@ -0,0 +1,28 @@
+<?php
+if (!$User->admin('edit')) {
+	require '403.inc.html';
+	return;
+}
+
+$hash = ltrim($Page->path, '/');
+if (!$hash) {
+	return TRUE;
+}
+
+print "<h2>Wijzigingen in $hash</h2>\n";
+
+$gitcmd = "git show ".$hash;
+$log = popen($gitcmd, 'r');
+if (!$log or strpos(fgets($log), "commit $hash") !== 0) {
+	$Page->place['warn'] = "Kon inhoud niet ophalen met <code>$gitcmd</code>";
+	return;
+}
+
+print '<pre>';
+while ( $line = fgets($log) ) {
+	print htmlspecialchars($line);
+}
+print "</pre>\n";
+
+pclose($log);
+return;
diff --git a/widget/login/commits.php b/widget/login/commits.php
index 18447a7..2060a33 100644
--- a/widget/login/commits.php
+++ b/widget/login/commits.php
@@ -4,8 +4,9 @@ if (!function_exists('popen')) {
 	return;
 }
 
+$baseurl = 'login/commits';
 $pagesize = intval(@$Page->place['n'] ?: @$_GET['n']) ?: 20;
-$gitcmd = "git log -n $pagesize --pretty='%at\t%an\t%s'";
+$gitcmd = "git log -n $pagesize --pretty='%h\t%at\t%an\t%s'";
 
 if ( $offset = intval(@$_GET['start']) ) {
 	$gitcmd .= " --skip=$offset";
@@ -15,12 +16,16 @@ if ( $log = popen($gitcmd, 'r') ) {
 	$lines = 0;
 	print "<ul>\n";
 	while ( $line = fgets($log) ) {
-		list ($atime, $author, $message) = explode("\t", $line, 3);
+		list ($id, $atime, $author, $message) = explode("\t", $line, 4);
 		list ($author) = explode(' ', $author); # first name only
-		printf('<li>%s <small class="date">%s • %s</small></li>'."\n",
-			htmlspecialchars($message),
+		$html = htmlspecialchars(rtrim($message));
+		$html .= sprintf(' <small class="date">%s • %s</small>',
 			htmlspecialchars($author), strftime('%F %H:%M', $atime)
 		);
+		$html = sprintf('<a href="%s">%s</a>',
+			"/$baseurl/$id", $html
+		);
+		print "<li>$html</li>\n";
 		$lines++;
 	}
 	print "</ul>\n\n";
@@ -29,6 +34,6 @@ if ( $log = popen($gitcmd, 'r') ) {
 	$limit = $offset + $lines + 1; # assume one more
 	print $Page->widget('nav', [
 		'start' => $offset, 'n' => $pagesize, 'total' => $limit,
-		'link' => $Page->link != 'login/commits' ? 'login/commits' : '',
+		'link' => $Page->link == $baseurl ? NULL : $baseurl,
 	]);
 }
-- 
2.30.0