Make sure we don't use a ton of GET query params for Kinto

common-voice · Jan 9, 2019 · a2408fa · a2408fa
1 parent eef26fa
commit a2408fa
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/shared/db/collections/sentences-meta.js b/shared/db/collections/sentences-meta.js
@@ -272,14 +272,11 @@ export default class SentencesMeta {
 
   async getAlreadyExistingSubset(language, sentences) {
     const idList = sentences.map(s => hash(s));
-    const collection = await this.getCollection(language);
-    const result = await collection.listRecords({
-      filters: {
-        in_id: idList,
-      },
-    });
+    const result = await this.getAll(language);
+    const existingIDs = result.map((existingSentence) => existingSentence.id);
 
-    return result.data;
+    const existingSubmittedSentencesIDs = idList.filter((id) => existingIDs.includes(id));
+    return result.filter((existingSentence) => existingSubmittedSentencesIDs.includes(existingSentence.id));
   }
 
   async getMySentences(language) {