alex/yomichan
1
Fork 0
Code Releases Activity

Merge pull request #66 from lae/fix/bulk-import-failure

Browse Source 
Import terms with transactional adds instead of bulkAdd
This commit is contained in:
Alex Yatskov 2017-07-27 14:28:28 -07:00 committed by GitHub
commit 9be66c7e7b
1 changed files with 13 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

27
ext/bg/js/database.js
View File

@ -189,20 +189,19 @@ class Database {
}; };
const termsLoaded = (title, entries, total, current) => { const termsLoaded = (title, entries, total, current) => {
const rows = []; return this.db.transaction('rw', this.db.terms, function() {
for (const [expression, reading, tags, rules, score, ...glossary] of entries) { for (const [expression, reading, tags, rules, score, ...glossary] of entries) {
rows.push({ this.db.terms.add({
expression, expression,
reading, reading,
tags, tags,
rules, rules,
score, score,
glossary, glossary,
dictionary: title dictionary: title
}); });
} }
}).then(() => {
return this.db.terms.bulkAdd(rows).then(() => {
if (callback) { if (callback) {
callback(total, current); callback(total, current);
} }