fix use of "new Date()" in pipelines

Unipisa · Nov 8, 2023 · 7ccb255 · 7ccb255
1 parent 7fe45bf
commit 7ccb255
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 5 deletions.
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "dm-manager",
-  "version": "1.3.9",
+  "version": "1.3.10",
   "private": true,
   "dependencies": {
     "@fortawesome/fontawesome-svg-core": "^6.1.2",

diff --git a/server/controllers/RoomController.js b/server/controllers/RoomController.js
@@ -15,8 +15,8 @@ class RoomController extends Controller {
         this.queryPipeline.push(
             // inserisce startDate e endDate con la data odierna
             {$addFields: {
-                startDate: new Date(),
-                endDate: new Date(),
+                startDate: "$$NOW",
+                endDate: "$$NOW",
             }},
             ...RoomAssignment.roomRoomAssignmentPipeline()
         ) 

diff --git a/server/controllers/public/seminars.js b/server/controllers/public/seminars.js
@@ -3,7 +3,11 @@ const EventSeminar = require('../../models/EventSeminar')
 async function seminarsQuery(req) {
     const pipeline = [
         { $match: {
-            startDatetime: {$gte: new Date()}
+            $expr: {
+                $gte: [
+                    "$startDatetime",
+                    "$$NOW"],
+            }
         }},
         { $lookup: {
             from: 'people',
@@ -70,7 +74,8 @@ async function seminarsQuery(req) {
         }}
     ]
 
-    // console.log(JSON.stringify({pipeline}))
+
+    console.log(JSON.stringify({pipeline}))
 
     const seminars = await EventSeminar.aggregate(pipeline)