Search api DB does not support nested conditionGroups

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

Thanks a lot for posting this issue and the MR fixing it!

Seems very sensible to me. The fact that tags on nested condition groups are ignored is clearly an oversight.
However, your code would still ignore tags nested more than one level deep (even though there is currently no use case for that). Instead, I suggest adding a new helper method and just calling it recursively – I think that would also make the code a bit cleaner.
I’m also adding a regression test.

Updated the MR, please test/review.

(Test failure is due to 🐛 Fix test failures against Drupal 11 Active .)

Tested and fixes the issue.
Recursion is of course the way to go

Thanks for looking into this so quickly Thomas!
This will make a new release for Facets 3.x go much smoother ;)

Good to hear, thanks for reporting back so quickly!
Merged. Thanks again!

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