| author | Lennard de Rijk <ljvderijk@gmail.com> |
| Wed, 12 Aug 2009 15:44:18 -0700 | |
| changeset 2766 | 93e4745ebdbd |
| parent 2363 | 9046c80a9137 |
| child 2855 | ec2ed1571e3a |
| permissions | -rw-r--r-- |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
1 |
#!/usr/bin/python2.5 |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
2 |
# |
|
1308
35b75ffcbb37
Partially reverted "Update the copyright notice for 2009."
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1307
diff
changeset
|
3 |
# Copyright 2008 the Melange authors. |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
4 |
# |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
5 |
# Licensed under the Apache License, Version 2.0 (the "License"); |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
6 |
# you may not use this file except in compliance with the License. |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
7 |
# You may obtain a copy of the License at |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
8 |
# |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
9 |
# http://www.apache.org/licenses/LICENSE-2.0 |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
10 |
# |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
11 |
# Unless required by applicable law or agreed to in writing, software |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
12 |
# distributed under the License is distributed on an "AS IS" BASIS, |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
13 |
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
14 |
# See the License for the specific language governing permissions and |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
15 |
# limitations under the License. |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
16 |
|
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
17 |
"""Helpers used to render lists. |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
18 |
""" |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
19 |
|
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
20 |
__authors__ = [ |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
21 |
'"Chen Lunpeng" <forever.clp@gmail.com>', |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
22 |
'"Pawel Solyga" <pawel.solyga@gmail.com>', |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
23 |
] |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
24 |
|
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
25 |
|
|
555
3cdfb42d941b
Split the list view code up in three pieces
Sverre Rabbelier <srabbelier@gmail.com>
parents:
553
diff
changeset
|
26 |
from soc.logic import dicts |
|
1802
407000879987
Allow developers to specify a higher limit for a list view
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1692
diff
changeset
|
27 |
from soc.logic.models.user import logic as user_logic |
|
316
9efdc7bc3565
Add missing blank lines between imports and sort all of the imports.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
277
diff
changeset
|
28 |
|
|
274
56e1c1721299
Move helpers/forms_helpers.py to helper/forms.py.
Todd Larsen <tlarsen@google.com>
parents:
268
diff
changeset
|
29 |
import soc.views.helper.forms |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
30 |
|
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
31 |
|
|
1810
ec0bae3632bb
Adhere to the DEF_... pattern for constants in lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1807
diff
changeset
|
32 |
DEF_DEFAULT_PAGINATION = 50 |
|
ec0bae3632bb
Adhere to the DEF_... pattern for constants in lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1807
diff
changeset
|
33 |
DEF_MAX_PAGINATION = 100 |
|
ec0bae3632bb
Adhere to the DEF_... pattern for constants in lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1807
diff
changeset
|
34 |
DEF_MAX_DEV_PAGINATION = 1000 |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
35 |
|
|
1811
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
36 |
DEF_PAGINATION_CHOICES = [ |
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
37 |
('10', '10 items per page'),
|
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
38 |
('25', '25 items per page'),
|
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
39 |
('50', '50 items per page'),
|
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
40 |
('100', '100 items per page'),
|
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
41 |
] |
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
42 |
|
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
43 |
DEF_DEVELOPER_CHOICES = [ |
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
44 |
('500', '500 items per page'),
|
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
45 |
('1000', '1000 items per page'),
|
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
46 |
] |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
47 |
|
|
228
2204287da374
Add placeholder for future (to-be-implemented) getPreferredListPagination()
Todd Larsen <tlarsen@google.com>
parents:
185
diff
changeset
|
48 |
|
|
2204287da374
Add placeholder for future (to-be-implemented) getPreferredListPagination()
Todd Larsen <tlarsen@google.com>
parents:
185
diff
changeset
|
49 |
def getPreferredListPagination(user=None): |
|
389
9b873166d7d5
Fix identions, too long lines, unused imports and some other mistakes.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
316
diff
changeset
|
50 |
"""Returns User's preferred list pagination limit. |
|
2049
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
51 |
|
|
389
9b873166d7d5
Fix identions, too long lines, unused imports and some other mistakes.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
316
diff
changeset
|
52 |
Args: |
|
9b873166d7d5
Fix identions, too long lines, unused imports and some other mistakes.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
316
diff
changeset
|
53 |
user: User entity containing the list pagination preference; |
|
9b873166d7d5
Fix identions, too long lines, unused imports and some other mistakes.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
316
diff
changeset
|
54 |
default is None, to use the current logged-in User |
|
9b873166d7d5
Fix identions, too long lines, unused imports and some other mistakes.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
316
diff
changeset
|
55 |
""" |
|
9b873166d7d5
Fix identions, too long lines, unused imports and some other mistakes.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
316
diff
changeset
|
56 |
# TODO: eventually this limit should be a User profile preference |
|
2049
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
57 |
# (stored in the site-wide User Model) preference |
|
1810
ec0bae3632bb
Adhere to the DEF_... pattern for constants in lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1807
diff
changeset
|
58 |
return DEF_DEFAULT_PAGINATION |
|
228
2204287da374
Add placeholder for future (to-be-implemented) getPreferredListPagination()
Todd Larsen <tlarsen@google.com>
parents:
185
diff
changeset
|
59 |
|
|
2204287da374
Add placeholder for future (to-be-implemented) getPreferredListPagination()
Todd Larsen <tlarsen@google.com>
parents:
185
diff
changeset
|
60 |
|
|
2166
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
61 |
OFFSET_KEY = 'offset_%d' |
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
62 |
LIMIT_KEY = 'limit_%d' |
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
63 |
|
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
64 |
|
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
65 |
def makeOffsetKey(limit_idx): |
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
66 |
return OFFSET_KEY % limit_idx |
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
67 |
|
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
68 |
|
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
69 |
def makeLimitKey(limit_idx): |
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
70 |
return LIMIT_KEY % limit_idx |
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
71 |
|
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
72 |
|
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
73 |
def getListParameters(request, list_index): |
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
74 |
"""Retrieves, converts and validates values for one list |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
75 |
|
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
76 |
Args: |
|
2166
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
77 |
list_index, int: which list to get the values for. |
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
78 |
(there may be multiple lists on one page, which are multiplexed |
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
79 |
by an integer.) |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
80 |
|
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
81 |
Returns: |
|
2166
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
82 |
a dictionary of str -> str. field name -> field value. |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
83 |
""" |
|
572
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
84 |
|
|
2166
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
85 |
offset = request.GET.get(makeOffsetKey(list_index)) |
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
86 |
limit = request.GET.get(makeLimitKey(list_index)) |
|
572
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
87 |
|
|
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
88 |
if offset is None: |
|
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
89 |
offset = '' |
|
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
90 |
|
|
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
91 |
if limit is None: |
|
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
92 |
limit = '' |
|
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
93 |
|
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
94 |
try: |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
95 |
offset = int(offset) |
|
572
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
96 |
except ValueError: |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
97 |
offset = 0 |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
98 |
|
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
99 |
try: |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
100 |
limit = int(limit) |
|
572
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
101 |
except ValueError: |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
102 |
limit = getPreferredListPagination() |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
103 |
|
|
572
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
104 |
offset = max(0, offset) |
|
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
105 |
limit = max(1, limit) |
|
1802
407000879987
Allow developers to specify a higher limit for a list view
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1692
diff
changeset
|
106 |
|
|
1810
ec0bae3632bb
Adhere to the DEF_... pattern for constants in lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1807
diff
changeset
|
107 |
if user_logic.isDeveloper(): |
|
ec0bae3632bb
Adhere to the DEF_... pattern for constants in lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1807
diff
changeset
|
108 |
limit = min(DEF_MAX_DEV_PAGINATION, limit) |
|
ec0bae3632bb
Adhere to the DEF_... pattern for constants in lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1807
diff
changeset
|
109 |
else: |
|
ec0bae3632bb
Adhere to the DEF_... pattern for constants in lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1807
diff
changeset
|
110 |
limit = min(DEF_MAX_PAGINATION, limit) |
|
572
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
111 |
|
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
112 |
return dict(limit=limit, offset=offset) |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
113 |
|
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
114 |
|
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
115 |
def generateLinkFromGetArgs(request, offset_and_limits): |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
116 |
"""Constructs the get args for the url. |
|
1166
558bd62ee9d4
Fix get args construction when there are multiple lists on the page
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1165
diff
changeset
|
117 |
""" |
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
118 |
|
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
119 |
args = ["%s=%s" % (k, v) for k, v in offset_and_limits.iteritems()] |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
120 |
link_suffix = '?' + '&'.join(args) |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
121 |
|
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
122 |
return request.path + link_suffix |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
123 |
|
|
1166
558bd62ee9d4
Fix get args construction when there are multiple lists on the page
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1165
diff
changeset
|
124 |
|
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
125 |
def generateLinkForRequest(request, base_params, updated_params): |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
126 |
"""Create a link to the same page as request but with different params |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
127 |
|
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
128 |
Params: |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
129 |
request: the request for the page |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
130 |
base_params: the base parameters |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
131 |
updated_params: the parameters to update |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
132 |
""" |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
133 |
params = base_params.copy() |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
134 |
params.update(updated_params) |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
135 |
return generateLinkFromGetArgs(request, params) |
|
2166
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
136 |
|
|
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
137 |
|
|
1667
95bc81b4cd5c
Updated list to use the new order in getForFields.
Lennard de Rijk <ljvderijk@gmail.com>
parents:
1604
diff
changeset
|
138 |
def getListContent(request, params, filter=None, order=None, |
|
95bc81b4cd5c
Updated list to use the new order in getForFields.
Lennard de Rijk <ljvderijk@gmail.com>
parents:
1604
diff
changeset
|
139 |
idx=0, need_content=False): |
|
539
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
140 |
"""Returns a dict with fields used for rendering lists. |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
141 |
|
|
2166
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
142 |
TODO(dbentley): we need better terminology. List, in this context, can have |
|
2049
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
143 |
one of two meanings. |
|
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
144 |
Meaning 1: the underlying list, which may be very large. |
|
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
145 |
Meaning 2: the returned list, which is at most 'limit' items. |
|
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
146 |
|
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
147 |
Args: |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
148 |
request: the Django HTTP request object |
|
555
3cdfb42d941b
Split the list view code up in three pieces
Sverre Rabbelier <srabbelier@gmail.com>
parents:
553
diff
changeset
|
149 |
params: a dict with params for the View this list belongs to |
|
3cdfb42d941b
Split the list view code up in three pieces
Sverre Rabbelier <srabbelier@gmail.com>
parents:
553
diff
changeset
|
150 |
filter: a filter for this list |
|
1667
95bc81b4cd5c
Updated list to use the new order in getForFields.
Lennard de Rijk <ljvderijk@gmail.com>
parents:
1604
diff
changeset
|
151 |
order: the order which should be used for the list (in getForFields format) |
|
572
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
152 |
idx: the index of this list |
|
1604
297444daac68
Two docstring fixes.
Lennard de Rijk <ljvderijk@gmail.com>
parents:
1459
diff
changeset
|
153 |
need_content: iff True will return None if there is no data |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
154 |
|
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
155 |
Returns: |
|
2049
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
156 |
A dictionary with the following values set: |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
157 |
|
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
158 |
{
|
|
539
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
159 |
'data': list data to be displayed |
|
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
160 |
'main': url to list main template |
|
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
161 |
'pagination': url to list pagination template |
|
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
162 |
'row': url to list row template |
|
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
163 |
'heading': url to list heading template |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
164 |
'limit': max amount of items per page, |
|
2049
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
165 |
'newest': url to first page of the list |
|
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
166 |
'prev': url to previous page |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
167 |
'next': url to next page |
|
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
168 |
'first': offset of the first item in the list |
|
2166
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
169 |
'last': offset of the last item in the list |
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
170 |
} |
|
539
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
171 |
""" |
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
172 |
# TODO(dbentley): this appears to be unnecessary indirection, |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
173 |
# as we only use this logic for getForFields, which is never overridden |
|
651
ef6e22d463cb
Remove the logic parameter from getListContent
Sverre Rabbelier <srabbelier@gmail.com>
parents:
632
diff
changeset
|
174 |
logic = params['logic'] |
|
ef6e22d463cb
Remove the logic parameter from getListContent
Sverre Rabbelier <srabbelier@gmail.com>
parents:
632
diff
changeset
|
175 |
|
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
176 |
limit_key, offset_key = makeLimitKey(idx), makeOffsetKey(idx) |
|
1166
558bd62ee9d4
Fix get args construction when there are multiple lists on the page
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1165
diff
changeset
|
177 |
|
|
2166
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
178 |
list_params = getListParameters(request, idx) |
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
179 |
limit, offset = list_params['limit'], list_params['offset'] |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
180 |
pagination_form = makePaginationForm(request, list_params['limit'], |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
181 |
limit_key) |
|
2313
c39a81bce1bd
Use offset_linkid instead of offset to scan >1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2312
diff
changeset
|
182 |
|
|
555
3cdfb42d941b
Split the list view code up in three pieces
Sverre Rabbelier <srabbelier@gmail.com>
parents:
553
diff
changeset
|
183 |
# Fetch one more to see if there should be a 'next' link |
|
1667
95bc81b4cd5c
Updated list to use the new order in getForFields.
Lennard de Rijk <ljvderijk@gmail.com>
parents:
1604
diff
changeset
|
184 |
data = logic.getForFields(filter=filter, limit=limit+1, offset=offset, |
|
95bc81b4cd5c
Updated list to use the new order in getForFields.
Lennard de Rijk <ljvderijk@gmail.com>
parents:
1604
diff
changeset
|
185 |
order=order) |
|
723
69e5130e4a0a
Make getForFields use the db.Query API
Sverre Rabbelier <srabbelier@gmail.com>
parents:
651
diff
changeset
|
186 |
|
|
1459
2584724403ff
Make it possible not to generate empty lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1308
diff
changeset
|
187 |
if need_content and not data: |
|
2584724403ff
Make it possible not to generate empty lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1308
diff
changeset
|
188 |
return None |
|
2584724403ff
Make it possible not to generate empty lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1308
diff
changeset
|
189 |
|
|
723
69e5130e4a0a
Make getForFields use the db.Query API
Sverre Rabbelier <srabbelier@gmail.com>
parents:
651
diff
changeset
|
190 |
more = len(data) > limit |
|
1807
1f8cde169f32
Add an export link to all list views
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1802
diff
changeset
|
191 |
|
|
185
2f3bd84bb106
Add list_helpers functions that are going to be used when rendering list views like (Users List, Sponsors List etc).
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
diff
changeset
|
192 |
if more: |
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
193 |
del data[limit:] |
|
2316
3156760b4d26
Use key_name instead of link_id
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2313
diff
changeset
|
194 |
|
|
2313
c39a81bce1bd
Use offset_linkid instead of offset to scan >1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2312
diff
changeset
|
195 |
newest = next = prev = export_link = '' |
|
c39a81bce1bd
Use offset_linkid instead of offset to scan >1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2312
diff
changeset
|
196 |
|
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
197 |
base_params = dict(i for i in request.GET.iteritems() if |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
198 |
i[0].startswith('offset_') or i[0].startswith('limit_'))
|
|
2166
c9c7c6111988
Last step of refactoring before adding support for > 1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2108
diff
changeset
|
199 |
|
|
2313
c39a81bce1bd
Use offset_linkid instead of offset to scan >1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2312
diff
changeset
|
200 |
if params.get('list_key_order'):
|
|
2363
9046c80a9137
Style fixes and wrong function call fix in soc.views.helper.lists module.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
2358
diff
changeset
|
201 |
export_link = generateLinkForRequest(request, base_params, {'export': idx})
|
|
2313
c39a81bce1bd
Use offset_linkid instead of offset to scan >1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2312
diff
changeset
|
202 |
|
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
203 |
if more: |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
204 |
# TODO(dbentley): here we need to implement a new field "last_key" |
|
2363
9046c80a9137
Style fixes and wrong function call fix in soc.views.helper.lists module.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
2358
diff
changeset
|
205 |
next = generateLinkForRequest(request, base_params, |
|
9046c80a9137
Style fixes and wrong function call fix in soc.views.helper.lists module.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
2358
diff
changeset
|
206 |
{offset_key: offset + limit,
|
|
9046c80a9137
Style fixes and wrong function call fix in soc.views.helper.lists module.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
2358
diff
changeset
|
207 |
limit_key: limit}) |
|
2313
c39a81bce1bd
Use offset_linkid instead of offset to scan >1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2312
diff
changeset
|
208 |
|
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
209 |
if offset > 0: |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
210 |
# TODO(dbentley): here we need to implement previous in the good way. |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
211 |
prev = generateLinkForRequest(request, base_params, |
|
2363
9046c80a9137
Style fixes and wrong function call fix in soc.views.helper.lists module.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
2358
diff
changeset
|
212 |
{offset_key: max(0, offset-limit),
|
|
9046c80a9137
Style fixes and wrong function call fix in soc.views.helper.lists module.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
2358
diff
changeset
|
213 |
limit_key: limit}) |
|
2313
c39a81bce1bd
Use offset_linkid instead of offset to scan >1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2312
diff
changeset
|
214 |
|
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
215 |
if offset > limit: |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
216 |
# Having a link to the first doesn't make sense on the first page (we're on |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
217 |
# it). It also doesn't make sense on the second page (because the first |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
218 |
# page is the previous page). |
|
2313
c39a81bce1bd
Use offset_linkid instead of offset to scan >1000 entities.
Daniel Bentley <dbentley@google.com>
parents:
2312
diff
changeset
|
219 |
|
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
220 |
# NOTE(dbentley): I personally disagree that it's simpler to do that way, |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
221 |
# because sometimes you want to go to the first page without having to |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
222 |
# consider what page you're on now. |
|
2363
9046c80a9137
Style fixes and wrong function call fix in soc.views.helper.lists module.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
2358
diff
changeset
|
223 |
newest = generateLinkForRequest(request, base_params, {offset_key: 0,
|
|
9046c80a9137
Style fixes and wrong function call fix in soc.views.helper.lists module.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
2358
diff
changeset
|
224 |
limit_key: limit}) |
|
539
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
225 |
|
|
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
226 |
content = {
|
|
2108
286aa6528e84
Properly handle exports when there are multiple lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
2077
diff
changeset
|
227 |
'idx': idx, |
|
553
c0cc20b4afc9
Make redirect generic using the new Lists object
Sverre Rabbelier <srabbelier@gmail.com>
parents:
539
diff
changeset
|
228 |
'data': data, |
|
1807
1f8cde169f32
Add an export link to all list views
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1802
diff
changeset
|
229 |
'export': export_link, |
|
2354
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
230 |
'first': offset+1, |
|
4cc66ab098e8
remove accidental head
Sverre Rabbelier <sverre@rabbelier.nl>
parents:
2316
diff
changeset
|
231 |
'last': len(data) > 1 and offset+len(data) or None, |
|
555
3cdfb42d941b
Split the list view code up in three pieces
Sverre Rabbelier <srabbelier@gmail.com>
parents:
553
diff
changeset
|
232 |
'logic': logic, |
|
553
c0cc20b4afc9
Make redirect generic using the new Lists object
Sverre Rabbelier <srabbelier@gmail.com>
parents:
539
diff
changeset
|
233 |
'limit': limit, |
|
2049
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
234 |
'newest': newest, |
|
553
c0cc20b4afc9
Make redirect generic using the new Lists object
Sverre Rabbelier <srabbelier@gmail.com>
parents:
539
diff
changeset
|
235 |
'next': next, |
|
572
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
236 |
'pagination_form': pagination_form, |
|
2049
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
237 |
'prev': prev, |
|
553
c0cc20b4afc9
Make redirect generic using the new Lists object
Sverre Rabbelier <srabbelier@gmail.com>
parents:
539
diff
changeset
|
238 |
} |
|
539
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
239 |
|
|
555
3cdfb42d941b
Split the list view code up in three pieces
Sverre Rabbelier <srabbelier@gmail.com>
parents:
553
diff
changeset
|
240 |
updates = dicts.rename(params, params['list_params']) |
|
3cdfb42d941b
Split the list view code up in three pieces
Sverre Rabbelier <srabbelier@gmail.com>
parents:
553
diff
changeset
|
241 |
content.update(updates) |
|
3cdfb42d941b
Split the list view code up in three pieces
Sverre Rabbelier <srabbelier@gmail.com>
parents:
553
diff
changeset
|
242 |
|
|
539
e30462354e26
Allow for multiple list objects
Sverre Rabbelier <srabbelier@gmail.com>
parents:
499
diff
changeset
|
243 |
return content |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
244 |
|
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
245 |
|
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
246 |
def makePaginationForm( |
|
572
1b3e7280743a
Made list pagination part of Lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
555
diff
changeset
|
247 |
request, limit, arg_name, choices=DEF_PAGINATION_CHOICES, |
|
632
1f20cfb91e11
Remove unused imports from some modules.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
572
diff
changeset
|
248 |
field_name_fmt=soc.views.helper.forms.DEF_SELECT_QUERY_ARG_FIELD_NAME_FMT): |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
249 |
"""Returns a customized pagination limit selection form. |
|
2049
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
250 |
|
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
251 |
Args: |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
252 |
request: the standard Django HTTP request object |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
253 |
limit: the initial value of the selection control |
|
632
1f20cfb91e11
Remove unused imports from some modules.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
572
diff
changeset
|
254 |
arg_name: see soc.views.helper.forms.makeSelectQueryArgForm(); default is 'limit' |
|
1f20cfb91e11
Remove unused imports from some modules.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
572
diff
changeset
|
255 |
choices: see soc.views.helper.forms.makeSelectQueryArgForm(); default is |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
256 |
DEF_PAGINATION_CHOICES |
|
632
1f20cfb91e11
Remove unused imports from some modules.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
572
diff
changeset
|
257 |
field_name_fmt: see soc.views.helper.forms.makeSelectQueryArgForm() |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
258 |
""" |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
259 |
choices = makeNewPaginationChoices(limit=limit, choices=choices) |
|
2049
d9adbaf1c30f
Some fixes to lists helper to reduce mutation.
Daniel Bentley <dbentley@google.com>
parents:
1821
diff
changeset
|
260 |
|
|
632
1f20cfb91e11
Remove unused imports from some modules.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
572
diff
changeset
|
261 |
return soc.views.helper.forms.makeSelectQueryArgForm( |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
262 |
request, arg_name, limit, choices) |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
263 |
|
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
264 |
|
|
1810
ec0bae3632bb
Adhere to the DEF_... pattern for constants in lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1807
diff
changeset
|
265 |
def makeNewPaginationChoices(limit=DEF_DEFAULT_PAGINATION, |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
266 |
choices=DEF_PAGINATION_CHOICES): |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
267 |
"""Updates the pagination limit selection form. |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
268 |
|
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
269 |
Args: |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
270 |
limit: the initial value of the selection control; |
|
1810
ec0bae3632bb
Adhere to the DEF_... pattern for constants in lists
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1807
diff
changeset
|
271 |
default is DEF_DEFAULT_PAGINATION |
|
632
1f20cfb91e11
Remove unused imports from some modules.
Pawel Solyga <Pawel.Solyga@gmail.com>
parents:
572
diff
changeset
|
272 |
choices: see soc.views.helper.forms.makeSelectQueryArgForm(); |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
273 |
default is DEF_PAGINATION_CHOICES |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
274 |
|
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
275 |
Returns: |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
276 |
a new pagination choices list if limit is not in |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
277 |
DEF_PAGINATION_CHOICES, or DEF_PAGINATION_CHOICES otherwise |
|
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
278 |
""" |
|
1811
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
279 |
|
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
280 |
new_choices = [] |
|
1811
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
281 |
new_choice = (str(limit), '%s items per page' % limit) |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
282 |
|
|
1811
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
283 |
new_choices.append(new_choice) |
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
284 |
new_choices.extend(choices) |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
285 |
|
|
1811
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
286 |
if user_logic.isDeveloper(): |
|
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
287 |
new_choices.extend(DEF_DEVELOPER_CHOICES) |
|
277
85f7d537e4d7
Speed up average case by appending remainder of original choices list unchanged
Todd Larsen <tlarsen@google.com>
parents:
274
diff
changeset
|
288 |
|
|
1811
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
289 |
new_choices = set(new_choices) |
|
265
3c2994f3b85f
List views should have a selectable pagination "page" length:
Todd Larsen <tlarsen@google.com>
parents:
228
diff
changeset
|
290 |
|
|
1811
75d3c1384736
Greatly simplify makeNewPaginationChoices
Sverre Rabbelier <srabbelier@gmail.com>
parents:
1810
diff
changeset
|
291 |
return sorted(new_choices, key=lambda (x, y): int(x)) |