Updating an array using conditionals with mongoose in node.js

Question

I am trying to update a MongoDB collection that I have created a schema for using Mongoose. My goal is to remove all elements in an array known as 'alarms' that are less than (in the past) of the current time.

Here is my Mongoose Schema:

var mongoose = require('mongoose');

var ReminderSchema = new mongoose.Schema({
    firstName: String,
    lastName: String,
    phone: Number,
    email: String,
    medication: String,
    alarms: [Date]
});

module.exports = mongoose.model('Reminder', ReminderSchema);

Here is the update. I am trying to remove all alarms that are less than the current date.

var date = new Date();
var Reminder = require('./models/Reminders.js');

//remove ALL alarms that are are before current time
Reminder.update({alarms: {$lte: date}}, {$pullAll: {alarms: {$lte: date}}}, function(err, updated){
    if(err){
         console.log(err);
     }
     else {
         console.log(updated);
     }
 });

I am currently getting this error:

{ [MongoError: $pullAll requires an array argument but was given a Object]
name: 'MongoError',
message: '$pullAll requires an array argument but was given a Object',
driver: true,
index: 0,
code: 2,
errmsg: '$pullAll requires an array argument but was given a Object' }

Any thoughts?

Just replace the $pullAll by $pull

iagowp
– iagowp

2016-01-26 18:22:57 +00:00
Commented Jan 26, 2016 at 18:22 — iagowp
– iagowp, Commented Jan 26, 2016 at 18:22
This works. Thank you.

Bill Headrick
– Bill Headrick

2016-01-26 18:31:46 +00:00
Commented Jan 26, 2016 at 18:31 — Bill Headrick
– Bill Headrick, Commented Jan 26, 2016 at 18:31

Alvaro Silvino · Accepted Answer · 2016-01-26 18:24:48Z

2

The $pullAll receives an array not an object:

{ $pullAll: { <field1>: [ <value1>, <value2> ... ], ... } }
The $pullAll operator removes all instances of the specified values from an existing array. Unlike the $pull operator that removes elements by specifying a query, $pullAll removes elements that match the listed values.

the solution you should use $pull :

Reminder.update({alarms: {$lte: date}},{$pull:{alarms: { $lte : new Date()}}}, 
 function(err, updated){
if(err){
     console.log(err);
 }
 else {
     console.log(updated);
 }
});

https://docs.mongodb.org/manual/reference/operator/update/pullAll/

edited Jan 26, 2016 at 18:24

answered Jan 26, 2016 at 18:15

Alvaro Silvino

9,83314 gold badges56 silver badges81 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Bill Headrick Over a year ago

This does the job. Thank you very much.

Collectives™ on Stack Overflow

Updating an array using conditionals with mongoose in node.js

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related