Clarify and add a TODO in the controller

This clarifies and adds a TODO regarding the filters and fields
API parameters.

Change-Id: I01cfd45b99dcfca4e320900caebdc68f6562ddd8

diff --git a/neutron/pecan_wsgi/controllers/root.py b/neutron/pecan_wsgi/controllers/root.py
index 396778bac880923cee39fac91b11724b6130348d..41a249932c1f48cf08affe301f2cfc6d0aa3bfb1 100644 (file)
--- a/neutron/pecan_wsgi/controllers/root.py
+++ b/neutron/pecan_wsgi/controllers/root.py
@@ -128,10 +128,12 @@ class CollectionsController(object):
 
     def get(self, *args, **kwargs):
         # list request
-        # TODO(kevinbenton): allow fields after policy enforced fields present
+        # TODO(kevinbenton): use user-provided fields in call to plugin
+        # after making sure policy enforced fields remain
         kwargs.pop('fields', None)
         _listify = lambda x: x if isinstance(x, list) else [x]
         filters = {k: _listify(v) for k, v in kwargs.items()}
+        # TODO(kevinbenton): convert these using api_common.get_filters
         lister = getattr(request.plugin, 'get_%s' % self.collection)
         return {self.collection: lister(request.context, filters=filters)}