Convert wrap in parenthesis (grouping) to an operator?

[jgornick]

I've created an example at https://stackblitz.com/edit/odata-parser-s38vfw

Notice how each of the operators are being wrapped in parenthesis. What I was expecting is a query that looks like:

foo eq 'bar' and quux eq 'baz' and name ne 'bax'

Parenthesis are a "grouping operator" according to the OData specification. Should they be treated as such in the filter builder?

For example, update the filter builder library to specify where they want to group. Another reason to allow grouping as an operator is some implementations of the OData specification don't support the grouping operator.

Right now the filter builder is producing OData strings implying that all implementations support the grouping operator.

Thoughts?

[bodia-uz]

You are right, the parenthesis is not required in most of the cases.
It was added because its easier to add them, then check if the rule is safe.

We need to get rid of them and add them only when required, but unfortunately, I don't have a time for that now:|

[jgornick]

I am looking to dedicate time soon to close off some loose ends on open source projects. This project is at the top of my list 😄

Thanks again!

[bodia-uz]

Cool. It would be great.