Patchwork [4,of,4,V4] hgweb: always run search when a query is entered (bc)

login
register
mail settings
Submitter Alexander Plavin
Date Aug. 27, 2013, 6:10 p.m.
Message ID <a43f5f2367860363a2aa.1377627036@debian-alexander.dolgopa>
Download mbox | patch
Permalink /patch/2278/
State Accepted
Headers show

Comments

Alexander Plavin - Aug. 27, 2013, 6:10 p.m.
# HG changeset patch
# User Alexander Plavin <alexander@plav.in>
# Date 1374185353 -14400
#      Fri Jul 19 02:09:13 2013 +0400
# Node ID a43f5f2367860363a2aab22fbf466b6a63000c3d
# Parent  1ea68b37d07701cc53741cd96cc042ac5c62b96a
hgweb: always run search when a query is entered (bc)

This changes the behavior for queries which point at a revision directly,
now the output is consistent to other cases: it results in only this matched
revision shown, not the log starting with it.
A new test checks this behaviour and fails for the old one.

Patch

diff -r 1ea68b37d077 -r a43f5f236786 mercurial/hgweb/webcommands.py
--- a/mercurial/hgweb/webcommands.py	Fri Jul 19 02:41:11 2013 +0400
+++ b/mercurial/hgweb/webcommands.py	Fri Jul 19 02:09:13 2013 +0400
@@ -213,11 +213,7 @@ 
     if 'node' in req.form:
         ctx = webutil.changectx(web.repo, req)
     elif 'rev' in req.form:
-        query = req.form['rev'][0]
-        try:
-            ctx = web.repo[query]
-        except (error.RepoError, error.LookupError):
-            return _search(web, req, tmpl) # XXX redirect to 404 page?
+        return _search(web, req, tmpl)
     else:
         ctx = web.repo['tip']
 
diff -r 1ea68b37d077 -r a43f5f236786 tests/test-hgweb-commands.t
--- a/tests/test-hgweb-commands.t	Fri Jul 19 02:41:11 2013 +0400
+++ b/tests/test-hgweb-commands.t	Fri Jul 19 02:09:13 2013 +0400
@@ -534,6 +534,8 @@ 
   </body>
   </html>
   
+  $ "$TESTDIR/get-with-headers.py" 127.0.0.1:$HGPORT 'log?rev=stable&style=raw' | grep 'revision:'
+  revision:    2
 
 File-related