Check the main dict id to be able to fallback.
Bug: 5095140 Change-Id: I02032923ca2a65bd8fbabc0abbe6a476f7542187
This commit is contained in:
parent
89fdb82753
commit
ee7daefd97
1 changed files with 20 additions and 7 deletions
|
@ -213,6 +213,13 @@ class BinaryDictionaryGetter {
|
|||
return cacheFiles;
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns the id of the main dict for a specified locale.
|
||||
*/
|
||||
private static String getMainDictId(final Locale locale) {
|
||||
return locale.toString();
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns a list of file addresses for a given locale, trying relevant methods in order.
|
||||
*
|
||||
|
@ -234,12 +241,18 @@ class BinaryDictionaryGetter {
|
|||
BinaryDictionaryFileDumper.cacheDictionariesFromContentProvider(locale, context);
|
||||
final File[] cachedDictionaryList = getCachedDictionaryList(locale, context);
|
||||
|
||||
final String mainDictId = getMainDictId(locale);
|
||||
|
||||
final DictPackSettings dictPackSettings = new DictPackSettings(context);
|
||||
|
||||
boolean foundMainDict = false;
|
||||
final ArrayList<AssetFileAddress> fileList = new ArrayList<AssetFileAddress>();
|
||||
// cachedDictionaryList may not be null, see doc for getCachedDictionaryList
|
||||
for (final File f : cachedDictionaryList) {
|
||||
final String wordListId = getWordListIdFromFileName(f.getName());
|
||||
if (wordListId.equals(mainDictId)) {
|
||||
foundMainDict = true;
|
||||
}
|
||||
if (!dictPackSettings.isWordListActive(wordListId)) continue;
|
||||
if (f.canRead()) {
|
||||
fileList.add(AssetFileAddress.makeFromFileName(f.getPath()));
|
||||
|
@ -248,14 +261,14 @@ class BinaryDictionaryGetter {
|
|||
}
|
||||
}
|
||||
|
||||
if (!fileList.isEmpty()) {
|
||||
return fileList;
|
||||
}
|
||||
// If the list is empty, fall through and return the fallback
|
||||
|
||||
if (!foundMainDict && dictPackSettings.isWordListActive(mainDictId)) {
|
||||
final AssetFileAddress fallbackAsset = loadFallbackResource(context, fallbackResId,
|
||||
locale);
|
||||
if (null == fallbackAsset) return null;
|
||||
return Arrays.asList(fallbackAsset);
|
||||
if (null != fallbackAsset) {
|
||||
fileList.add(fallbackAsset);
|
||||
}
|
||||
}
|
||||
|
||||
return fileList;
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in a new issue