tree 18bc236714badc6fd8c2d298882c23c2c7921279
parent ba868a93d4e9685e0b93025a0a6e342a0f132d74
author Luca Milanesio <luca.milanesio@gmail.com> 1718285956 +0100
committer Luca Milanesio <luca.milanesio@gmail.com> 1718291705 +0000

Fix permission backend extra query with parens

During the review the If144832047b4 the way the extra
query added by the permission backend was overlooked and,
in conjunction of a an empty query in the integration test
allowed the following ambiguity:

Query without permissions backend limitations
- author:self OR status:open => returns all open changes
  or authored by the current user

Query with a permissions backend hiding a testHiddenProject
- author:self OR status:open -project:testHiddenProject

The above query is not doing what the permission backend
would expect, because is evaluated as follows:
- author:self OR (status:open AND -project:testHiddenProject)

The above query would return all changes authored by the
current user, including the ones of testHiddenProject.

Release-Notes: Fix evaluation of extra query provided by a permission backend
Change-Id: I94dea44d5c5f00a9a90c29136cf265d4759b2165