delete(): filter() behavior changed from python 2 to 3

Author: Cosimo Streppone

pysolr.py

@@ -926,7 +926,7 @@ def delete(self, id=None, q=None, commit=True, softCommit=False, waitFlush=None,
             if not isinstance(id, (list, set, tuple)):
                 id = [id]
             else:
-                id = filter(None, id)
+                id = list(filter(None, id))
             if id:
                 m = '<delete>%s</delete>' % ''.join('<id>%s</id>' % i for i in id)
             else:

tests/test_client.py

@@ -661,7 +661,7 @@ def test_delete(self):
         def leaf_doc(doc):
             return 'price' in doc and NESTED_DOC_KEY not in doc
 
-        to_delete_docs = filter(leaf_doc, self.docs)
+        to_delete_docs = list(filter(leaf_doc, self.docs))
         to_delete_ids = [doc['id'] for doc in to_delete_docs]
 
         self.solr.add(to_delete_docs)