From 9f348cfa16d857f665160457fc68af2b8f0a8ecc Mon Sep 17 00:00:00 2001 From: Hans Joachim Desserud Date: Sat, 10 Jan 2015 19:36:29 +0100 Subject: [PATCH] The Version-less constructor for StopFilter will simply default to Version.LATEST under the hood which is exactly what we send in. Former-commit-id: cc3010532e9203d663d977f0df0892d8f5694b5f --- .../dependencycheck/data/cpe/CpeMemoryIndex.java | 8 ++++---- .../dependencycheck/data/lucene/FieldAnalyzer.java | 13 ++----------- .../data/lucene/SearchFieldAnalyzer.java | 12 ++---------- .../data/lucene/FieldAnalyzerTest.java | 6 +++--- 4 files changed, 11 insertions(+), 28 deletions(-) diff --git a/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/cpe/CpeMemoryIndex.java b/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/cpe/CpeMemoryIndex.java index edd7da7c0..e3fc75bae 100644 --- a/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/cpe/CpeMemoryIndex.java +++ b/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/cpe/CpeMemoryIndex.java @@ -153,7 +153,7 @@ public final class CpeMemoryIndex { private Analyzer createIndexingAnalyzer() { final Map fieldAnalyzers = new HashMap(); fieldAnalyzers.put(Fields.DOCUMENT_KEY, new KeywordAnalyzer()); - return new PerFieldAnalyzerWrapper(new FieldAnalyzer(LuceneUtils.CURRENT_VERSION), fieldAnalyzers); + return new PerFieldAnalyzerWrapper(new FieldAnalyzer(), fieldAnalyzers); } /** @@ -165,12 +165,12 @@ public final class CpeMemoryIndex { private Analyzer createSearchingAnalyzer() { final Map fieldAnalyzers = new HashMap(); fieldAnalyzers.put(Fields.DOCUMENT_KEY, new KeywordAnalyzer()); - productSearchFieldAnalyzer = new SearchFieldAnalyzer(LuceneUtils.CURRENT_VERSION); - vendorSearchFieldAnalyzer = new SearchFieldAnalyzer(LuceneUtils.CURRENT_VERSION); + productSearchFieldAnalyzer = new SearchFieldAnalyzer(); + vendorSearchFieldAnalyzer = new SearchFieldAnalyzer(); fieldAnalyzers.put(Fields.PRODUCT, productSearchFieldAnalyzer); fieldAnalyzers.put(Fields.VENDOR, vendorSearchFieldAnalyzer); - return new PerFieldAnalyzerWrapper(new FieldAnalyzer(LuceneUtils.CURRENT_VERSION), fieldAnalyzers); + return new PerFieldAnalyzerWrapper(new FieldAnalyzer(), fieldAnalyzers); } /** diff --git a/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/lucene/FieldAnalyzer.java b/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/lucene/FieldAnalyzer.java index 8eb1c1b35..797733edb 100644 --- a/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/lucene/FieldAnalyzer.java +++ b/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/lucene/FieldAnalyzer.java @@ -25,7 +25,6 @@ import org.apache.lucene.analysis.core.LowerCaseFilter; import org.apache.lucene.analysis.core.StopAnalyzer; import org.apache.lucene.analysis.core.StopFilter; import org.apache.lucene.analysis.miscellaneous.WordDelimiterFilter; -import org.apache.lucene.util.Version; /** *

@@ -36,19 +35,11 @@ import org.apache.lucene.util.Version; */ public class FieldAnalyzer extends Analyzer { - /** - * The Lucene Version used. - */ - private final Version version; - /** * Creates a new FieldAnalyzer. * - * @param version the Lucene version */ - public FieldAnalyzer(Version version) { - this.version = version; - } + public FieldAnalyzer() { } /** * Creates the TokenStreamComponents @@ -73,7 +64,7 @@ public class FieldAnalyzer extends Analyzer { | WordDelimiterFilter.STEM_ENGLISH_POSSESSIVE, null); stream = new LowerCaseFilter(stream); - stream = new StopFilter(version, stream, StopAnalyzer.ENGLISH_STOP_WORDS_SET); + stream = new StopFilter(stream, StopAnalyzer.ENGLISH_STOP_WORDS_SET); return new TokenStreamComponents(source, stream); } diff --git a/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/lucene/SearchFieldAnalyzer.java b/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/lucene/SearchFieldAnalyzer.java index b5b8928d1..72880fedf 100644 --- a/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/lucene/SearchFieldAnalyzer.java +++ b/dependency-check-core/src/main/java/org/owasp/dependencycheck/data/lucene/SearchFieldAnalyzer.java @@ -25,7 +25,6 @@ import org.apache.lucene.analysis.core.LowerCaseFilter; import org.apache.lucene.analysis.core.StopAnalyzer; import org.apache.lucene.analysis.core.StopFilter; import org.apache.lucene.analysis.miscellaneous.WordDelimiterFilter; -import org.apache.lucene.util.Version; /** * A Lucene field analyzer used to analyzer queries against the CPE data. @@ -34,10 +33,6 @@ import org.apache.lucene.util.Version; */ public class SearchFieldAnalyzer extends Analyzer { - /** - * The Lucene Version used. - */ - private final Version version; /** * A local reference to the TokenPairConcatenatingFilter so that we can clear any left over state if this analyzer * is re-used. @@ -47,11 +42,8 @@ public class SearchFieldAnalyzer extends Analyzer { /** * Constructs a new SearchFieldAnalyzer. * - * @param version the Lucene version */ - public SearchFieldAnalyzer(Version version) { - this.version = version; - } + public SearchFieldAnalyzer() { } /** * Creates a the TokenStreamComponents used to analyze the stream. @@ -78,7 +70,7 @@ public class SearchFieldAnalyzer extends Analyzer { stream = new UrlTokenizingFilter(stream); concatenatingFilter = new TokenPairConcatenatingFilter(stream); stream = concatenatingFilter; - stream = new StopFilter(version, stream, StopAnalyzer.ENGLISH_STOP_WORDS_SET); + stream = new StopFilter(stream, StopAnalyzer.ENGLISH_STOP_WORDS_SET); return new TokenStreamComponents(source, stream); } diff --git a/dependency-check-core/src/test/java/org/owasp/dependencycheck/data/lucene/FieldAnalyzerTest.java b/dependency-check-core/src/test/java/org/owasp/dependencycheck/data/lucene/FieldAnalyzerTest.java index def2af8b6..99fb652e3 100644 --- a/dependency-check-core/src/test/java/org/owasp/dependencycheck/data/lucene/FieldAnalyzerTest.java +++ b/dependency-check-core/src/test/java/org/owasp/dependencycheck/data/lucene/FieldAnalyzerTest.java @@ -69,7 +69,7 @@ public class FieldAnalyzerTest { @Test public void testAnalyzers() throws Exception { - Analyzer analyzer = new FieldAnalyzer(LuceneUtils.CURRENT_VERSION); + Analyzer analyzer = new FieldAnalyzer(); Directory index = new RAMDirectory(); String field1 = "product"; @@ -83,8 +83,8 @@ public class FieldAnalyzerTest { //Analyzer searchingAnalyzer = new SearchFieldAnalyzer(LuceneUtils.CURRENT_VERSION); String querystr = "product:\"(Spring Framework Core)\" vendor:(SpringSource)"; - SearchFieldAnalyzer searchAnalyzerProduct = new SearchFieldAnalyzer(LuceneUtils.CURRENT_VERSION); - SearchFieldAnalyzer searchAnalyzerVendor = new SearchFieldAnalyzer(LuceneUtils.CURRENT_VERSION); + SearchFieldAnalyzer searchAnalyzerProduct = new SearchFieldAnalyzer(); + SearchFieldAnalyzer searchAnalyzerVendor = new SearchFieldAnalyzer(); HashMap map = new HashMap(); map.put(field1, searchAnalyzerProduct); map.put(field2, searchAnalyzerVendor);