Replace filter() calls with list comprehensions

In Python 3, filter() returns an iterator.
Use list comprehensions instead.

Reviewed-By: Christian Heimes <cheimes@redhat.com>
Reviewed-By: Jan Cholasta <jcholast@redhat.com>

ipalib/frontend.py

@@ -1128,7 +1128,7 @@ class Object(HasParam):
             self.__get_attrs('Method'), sort=False, name_attr='attr_name'
         )
         self._create_param_namespace('params')
-        pkeys = filter(lambda p: p.primary_key, self.params())
+        pkeys = [p for p in self.params() if p.primary_key]
         if len(pkeys) > 1:
             raise ValueError(
                 '%s (Object) has multiple primary keys: %s' % (
@@ -1139,7 +1139,7 @@ class Object(HasParam):
         if len(pkeys) == 1:
             self.primary_key = pkeys[0]
             self.params_minus_pk = NameSpace(
-                filter(lambda p: not p.primary_key, self.params()), sort=False
+                [p for p in self.params() if not p.primary_key], sort=False
             )
         else:
             self.primary_key = None

ipalib/parameters.py

@@ -792,9 +792,8 @@ class Param(ReadOnly):
             if type(value) not in (tuple, list):
                 value = (value,)
             values = tuple(
-                self._convert_scalar(v, i) for (i, v) in filter(
-                    lambda iv: not _is_null(iv[1]), enumerate(value)
-                )
+                self._convert_scalar(v, i)
+                for (i, v) in enumerate(value) if not _is_null(v)
             )
             if len(values) == 0:
                 return

ipaplatform/setup.py.in

@@ -63,7 +63,7 @@ def setup_package():
             description = DOCLINES[0],
             long_description = "\n".join(DOCLINES[2:]),
             download_url = "http://www.freeipa.org/page/Downloads",
-            classifiers=filter(None, CLASSIFIERS.split('\n')),
+            classifiers=[line for line in CLASSIFIERS.split('\n') if line],
             package_dir = {'ipaplatform': ''},
             packages = ["ipaplatform",
                         "ipaplatform.base",

ipapython/setup.py.in

@@ -62,7 +62,7 @@ def setup_package():
             description = DOCLINES[0],
             long_description = "\n".join(DOCLINES[2:]),
             download_url = "http://www.freeipa.org/page/Downloads",
-            classifiers=filter(None, CLASSIFIERS.split('\n')),
+            classifiers=[line for line in CLASSIFIERS.split('\n') if line],
             platforms = ["Linux", "Solaris", "Unix"],
             package_dir = {'ipapython': ''},
             packages = ["ipapython",

ipaserver/install/adtrustinstance.py

@@ -327,7 +327,7 @@ class ADTRUSTInstance(service.Service):
                                   r.single_value.get('ipaBaseRID'),
                                   r.single_value.get('ipaSecondaryBaseRID')))
 
-            ranges_with_no_rid_base = filter(no_rid_base_set, ranges)
+            ranges_with_no_rid_base = [r for r in ranges if no_rid_base_set(r)]
 
             # Return if no range is without RID base
             if len(ranges_with_no_rid_base) == 0:

ipaserver/install/ipa_otptoken_import.py

@@ -403,7 +403,7 @@ class PSKCKeyPackage(object):
 
     def __dates(self, out, data, key, reducer):
         dates = (data.get(key + '.sw', None), data.get(key + '.hw', None))
-        dates = filter(lambda x: x is not None, dates)
+        dates = [x for x in dates if x is not None]
         if dates:
             out['ipatoken' + key] = unicode(reducer(dates).strftime("%Y%m%d%H%M%SZ"))
 

ipatests/setup.py.in

@@ -63,7 +63,7 @@ def setup_package():
             description = DOCLINES[0],
             long_description = "\n".join(DOCLINES[2:]),
             download_url = "http://www.freeipa.org/page/Downloads",
-            classifiers=filter(None, CLASSIFIERS.split('\n')),
+            classifiers=[line for line in CLASSIFIERS.split('\n') if line],
             package_dir = {'ipatests': ''},
             packages = ["ipatests",
                         "ipatests.pytest_plugins",

ipatests/test_integration/config.py

@@ -69,7 +69,7 @@ class Config(pytest_multihost.config.Config):
 
     @property
     def ad_domains(self):
-        return filter(lambda d: d.type == 'AD', self.domains)
+        return [d for d in self.domains if d.type == 'AD']
 
     def get_all_hosts(self):
         for domain in self.domains: