From 7f3b7e936e26b26a092fcf04cd8bca74684380ae Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Y=C3=BBki=20Vachot?= <yuki@vachot.fr>
Date: Mon, 10 Jan 2022 17:11:43 +0100
Subject: [PATCH] Update: User.ad randomize

---
 controllers/user.controller.js | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/controllers/user.controller.js b/controllers/user.controller.js
index 9af3856..d01b6d2 100644
--- a/controllers/user.controller.js
+++ b/controllers/user.controller.js
@@ -401,7 +401,7 @@ exports.ad = (req, res) => {
           {$sort: {watchedDates: -1}},
           {$limit: limit},
           {$unwind: '$interest'},
-          {$group: {_id: null, interests: {$push: '$interest'}}}
+          {$group: {_id: null, interests: {$addToSet: '$interest'}}}
         ])
           .then(data => {
             if(typeof data[0] !== 'undefined' &&
@@ -410,15 +410,13 @@ exports.ad = (req, res) => {
               data[0].interests !== null){
               interests = interests.concat(data[0].interests);
             }
-            let match, pick;
+            let match;
             if(interests.length > 0){
               match = {$match: {isVisible: true, isActive: true, interests: {$elemMatch: {interest: {$in: interests}}}}};
-              pick = {$limit: parseInt(quantity, 10)}
             } else {
               match = {$match: {isVisible: true, isActive: true}};
-              pick = {$sample: {size: parseInt(quantity, 10)}};
             }
-
+            const pick = {$sample: {size: parseInt(quantity, 10)}};
             Ad.aggregate([
                 match,
                 pick