fix code analyses warnings/errors

plone · Feb 12, 2019 · dbc9e9e · dbc9e9e · jenkins-plone-org · Feb 12, 2019
1 parent 5718599
commit dbc9e9e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/src/plone/api/content.py b/src/plone/api/content.py
@@ -675,14 +675,14 @@ def find(context=None, depth=None, **kwargs):
         operator = 'or'
         ifaces = object_provides
         if isinstance(object_provides, dict):
-             operator = object_provides.get('operator', operator)
-             ifaces = object_provides.get('query', [])
+            operator = object_provides.get('operator', operator)
+            ifaces = object_provides.get('query', [])
         elif not isinstance(object_provides, (list, tuple)):
             ifaces = [object_provides]
 
         query['object_provides'] = {
             'query': [getattr(x, '__identifier__', x) for x in ifaces],
-            'operator': operator
+            'operator': operator,
         }
 
     # Make sure we don't dump the whole catalog.

diff --git a/src/plone/api/tests/test_content.py b/src/plone/api/tests/test_content.py
@@ -1034,17 +1034,17 @@ def test_find_interface_dict(self):
                 'query': [
                     IContentish.__identifier__,
                     INavigationRoot.__identifier__],
-                'operator': 'and'
-            }
+                'operator': 'and',
+            },
         )
         self.assertEqual(len(brains), 1)
 
         # plone.api query using interfaces
         brains = api.content.find(
             object_provides={
                 'query': [IContentish, INavigationRoot],
-                'operator': 'and'
-            }
+                'operator': 'and',
+            },
         )
         self.assertEqual(len(brains), 1)