[
https://issues.apache.org/jira/browse/TIKA-2439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16110746#comment-16110746
]
ASF GitHub Bot commented on TIKA-2439:
--------------------------------------
tballison closed pull request #199: TIKA-2439 throwing IllegalStateException in
OptimaizeLangDetector.detectAll
URL: https://github.com/apache/tika/pull/199
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> Avoid NullPointerException in
> org.apache.tika.langdetect.OptimaizeLangDetector if models haven't been loaded
> ------------------------------------------------------------------------------------------------------------
>
> Key: TIKA-2439
> URL: https://issues.apache.org/jira/browse/TIKA-2439
> Project: Tika
> Issue Type: Improvement
> Components: languageidentifier
> Affects Versions: 1.17
> Reporter: Karl Richter
>
> In an intuitive usage of
> {code:java}
> LanguageDetector languageDetector =
> LanguageDetector.getDefaultLanguageDetector();
> List<LanguageResult> languageResults =
> languageDetector.detectAll(someNonEmptyString);
> {code}
> `org.apache.tika.langdetect.OptimaizeLangDetector` might/will (?) be chosen
> as default detector, but since there's no call to
> `LanguageDetector.loadModels()` `LanguageDetector.detector` is `null` in
> `detectAll` which causes an unhelpful error situation which one has to
> investigate in the code. A simple check whether `detector` is `null` and
> throwing an `IllegalStateException("models haven't been loaded yet (forgot to
> call loadModels?")` would be much more intuitive.
> If that corresponds to the expected behaviour (it's my first week with Tika),
> I can provide a patch or pull request.
> experienced with 1.16-75-g4455a6f08
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
