tree a56c3b7638ac2cf346378f4e0fc36284e3f81c8b
parent 1983ac7a8cf43416c76de26bdffde97ae8a7ef34
author Edwin Kempin <ekempin@google.com> 1673940854 +0100
committer Edwin Kempin <ekempin@google.com> 1673943084 +0000

Reland "Fix error when using the message operator with regex on ..."

This relands change I99e2b8a3d that was reverted by change I4e551ac53.

The problem with the original change was that it no longer used the
checkFieldAvailable method to throw the QueryParseException. This method
is overridden in SubmitRequirementChangeQueryBuilder to suppress field
checks for operators that are used in submit requirement expressions.
Since submit requirement expressions are evaluated in memory the version
of the index and the available fields do not matter for them.

Fix this by changing the checkFieldAvailable method so that it supports
an arbitrary error message. This way we can also use it to throw the
QueryParseException with the improved error message. Checking for an
unsupported operator is done by the checkOperatorAvailable method now
that delegates to the checkFieldAvailable method. This way all
QueryParseExceptions are thrown from the checkFieldAvailable method and
overriding it in SubmitRequirementChangeQueryBuilder to suppress the
exceptions for submit requirement expressions works in all cases.

Change-Id: I6c73ecadfa0bfaaa70aaf7c2050fc1df555760f7
Bug: Google b/264478714
Release-Notes: skip