alex/yomichan
1
Fork 0
Code Releases Activity

Translator data format improvements (#1500)

Browse Source 
* Update _createTermDetailsList usage and docs

* Update source of expression/reading/furiganaSegments
This commit is contained in:
toasted-nutbread 2021-03-07 12:18:44 -05:00 committed by GitHub
parent 49bf562690
commit 92fe1571ae
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
ext/js/language/translator.js
View File

@ -190,8 +190,8 @@ class Translator {
} }
for (const groupedDefinition of this._groupTerms(unsequencedDefinitions, enabledDictionaryMap)) { for (const groupedDefinition of this._groupTerms(unsequencedDefinitions, enabledDictionaryMap)) {
const {reasons, score, expression, reading, source, rawSource, sourceTerm, furiganaSegments, termTags, definitions: definitions2} = groupedDefinition; const {reasons, score, expression, reading, source, rawSource, definitions: definitions2} = groupedDefinition;
const termDetailsList = [this._createTermDetails(sourceTerm, expression, reading, furiganaSegments, termTags)]; const termDetailsList = this._createTermDetailsList(definitions2);
const compatibilityDefinition = this._createMergedTermDefinition( const compatibilityDefinition = this._createMergedTermDefinition(
source, source,
rawSource, rawSource,
@ -1119,8 +1119,13 @@ class Translator {
}; };
} }
/**
* Creates a grouped definition from an array of 'term' definitions.
* @param definitions An array of 'term' definitions.
* @returns A single 'termGrouped' definition.
*/
_createGroupedTermDefinition(definitions) { _createGroupedTermDefinition(definitions) {
const {expression, reading, furiganaSegments, reasons, source, rawSource, sourceTerm} = definitions[0]; const {reasons, source, rawSource, sourceTerm, expressions: [{expression, reading, furiganaSegments}]} = definitions[0];
const score = this._getMaxDefinitionScore(definitions); const score = this._getMaxDefinitionScore(definitions);
const dictionaryOrder = this._getBestDictionaryOrder(definitions); const dictionaryOrder = this._getBestDictionaryOrder(definitions);
const dictionaryNames = this._getUniqueDictionaryNames(definitions); const dictionaryNames = this._getUniqueDictionaryNames(definitions);
@ -1235,6 +1240,11 @@ class Translator {
}; };
} }
/**
* Creates a list of term details from an array of 'term' definitions.
* @param definitions An array of 'term' definitions.
* @returns An array of term details.
*/
_createTermDetailsList(definitions) { _createTermDetailsList(definitions) {
const termInfoMap = new Map(); const termInfoMap = new Map();
for (const {expression, reading, sourceTerm, furiganaSegments, termTags} of definitions) { for (const {expression, reading, sourceTerm, furiganaSegments, termTags} of definitions) {