app/soc/views/helper/lists.py
changeset 2363 9046c80a9137
parent 2358 e5821b87f6e3
child 2855 ec2ed1571e3a
--- a/app/soc/views/helper/lists.py	Wed May 27 02:12:56 2009 +0200
+++ b/app/soc/views/helper/lists.py	Wed May 27 03:22:37 2009 +0200
@@ -198,18 +198,19 @@
                      i[0].startswith('offset_') or i[0].startswith('limit_'))
 
   if params.get('list_key_order'):
-    export_link = generateLinkForRequest(request, base_params, {'export' : idx})
+    export_link = generateLinkForRequest(request, base_params, {'export': idx})
 
   if more:
     # TODO(dbentley): here we need to implement a new field "last_key"
-    next = generateLinkForRequest(request, base_params, {offset_key : offset+limit,
-                                                         limit_key : limit})
+    next = generateLinkForRequest(request, base_params, 
+                                  {offset_key: offset + limit,
+                                   limit_key: limit})
 
   if offset > 0:
     # TODO(dbentley): here we need to implement previous in the good way.
     prev = generateLinkForRequest(request, base_params,
-                                  { offset_key : max(0, offset-limit),
-                                    limit_key : limit })
+                                  {offset_key: max(0, offset-limit),
+                                   limit_key: limit})
 
   if offset > limit:
     # Having a link to the first doesn't make sense on the first page (we're on
@@ -219,8 +220,8 @@
     # NOTE(dbentley): I personally disagree that it's simpler to do that way,
     # because sometimes you want to go to the first page without having to
     # consider what page you're on now.
-    newest = generateLinkForGetArgs(request, base_params, {offset_key : 0,
-                                                           limit_key : limit})
+    newest = generateLinkForRequest(request, base_params, {offset_key: 0,
+                                                           limit_key: limit})
 
   content = {
       'idx': idx,