From: Michael Tremer <michael.tremer@ipfire.org>
Date: Thu, 6 Dec 2018 11:11:20 +0000 (+0000)
Subject: wiki: Remove doubly-used variable page
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=5bc8e262c43cf8605bb31965b5570e47a5f098d3;p=ipfire.org.git

wiki: Remove doubly-used variable page

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
---

diff --git a/src/web/wiki.py b/src/web/wiki.py
index 2a1dd45b..1aae7e1d 100644
--- a/src/web/wiki.py
+++ b/src/web/wiki.py
@@ -158,16 +158,16 @@ class PageHandler(auth.CacheMixin, base.BaseHandler):
 		super().write_error(status_code, **kwargs)
 
 	@tornado.web.removeslash
-	def get(self, page):
+	def get(self, path):
 		# Check permissions
-		if not self.backend.wiki.check_acl(page, self.current_user):
-			raise tornado.web.HTTPError(403, "Access to %s not allowed for %s" % (page, self.current_user))
+		if not self.backend.wiki.check_acl(path, self.current_user):
+			raise tornado.web.HTTPError(403, "Access to %s not allowed for %s" % (path, self.current_user))
 
 		# Check if we are asked to render a certain revision
 		revision = self.get_argument("revision", None)
 
 		# Fetch the wiki page
-		page = self.backend.wiki.get_page(page, revision=revision)
+		page = self.backend.wiki.get_page(path, revision=revision)
 
 		# Diff
 		if self.action == "diff":
@@ -176,8 +176,8 @@ class PageHandler(auth.CacheMixin, base.BaseHandler):
 			b = self.get_argument("b")
 
 			# Fetch both versions of the page
-			a = self.backend.wiki.get_page(page.page, revision=a)
-			b = self.backend.wiki.get_page(page.page, revision=b)
+			a = self.backend.wiki.get_page(path, revision=a)
+			b = self.backend.wiki.get_page(path, revision=b)
 			if not a or not b:
 				raise tornado.web.HTTPError(404)