D2826: hgweb: don't pass wsgireq to makeindex and other functions
indygreg (Gregory Szorc)
phabricator at mercurial-scm.org
Mon Mar 12 21:16:52 UTC 2018
indygreg created this revision.
Herald added a subscriber: mercurial-devel.
Herald added a reviewer: hg-reviewers.
REVISION SUMMARY
We only ever access attributes that are available on our newer
request type. So we no longer need this argument.
REPOSITORY
rHG Mercurial
REVISION DETAIL
https://phab.mercurial-scm.org/D2826
AFFECTED FILES
mercurial/hgweb/hgwebdir_mod.py
CHANGE DETAILS
diff --git a/mercurial/hgweb/hgwebdir_mod.py b/mercurial/hgweb/hgwebdir_mod.py
--- a/mercurial/hgweb/hgwebdir_mod.py
+++ b/mercurial/hgweb/hgwebdir_mod.py
@@ -119,7 +119,7 @@
return archives
-def rawindexentries(ui, repos, wsgireq, req, subdir=''):
+def rawindexentries(ui, repos, req, subdir=''):
descend = ui.configbool('web', 'descend')
collapse = ui.configbool('web', 'collapse')
seenrepos = set()
@@ -161,7 +161,7 @@
pass
parts = [
- wsgireq.req.apppath.strip('/'),
+ req.apppath.strip('/'),
subdir.strip('/'),
name.strip('/'),
]
@@ -245,10 +245,10 @@
yield row
-def indexentries(ui, repos, wsgireq, req, stripecount, sortcolumn='',
+def indexentries(ui, repos, req, stripecount, sortcolumn='',
descending=False, subdir=''):
- rows = rawindexentries(ui, repos, wsgireq, req, subdir=subdir)
+ rows = rawindexentries(ui, repos, req, subdir=subdir)
sortdefault = None, False
@@ -401,15 +401,15 @@
if (not virtual or virtual == 'index') and virtual not in repos:
wsgireq.respond(HTTP_OK, ctype)
- return self.makeindex(wsgireq, tmpl)
+ return self.makeindex(req, tmpl)
# nested indexes and hgwebs
if virtual.endswith('/index') and virtual not in repos:
subdir = virtual[:-len('index')]
if any(r.startswith(subdir) for r in repos):
wsgireq.respond(HTTP_OK, ctype)
- return self.makeindex(wsgireq, tmpl, subdir)
+ return self.makeindex(req, tmpl, subdir)
def _virtualdirs():
# Check the full virtual path, each parent, and the root ('')
@@ -443,7 +443,7 @@
subdir = virtual + '/'
if [r for r in repos if r.startswith(subdir)]:
wsgireq.respond(HTTP_OK, ctype)
- return self.makeindex(wsgireq, tmpl, subdir)
+ return self.makeindex(req, tmpl, subdir)
# prefixes not found
wsgireq.respond(HTTP_NOT_FOUND, ctype)
@@ -455,9 +455,7 @@
finally:
tmpl = None
- def makeindex(self, wsgireq, tmpl, subdir=""):
- req = wsgireq.req
-
+ def makeindex(self, req, tmpl, subdir=""):
self.refresh()
sortable = ["name", "description", "contact", "lastchange"]
sortcolumn, descending = None, False
@@ -476,7 +474,7 @@
self.refresh()
- entries = indexentries(self.ui, self.repos, wsgireq, req,
+ entries = indexentries(self.ui, self.repos, req,
self.stripecount, sortcolumn=sortcolumn,
descending=descending, subdir=subdir)
To: indygreg, #hg-reviewers
Cc: mercurial-devel
More information about the Mercurial-devel
mailing list