diff options
author | Ishita Mandhan <imandha@us.ibm.com> | 2015-10-14 16:22:54 -0700 |
---|---|---|
committer | Ishita Mandhan <imandha@us.ibm.com> | 2015-10-16 08:11:30 -0700 |
commit | 0746e70cb5e367d89980f4e1166458afed194fe1 (patch) | |
tree | 51e14f0b881c03ef4dcea441d70c5c4a637049bb /trove/configuration | |
parent | 87753afeb2ec8a136ddb626fdd14f3e5c6d4c623 (diff) | |
download | trove-0746e70cb5e367d89980f4e1166458afed194fe1.tar.gz |
Pagination limit code refactor
The logic to find the pagination limit is
repeated several times and is not in one
function. This patch creates a function that
is called in various locations replacing
repetitive code.
Change-Id: I3eced6a4236e4189e57ea5a1589e8f0d411cc4e3
Closes-Bug: #1308285
Diffstat (limited to 'trove/configuration')
-rw-r--r-- | trove/configuration/models.py | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/trove/configuration/models.py b/trove/configuration/models.py index 5e1f9178..3d6c081e 100644 --- a/trove/configuration/models.py +++ b/trove/configuration/models.py @@ -54,10 +54,8 @@ class Configurations(object): LOG.debug("No configurations found for tenant %s" % context.tenant) - limit = int(context.limit or Configurations.DEFAULT_LIMIT) - if limit > Configurations.DEFAULT_LIMIT: - limit = Configurations.DEFAULT_LIMIT - + limit = utils.pagination_limit(context.limit, + Configurations.DEFAULT_LIMIT) data_view = DBConfiguration.find_by_pagination('configurations', db_info, "foo", |