Added limit query to products

Author: Joker-Bat

controllers/productController.js

@@ -23,18 +23,19 @@ function ifObjectIsEmpty(object) {
 
 exports.getAllProducts = catchAsync(async (req, res, next) => {
   const products = new APIFeatures(Product.find(), req.query)
-    .filter()
+    ?.filter()
     ?.sort()
-    ?.limitFields();
+    ?.limitFields()
+    ?.limitResults();
 
   // Check if valid promise is got
-  if (!products)
-    return next(
-      new AppError(
-        'Please provide a valid query parameters check documentation!',
-        400
-      )
-    );
+  // if (!products)
+  //   return next(
+  //     new AppError(
+  //       'Please provide a valid query parameters check documentation!',
+  //       400
+  //     )
+  //   );
 
   const result = await products.model;
   // check if first item got any item in there

utils/apiFeatures.js

@@ -5,14 +5,20 @@ class APIFeatures {
   }
 
   filter() {
-    const queryObj = { ...this.queryString };
-    const excludedFields = ['sort', 'fields', 'limit'];
-    excludedFields.forEach((el) => delete queryObj[el]);
+    if (this.queryString) {
+      const queryObj = { ...this.queryString };
+      const excludedFields = ['sort', 'fields', 'limit'];
+      excludedFields.forEach((el) => delete queryObj[el]);
 
-    let queryStr = JSON.stringify(queryObj);
-    queryStr = queryStr.replace(/\b(gt|gte|lt|lte)\b/g, (match) => `$${match}`);
+      let queryStr = JSON.stringify(queryObj);
+      queryStr = queryStr.replace(
+        /\b(gt|gte|lt|lte)\b/g,
+        (match) => `$${match}`
+      );
 
-    this.model = this.model.find(JSON.parse(queryStr));
+      this.model = this.model.find(JSON.parse(queryStr));
+      return this;
+    }
     return this;
   }
 
@@ -40,6 +46,14 @@ class APIFeatures {
 
     return this;
   }
+
+  limitResults() {
+    if (this.queryString.limit) {
+      this.model = this.model.limit(this.queryString.limit * 1);
+      return this;
+    }
+    return this;
+  }
 }
 
 module.exports = APIFeatures;