Ticket #2: viewmtn_shortchanges.diff

handlers.py

@@ -215 +215 @@
         return rv
     
     def determine_bounds(self, from_change, to_change):
-        per_page = 10
+        per_page = 50
         max_per_page = 100
         if from_change: from_change = int(from_change)
         else: from_change = 0
@@ -313 +313 @@
     def GET(self, ctxt, branch, from_change, to_change):
         Changes.Branch_GET(self, ctxt, branch, from_change, to_change, "branchchanges.html")
 
+class HTMLBranchShortChanges(Changes):
+    def GET(self, ctxt, branch, from_change, to_change):
+        Changes.Branch_GET(self, ctxt, branch, from_change, to_change, "branchshortchanges.html")
+
 class RSSBranchChanges(Changes):
     def GET(self, ctxt, branch, from_change, to_change):
         Changes.Branch_GET(self, ctxt, branch, from_change, to_change, "branchchangesrss.html")
@@ -797 +804 @@
                 # the goal is just to let a robot trawl through the most recent changes, and deny access 
                 # to expensive pointless things. We don't want a robot indexing every file in every revision, 
                 # as this is an enormous amount of information.
-                for access_method in ['/revision/', '/branch/head/', '/branch/anyhead/', '/branch/changes/from/', '/json/', '/mimeicon/']:
+                for access_method in ['/revision/', '/branch/head/', '/branch/anyhead/', '/branch/changes/from/', '/branch/shortchanges/from/', '/json/', '/mimeicon/']:
                     disallow = access_method + revision_page
                     if not dbname is None:
                         disallow = '/' + dbname + disallow

links.py

@@ -88 +88 @@
     def __init__(self, branch, **kwargs):
         Link.__init__(*(self, ), **kwargs)
         self.json_args = [branch.name]
+        if kwargs.get('short_changes'):
+            prefix = 'branch/shortchanges'
+        else:
+            prefix = 'branch/changes'
         from_change, to_change = kwargs.get('from_change'), kwargs.get('to_change')
         if from_change and to_change:
-            self.relative_uri = 'branch/changes/%s/from/%d/to/%d' % (urllib.quote(branch.name, safe = ''), from_change, to_change)
+            self.relative_uri = '%s/%s/from/%d/to/%d' % (prefix, urllib.quote(branch.name, safe = ''), from_change, to_change)
         else:
-            self.relative_uri = 'branch/changes/' + urllib.quote(branch.name, safe = '')
+            self.relative_uri = '%s/%s' % (prefix, urllib.quote(branch.name, safe = ''))
         self.description = hq(branch.name)
 
 class DiffLink(Link):

templates/branch.html

@@ -2 +2 @@
 
 #def extramenu
 <strong>Branch $branch.name</strong>:
+<a href="$perdb_join('branch/shortchanges/%s' % $urllib_quote($branch.name, safe=''))">Concise Changes</a> |
 <a href="$perdb_join('branch/changes/%s' % $urllib_quote($branch.name, safe=''))">Changes</a> |
 <a href="$perdb_join('branch/head/info/%s' % $urllib_quote($branch.name, safe=''))">Head revision</a> |
 <a href="$perdb_join('branch/tags/%s' % $urllib_quote($branch.name, safe=''))">Tags</a> |

urls.py

@@ -53 +53 @@
 
     r'branch/changes/(.*)/from/(\d+)/to/(\d+)', 'HTMLBranchChanges',
     r'branch/changes/([^/]+)()()', 'HTMLBranchChanges',
+    r'branch/shortchanges/(.*)/from/(\d+)/to/(\d+)', 'HTMLBranchShortChanges',
+    r'branch/shortchanges/([^/]+)()()', 'HTMLBranchShortChanges',
     r'branch/changes/(.*)/from/(\d+)/to/(\d+)/rss', 'RSSBranchChanges',
     r'branch/changes/([^/]+)()()/rss', 'RSSBranchChanges',
     r'branch/tags/([^/]+)', 'Tags',