diff options
author | toasted-nutbread <toasted-nutbread@users.noreply.github.com> | 2020-06-21 15:54:34 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-06-21 15:54:34 -0400 |
commit | 0c69e54fde259568ebd89a3c9e8481cf28860ea8 (patch) | |
tree | a472ac24e87a3718d80cb4a9903f74357d804e28 /ext/bg | |
parent | 9e6153f172fd9043eaa34cd2de639ce2b9c3f4a3 (diff) |
Optimize anki note field generation (#611)
Diffstat (limited to 'ext/bg')
-rw-r--r-- | ext/bg/js/anki-note-builder.js | 16 | ||||
-rw-r--r-- | ext/bg/js/backend.js | 7 |
2 files changed, 17 insertions, 6 deletions
diff --git a/ext/bg/js/anki-note-builder.js b/ext/bg/js/anki-note-builder.js index 31e67394..7fe8962a 100644 --- a/ext/bg/js/anki-note-builder.js +++ b/ext/bg/js/anki-note-builder.js @@ -28,8 +28,9 @@ class AnkiNoteBuilder { const modeOptions = isKanji ? options.anki.kanji : options.anki.terms; const modeOptionsFieldEntries = Object.entries(modeOptions.fields); + const fields = {}; const note = { - fields: {}, + fields, tags, deckName: modeOptions.deck, modelName: modeOptions.model, @@ -38,8 +39,17 @@ class AnkiNoteBuilder { } }; - for (const [fieldName, fieldValue] of modeOptionsFieldEntries) { - note.fields[fieldName] = await this.formatField(fieldValue, definition, mode, context, options, templates, null); + const formattedFieldValuePromises = []; + for (const [, fieldValue] of modeOptionsFieldEntries) { + const formattedFieldValuePromise = this.formatField(fieldValue, definition, mode, context, options, templates, null); + formattedFieldValuePromises.push(formattedFieldValuePromise); + } + + const formattedFieldValues = await Promise.all(formattedFieldValuePromises); + for (let i = 0, ii = modeOptionsFieldEntries.length; i < ii; ++i) { + const fieldName = modeOptionsFieldEntries[i][0]; + const formattedFieldValue = formattedFieldValues[i]; + fields[fieldName] = formattedFieldValue; } return note; diff --git a/ext/bg/js/backend.js b/ext/bg/js/backend.js index 93ba620f..a8ee9f83 100644 --- a/ext/bg/js/backend.js +++ b/ext/bg/js/backend.js @@ -579,13 +579,14 @@ class Backend { const states = []; try { - const notes = []; + const notePromises = []; for (const definition of definitions) { for (const mode of modes) { - const note = await this.ankiNoteBuilder.createNote(definition, mode, context, options, templates); - notes.push(note); + const notePromise = this.ankiNoteBuilder.createNote(definition, mode, context, options, templates); + notePromises.push(notePromise); } } + const notes = await Promise.all(notePromises); const cannotAdd = []; const results = await this.anki.canAddNotes(notes); |