elasticsearch-plugin-bundle icon indicating copy to clipboard operation
elasticsearch-plugin-bundle copied to clipboard

#47 Can't set languages for fields of type langdetect when profile is shorttext

Open ptanov opened this issue 3 years ago • 0 comments

In this fix we first try to get the value of languages list (if available) before using the default value for languages_short_text. We don't need initialization of languages in DEFAULT_SETTINGS because we provide a default value when getting the languages (https://github.com/jprante/elasticsearch-plugin-bundle/blob/e75891afbc280de6e5b924c57fd029642b268777/src/main/java/org/xbib/elasticsearch/plugin/bundle/common/langdetect/LangdetectService.java#L174). By doing that we ensure that DEFAULT_LANGUAGES_SHORT_TEXT will be used if no configuration is provided (default constructor). The only drawback of this approach is if someone provide settings that contains languages but not contains languages_short_text and actually want to use DEFAULT_LANGUAGES_SHORT_TEXT. This case could be avoided if the default constructor is used in the situation.

However test case needs to be created...

ptanov avatar Feb 18 '22 14:02 ptanov