You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tika.apache.org by ni...@apache.org on 2015/04/29 21:01:22 UTC
svn commit: r1676817 -
/tika/trunk/tika-core/src/main/java/org/apache/tika/parser/DefaultParser.java
Author: nick
Date: Wed Apr 29 19:01:21 2015
New Revision: 1676817
URL: http://svn.apache.org/r1676817
Log:
Spotted when looking at TIKA-1617 - DefaultParser should override getAllComponentParsers to mirror getParsers behaviour when a dynamic service loader exists
Modified:
tika/trunk/tika-core/src/main/java/org/apache/tika/parser/DefaultParser.java
Modified: tika/trunk/tika-core/src/main/java/org/apache/tika/parser/DefaultParser.java
URL: http://svn.apache.org/viewvc/tika/trunk/tika-core/src/main/java/org/apache/tika/parser/DefaultParser.java?rev=1676817&r1=1676816&r2=1676817&view=diff
==============================================================================
--- tika/trunk/tika-core/src/main/java/org/apache/tika/parser/DefaultParser.java (original)
+++ tika/trunk/tika-core/src/main/java/org/apache/tika/parser/DefaultParser.java Wed Apr 29 19:01:21 2015
@@ -16,6 +16,7 @@
*/
package org.apache.tika.parser;
+import java.util.ArrayList;
import java.util.Collection;
import java.util.Collections;
import java.util.Comparator;
@@ -116,4 +117,13 @@ public class DefaultParser extends Compo
return map;
}
+ @Override
+ public List<Parser> getAllComponentParsers() {
+ List<Parser> parsers = super.getAllComponentParsers();
+ if (loader != null) {
+ parsers = new ArrayList<Parser>(parsers);
+ parsers.addAll(loader.loadDynamicServiceProviders(Parser.class));
+ }
+ return parsers;
+ }
}