[SOLR-3453] edismax lowercaseOperators=false broken by SOLR-3026

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.6
• Fix Version/s: 4.0-ALPHA, 6.0
• Component/s: search
• Assignee: Unassigned
• Reporter: Michael Ryan
• Created: 14/May/12 04:50
• Updated: 09/May/16 18:46
• Resolved: 20/Jun/12 18:15

Description

The edismax lowercaseOperators=false option seems to have been broken by SOLR-3026. "foo and bar" and "foo or bar" are treated as "foo AND bar" and "foo OR bar", respectively, even when lowercaseOperators=false.

Fix is rather simple, I think (though I haven't tested this). Current code:

        if (i>0 && i+1<clauses.size()) {
          if ("AND".equalsIgnoreCase(s)) {
            s="AND";
          } else if ("OR".equalsIgnoreCase(s)) {
            s="OR";
          }
        }

Proposed code:

        if (lowercaseOperators) {
          if (i>0 && i+1<clauses.size()) {
            if ("AND".equalsIgnoreCase(s)) {
              s="AND";
            } else if ("OR".equalsIgnoreCase(s)) {
              s="OR";
            }
          }
        }

Also interesting is the treatment of "Or" and "oR", but I'll leave that as an exercise to the reader.

Issue Links

• relates to: SOLR-3026 eDismax: Locking down which fields can be explicitly queried (user fields aka uf)