Adds brackets around expressions when several are used

Author: developer992

pyodata/v2/service.py

@@ -1268,8 +1268,15 @@ def _decode_expression(self, expr, val):
 
     # pylint: disable=no-self-use
     def _combine_expressions(self, expressions):
-        return ' and '.join(expressions)
+        if len(expressions) > 1:
+            combined = ""
+            for counter, expr in enumerate(expressions):
+                combined += f"{' and ' if counter > 0 else ''}({expr})"
 
+            return combined
+
+        return expressions[0]
+    
     # pylint: disable=too-many-return-statements, too-many-branches
     def _build_expression(self, field_name, operator, value):
         target_field = self.proprty_obj(field_name)

tests/test_service_v2.py

@@ -2553,6 +2553,26 @@ def test_count_with_chainable_filter_multiple(service):
     assert request.execute() == 3
 
 
+@responses.activate
+def test_chaining_fix(service):
+    from pyodata.v2.service import FilterExpression as Q
+
+    url = f"{service.url}/Employees?%24filter=startswith%28NameFirst%2C+%27Steve%27%29+eq+true+and+ID+eq+23+or+ID+eq+25+or+ID+eq+28"
+    
+    responses.add(
+        responses.GET,
+        url,
+        json=3,
+        status=200)
+    
+    employees = service.entity_sets.Employees.get_entities()
+    request = employees.filter(Q(NameFirst__startswith="Steve"), Q(ID__in=[23, 25, 28]))
+
+    assert isinstance(request, pyodata.v2.service.GetEntitySetRequest)
+
+    assert request.execute() == 3
+
+
 @responses.activate
 def test_count_with_chainable_filter_or(service):
     """Check getting $count with $filter with FilterExpression syntax or"""