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;
+    }
 }