Permalink
Browse files

review comments from dmitriy

  • Loading branch information...
1 parent bfca836 commit 614793993c458180c2f7b132f3514175a40fef72 @dizzzz dizzzz committed Jan 23, 2014
Showing with 3 additions and 2 deletions.
  1. +3 −2 extensions/indexes/lucene/src/org/exist/indexing/lucene/AnalyzerConfig.java
View
5 extensions/indexes/lucene/src/org/exist/indexing/lucene/AnalyzerConfig.java
@@ -152,13 +152,13 @@ protected static Analyzer configureAnalyzer(Element config) throws DatabaseConfi
} catch (ClassNotFoundException e) {
LOG.error(String.format("Lucene index: analyzer class %s not found. (%s)", className, e.getMessage()));
- return newAnalyzer;
+ return null;
}
// CHeck if class is an Analyzer
if (!Analyzer.class.isAssignableFrom(clazz)) {
LOG.error(String.format("Lucene index: analyzer class has to be a subclass of %s", Analyzer.class.getName()));
- return newAnalyzer;
+ return null;
}
// Get list of parameters
@@ -240,6 +240,7 @@ private static Analyzer createInstance(Class<?> clazz, Class<?>[] vcParamClasses
if(LOG.isDebugEnabled()){
LOG.debug(String.format("Using analyzer %s", className));
}
+
return (Analyzer) cstr.newInstance(vcParamValues);
} catch (IllegalArgumentException e) {

0 comments on commit 6147939

Please sign in to comment.