[LUCENE-1245] MultiFieldQueryParser is not friendly for overriding getFieldQuery(String,String,int)

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Not A Problem
• Affects Version/s: 2.3.2
• Component/s: core/queryparser
• Lucene Fields: New, Patch Available
• Assignee: Unassigned
• Reporter: Trejkaz
• Created: 26/Mar/08 00:14
• Updated: 28/Aug/22 11:48
• Resolved: 02/Mar/13 06:51

Description

LUCENE-1213 fixed an issue in MultiFieldQueryParser where the slop parameter wasn't being properly applied. Problem is, the fix which eventually got committed is calling super.getFieldQuery(String,String), bypassing any possibility of customising the query behaviour.

This should be relatively simply fixable by modifying getFieldQuery(String,String,int) to, if field is null, recursively call getFieldQuery(String,String,int) instead of setting the slop itself. This gives subclasses which override either getFieldQuery method a chance to do something different.