CCL is unable to handle escaped parenthesis about concourse HOT 4 CLOSED

[default/cash]$ find("location = 'Atlanta \\(HQ\\)'")
Returned '[]' in 0.007 sec

If the phrase with the parens is quoted, then it parses, but the wrong result is returned because the CCL Parser alters all parens to have a space before and after

public static Queue<PostfixNotationSymbol> toPostfixNotation(String ccl) {
        // This method uses a value buffer to correct cases when a string value
        // is specified without quotes (because its a common mistake to make).
        // If an operator other than BETWEEN is specified, we use logic that
        // will buffer all the subsequent tokens until we reach a (parenthesis),
        // (conjunction) or (at) and assume that the tokens belong to the same
        // value.
        ccl = ccl.replace("(", "( ");
        ccl = ccl.replace(")", " )");

from concourse.

The reason that the parens are padded with spaces is so that they are split and handled as separate tokens. We could add an option to the StringSplitter to split on and return parens...

from concourse.

...or we could fix the regex to only do the replace on parens that

• are not escaped (so not preceeded by '')
• and are proceeded by whatever characters are allowed to be in a key ([A-Za-z0-9_])

from concourse.

Actually, if we add this logic to StringSplitter we won't need to deal with escaping the parens. The QuoteAware string splitter won't split or apply any of the options within quotes

from concourse.