hgweb: make 'summary' work with hidden changesets (issue3810) stable
authorKevin Bullock <kbullock@ringworld.org>
Tue, 05 Feb 2013 11:31:43 -0600
branchstable
changeset 18563 6d098adc5a46
parent 18561 7365d031d457
child 18564 2fefd1170bf2
child 18568 cd403d6d96ef
hgweb: make 'summary' work with hidden changesets (issue3810) Since the 'summary' view used by e.g. gitweb and monoblue shows both a changelog and a bookmarks list, the same changes are needed here as were made to the 'changelog' and 'bookmarks' web commands (56ca4443a343 and 886936ecc21b, respectively).
mercurial/hgweb/webcommands.py
tests/test-hgweb-commands.t
--- a/mercurial/hgweb/webcommands.py	Tue Feb 05 12:58:21 2013 +0100
+++ b/mercurial/hgweb/webcommands.py	Tue Feb 05 11:31:43 2013 -0600
@@ -497,8 +497,8 @@
 
     def bookmarks(**map):
         parity = paritygen(web.stripecount)
-        b = web.repo._bookmarks.items()
-        for k, n in sorted(b)[:10]:  # limit to 10 bookmarks
+        marks = [b for b in web.repo._bookmarks.items() if b[1] in web.repo]
+        for k, n in sorted(marks)[:10]:  # limit to 10 bookmarks
             yield {'parity': parity.next(),
                    'bookmark': k,
                    'date': web.repo[n].date(),
@@ -518,7 +518,10 @@
     def changelist(**map):
         parity = paritygen(web.stripecount, offset=start - end)
         l = [] # build a list in forward order for efficiency
-        for i in xrange(start, end):
+        revs = []
+        if start < end:
+            revs = web.repo.changelog.revs(start, end - 1)
+        for i in revs:
             ctx = web.repo[i]
             n = ctx.node()
             hn = hex(n)
--- a/tests/test-hgweb-commands.t	Tue Feb 05 12:58:21 2013 +0100
+++ b/tests/test-hgweb-commands.t	Tue Feb 05 11:31:43 2013 -0600
@@ -1399,6 +1399,14 @@
   $ grep Status search
   Status: 200 Script output follows\r (esc)
 
+summary works with filtering (issue3810)
+
+  $ PATH_INFO=/summary; export PATH_INFO
+  $ QUERY_STRING='style=monoblue'; export QUERY_STRING
+  $ python hgweb.cgi > summary.out
+  $ grep "^Status" summary.out
+  Status: 200 Script output follows\r (esc)
+
 proper status for filtered revision