From f9db574a6006d2c7160e2156715b6c5a64f4d1ab Mon Sep 17 00:00:00 2001
From: Michael Tremer <michael.tremer@ipfire.org>
Date: Mon, 12 Nov 2018 17:22:26 +0000
Subject: [PATCH] wiki: Show recent changes

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
---
 Makefile.am                            |  4 +++-
 src/backend/wiki.py                    |  5 +++--
 src/templates/base.html                | 10 +++++++++-
 src/templates/wiki/modules/list.html   | 14 ++++++++++++++
 src/templates/wiki/recent-changes.html | 21 +++++++++++++++++++++
 src/web/__init__.py                    |  5 +++++
 src/web/wiki.py                        | 12 ++++++++++++
 7 files changed, 67 insertions(+), 4 deletions(-)
 create mode 100644 src/templates/wiki/modules/list.html
 create mode 100644 src/templates/wiki/recent-changes.html

diff --git a/Makefile.am b/Makefile.am
index 6bbd6078..08ef93a8 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -255,11 +255,13 @@ templates_staticdir = $(templatesdir)/static
 
 templates_wiki_DATA = \
 	src/templates/wiki/base.html \
-	src/templates/wiki/page.html
+	src/templates/wiki/page.html \
+	src/templates/wiki/recent-changes.html
 
 templates_wikidir = $(templatesdir)/wiki
 
 templates_wiki_modules_DATA = \
+	src/templates/wiki/modules/list.html \
 	src/templates/wiki/modules/navbar.html
 
 templates_wiki_modulesdir = $(templates_wikidir)/modules
diff --git a/src/backend/wiki.py b/src/backend/wiki.py
index 9dabb55c..26d96b4e 100644
--- a/src/backend/wiki.py
+++ b/src/backend/wiki.py
@@ -48,9 +48,10 @@ class Wiki(misc.Object):
 		if res:
 			return Page(self.backend, res.id, data=res)
 
-	def get_recent_changes(self):
+	def get_recent_changes(self, limit=None):
 		return self._get_pages("SELECT * FROM wiki \
-			WHERE timestamp >= NOW() - INTERVAL '4 weeks' ORDER BY timestamp DESC")
+			WHERE timestamp >= NOW() - INTERVAL '4 weeks' \
+			ORDER BY timestamp DESC LIMIT %s", limit)
 
 	def create_page(self, page, author, markdown):
 		page = Page.sanitise_page_name(page)
diff --git a/src/templates/base.html b/src/templates/base.html
index 858dca2b..7a7c314b 100644
--- a/src/templates/base.html
+++ b/src/templates/base.html
@@ -197,7 +197,15 @@
 						</button>
 
 						<div class="collapse navbar-collapse" id="navbar">
-							<form class="form-inline ml-lg-auto my-2 my-lg-0" action="/search" method="GET">
+							<ul class="navbar-nav ml-auto">
+								<li class="nav-item">
+									<a class="nav-link {% if request.path == "/recent-changes" %}active{% end %}" href="/recent-changes">
+										{{ _("Recent Changes") }}
+									</a>
+								</li>
+							</ul>
+
+							<form class="form-inline my-2 my-lg-0" action="/search" method="GET">
 								<input class="form-control form-control-sm" type="search" name="q"
 									placeholder="{{ _("Search...") }}" aria-label="{{ _("Search") }}" value="{% try %}{{ q }}{% except %}{% end %}">
 							</form>
diff --git a/src/templates/wiki/modules/list.html b/src/templates/wiki/modules/list.html
new file mode 100644
index 00000000..7e0a3a34
--- /dev/null
+++ b/src/templates/wiki/modules/list.html
@@ -0,0 +1,14 @@
+{% for page in pages %}
+	<strong class="mb-0">
+		<a href="{{ page.url }}">{{ page.title }}</a>
+	</strong>
+
+	<p class="text-muted small">
+		{{ locale.format_date(page.timestamp, shorter=True, relative=False) }}
+
+		{% if page.author %}
+			{{ _("by") }}
+			<a href="/users/{{ page.author.uid }}">{{ page.author }}</a>
+		{% end %}
+	</p>
+{% end %}
diff --git a/src/templates/wiki/recent-changes.html b/src/templates/wiki/recent-changes.html
new file mode 100644
index 00000000..b333f4a6
--- /dev/null
+++ b/src/templates/wiki/recent-changes.html
@@ -0,0 +1,21 @@
+{% extends "../base.html" %}
+
+{% block title %}{{ _("Recent Changes") }}{% end block %}
+
+{% block content %}
+	<section>
+		<div class="container">
+			<div class="row">
+				<div class="col col-lg-6">
+					<h1>{{ _("Recent Changes") }}</h1>
+				</div>
+			</div>
+		</div>
+	</section>
+
+	<div class="card">
+		<div class="card-body">
+			{% module WikiList(recent_changes) %}
+		</div>
+	</div>
+{% end block %}
diff --git a/src/web/__init__.py b/src/web/__init__.py
index 88ee2ca2..f961c3bf 100644
--- a/src/web/__init__.py
+++ b/src/web/__init__.py
@@ -91,6 +91,7 @@ class Application(tornado.web.Application):
 
 				# Wiki
 				"WikiNavbar"           : wiki.WikiNavbarModule,
+				"WikiList"             : wiki.WikiListModule,
 
 				# Misc
 				"Map"                  : ui_modules.MapModule,
@@ -283,7 +284,11 @@ class Application(tornado.web.Application):
 		self.add_handlers(r"wiki(\.dev)?\.ipfire\.org",
 			authentication_handlers + [
 
+			# Handlers
+			(r"/recent\-changes", wiki.RecentChangesHandler),
 			(r"/search", wiki.SearchHandler),
+
+			# Render pages
 			(r"([A-Za-z0-9\-_\/]+)?", wiki.PageHandler),
 		])
 
diff --git a/src/web/wiki.py b/src/web/wiki.py
index e2b5a57a..5ba76af0 100644
--- a/src/web/wiki.py
+++ b/src/web/wiki.py
@@ -51,6 +51,18 @@ class SearchHandler(auth.CacheMixin, base.BaseHandler):
 		self.render("wiki/search-results.html", q=q, pages=pages)
 
 
+class RecentChangesHandler(auth.CacheMixin, base.BaseHandler):
+	def get(self):
+		recent_changes = self.backend.wiki.get_recent_changes(limit=50)
+
+		self.render("wiki/recent-changes.html", recent_changes=recent_changes)
+
+
+class WikiListModule(ui_modules.UIModule):
+	def render(self, pages):
+		return self.render_string("wiki/modules/list.html", pages=pages)
+
+
 class WikiNavbarModule(ui_modules.UIModule):
 	def render(self, page, suffix=None):
 		breadcrumbs = self.backend.wiki.make_breadcrumbs(page.url)
-- 
2.47.3