Add isEmpty() function to ConditionGroupInterface

Reviews welcome!

drunken monkey → made their first commit to this issue’s fork.

Thanks a lot for creating the issue and MR.
Your code already looks pretty good, just some code style issues. However, if we have that method, we should aim to cover all cases, so it would be great to also return FALSE if a condition group contains nothing but other empty condition groups. In that case, no condition would be added to the query, but $this->conditions is non-empty. I amended that, explained it in the doc comment and also added a bit more of testing.

Please review and I’ll merge. Thanks again, in any case!

That makes sense to me! Thanks @drunken monkey!

Great to hear, thanks for reporting back!
Merged. Thanks again!

Automatically closed - issue fixed for 2 weeks with no activity.