Merge pull request #49 from VirtusLab/fix/query-limits

fix: drop query limits in the service
VirtusLab-Open-Source · Mar 16, 2021 · d2dff1c · d2dff1c
2 parents 2e1fca7 + 6c9dc56
commit d2dff1c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "strapi-plugin-navigation",
-  "version": "1.0.0-beta.18",
+  "version": "1.0.0-beta.19",
   "description": "Strapi - Navigation plugin",
   "strapi": {
     "name": "Navigation",

diff --git a/services/navigation.js b/services/navigation.js
@@ -61,7 +61,9 @@ module.exports = {
     if (additionalFields.includes(configAdditionalFields.AUDIENCE)) {
       const audienceItems = await strapi
         .query(audienceModel.modelName, pluginName)
-        .find({});
+        .find({
+          _limit: -1,
+        });
       extendedResult = {
         ...extendedResult,
         availableAudience: audienceItems.map((_) =>
@@ -156,7 +158,9 @@ module.exports = {
     const { pluginName, masterModel } = utilsFunctions.extractMeta(strapi.plugins);
     const entities = await strapi
       .query(masterModel.modelName, pluginName)
-      .find({}, []);
+      .find({
+        _limit: -1,
+      }, []);
     return entities.map((_) => sanitizeEntity(_, { model: masterModel }));
   },
 
@@ -171,6 +175,7 @@ module.exports = {
       .query(itemModel.modelName, pluginName)
       .find({
         master: id,
+        _limit: -1,
         _sort: 'order:asc',
       }, ['related', 'audience']);
 
@@ -259,6 +264,7 @@ module.exports = {
         {
           master: entity.id,
           ...itemCriteria,
+          _limit: -1,
           _sort: 'order:asc',
         },
         ["related", "audience"],