Remove redundant limit param check in API

limit param has been checked in get_pagination_params
method before calling _get_all method

Change-Id: I987f88981bcca7219142aeed4a8c730832f00348

5 files changed, 0 insertions, 50 deletions

diff --git a/karbor/api/v1/operation_logs.py b/karbor/api/v1/operation_logs.py
index adf5e7c..670fb5b 100644
--- a/karbor/api/v1/operation_logs.py
+++ b/karbor/api/v1/operation_logs.py
@@ -169,16 +169,6 @@ class OperationLogsController(wsgi.Controller):
 
         all_tenants = utils.get_bool_param('all_tenants', filters)
 
-        try:
-            if limit is not None:
-                limit = int(limit)
-                if limit < 0:
-                    msg = _('limit param must be positive')
-                    raise exception.InvalidInput(reason=msg)
-        except ValueError:
-            msg = _('limit param must be an integer')
-            raise exception.InvalidInput(reason=msg)
-
         if filters:
             LOG.debug("Searching by: %s.", six.text_type(filters))
 
diff --git a/karbor/api/v1/plans.py b/karbor/api/v1/plans.py
index f145d1d..8f0d61c 100644
--- a/karbor/api/v1/plans.py
+++ b/karbor/api/v1/plans.py
@@ -212,16 +212,6 @@ class PlansController(wsgi.Controller):
 
         all_tenants = utils.get_bool_param('all_tenants', filters)
 
-        try:
-            if limit is not None:
-                limit = int(limit)
-                if limit < 0:
-                    msg = _('limit param must be positive')
-                    raise exception.InvalidInput(reason=msg)
-        except ValueError:
-            msg = _('limit param must be an integer')
-            raise exception.InvalidInput(reason=msg)
-
         if filters:
             LOG.debug("Searching by: %s.", six.text_type(filters))
 
diff --git a/karbor/api/v1/providers.py b/karbor/api/v1/providers.py
index 095ffa5..96854d1 100644
--- a/karbor/api/v1/providers.py
+++ b/karbor/api/v1/providers.py
@@ -228,16 +228,6 @@ class ProvidersController(wsgi.Controller):
         if filters is None:
             filters = {}
 
-        try:
-            if limit is not None:
-                limit = int(limit)
-                if limit <= 0:
-                    msg = _('limit param must be positive')
-                    raise exception.InvalidInput(reason=msg)
-        except ValueError:
-            msg = _('limit param must be an integer')
-            raise exception.InvalidInput(reason=msg)
-
         if filters:
             LOG.debug("Searching by: %s.", six.text_type(filters))
 
diff --git a/karbor/api/v1/restores.py b/karbor/api/v1/restores.py
index 3dea082..08f91ce 100644
--- a/karbor/api/v1/restores.py
+++ b/karbor/api/v1/restores.py
@@ -168,16 +168,6 @@ class RestoresController(wsgi.Controller):
 
         all_tenants = utils.get_bool_param('all_tenants', filters)
 
-        try:
-            if limit is not None:
-                limit = int(limit)
-                if limit <= 0:
-                    msg = _('limit param must be positive')
-                    raise exception.InvalidInput(reason=msg)
-        except ValueError:
-            msg = _('limit param must be an integer')
-            raise exception.InvalidInput(reason=msg)
-
         if filters:
             LOG.debug("Searching by: %s.", six.text_type(filters))
 
diff --git a/karbor/api/v1/verifications.py b/karbor/api/v1/verifications.py
index b128911..bfe0677 100644
--- a/karbor/api/v1/verifications.py
+++ b/karbor/api/v1/verifications.py
@@ -166,16 +166,6 @@ class VerificationsController(wsgi.Controller):
 
         all_tenants = utils.get_bool_param('all_tenants', filters)
 
-        try:
-            if limit is not None:
-                limit = int(limit)
-                if limit <= 0:
-                    msg = _('limit param must be positive')
-                    raise exception.InvalidInput(reason=msg)
-        except ValueError:
-            msg = _('limit param must be an integer')
-            raise exception.InvalidInput(reason=msg)
-
         if filters:
             LOG.debug("Searching by: %s.", six.text_type(filters))