[ https://issues.apache.org/jira/browse/TIKA-2439?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16110844#comment-16110844 ]
Hudson commented on TIKA-2439: ------------------------------ SUCCESS: Integrated in Jenkins build Tika-trunk #1341 (See [https://builds.apache.org/job/Tika-trunk/1341/]) TIKA-2439 throwing IllegalStateException in (krichter722: [https://github.com/apache/tika/commit/57ee179862d6f1a15cea6745ca3d1c8e8e7ff800]) * (edit) tika-langdetect/src/main/java/org/apache/tika/langdetect/OptimaizeLangDetector.java > 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 > Fix For: 1.17 > > > 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)