Использование findOneAndUpdate $ pull и заполнение не обновляет и не заполняет

Question

Использование findOneAndUpdate $ pull и заполнение не обновляет и не заполняет

1

Это мои данные:

{
    "_id" : ObjectId("594e762b03cc52508686ceef"),
    "_members" : [ 
        "59527bd5e521801bf07eab98", 
        "594ecca47a699d1775c2a2db"
    ],
}

Я хочу удалить 59527bd5e521801bf07eab98 из _members.

PostSchema:

const PostsSchema = new Schema({
    // ...
    _members: [{ type: Schema.Types.ObjectId, ref: 'user' }],
    // ...
})

UserSchema:

const UserSchema = new Schema({
    // ...
    posts : [{ type: Schema.Types.ObjectId, ref: 'post' }],
    // ...
})

Найти и обновить:

let id = mongoose.mongo.ObjectID(req.params.id)
let member = mongoose.mongo.ObjectID(req.params.member)
let populateQuery = [
    {path:'_members', select:'_id'}
]

Post.
findOneAndUpdate(
    {'_id' : id} ,
    {
       $pull: { '_members': { '_id': member }}
     },
     {'new': true}
).
populate(populateQuery).
exec(
  function (err, data) {
    res.send(data)
  }
);

user1788781 01 июль 2017, в 02:28

Источник

0

Задавая вопросы, пожалуйста, включите сам вопрос, а также любое сообщение об ошибке, которое вы видите. Непонятно, что у вас не работает: проблема в соединении, ошибка клиента, синтаксическая ошибка и т. Д.?
Eye 30 июнь 2017, в 23:57
0

@Eye Можно было бы заявить более четко, но «ошибкой» в этом случае является то, что .populate() не делает то, что должен, и извлекает связанные данные. Что является темой вопроса. Изменено название, чтобы отразить это
Neil Lunn 01 июль 2017, в 01:22

Теги:

javascript

node.js

mongodb

mongoose

1 ответ

Ещё вопросы

Задавая вопросы, пожалуйста, включите сам вопрос, а также любое сообщение об ошибке, которое вы видите. Непонятно, что у вас не работает: проблема в соединении, ошибка клиента, синтаксическая ошибка и т. Д.?
@Eye Можно было бы заявить более четко, но «ошибкой» в этом случае является то, что .populate() не делает то, что должен, и извлекает связанные данные. Что является темой вопроса. Изменено название, чтобы отразить это

Neil Lunn · Accepted Answer · 2017-06-30T22-36-00.000Z

Интересно, что "население", похоже, работает только с указанием варианта model для .populate() в этом случае.

Кроме того, вам нужно всего лишь предоставить member прямо в $pull поскольку _id самом деле не является свойством, до "заселенного". "_members" представляет собой просто массив значений ObjectId внутри самой Post. Поэтому подайте вход напрямую:

Post.findOneAndUpdate(
  {'_id' : id} ,
  {
    $pull: { '_members': member }
  },
  {'new': true}
)
.populate({path:'_members', select:'_id', model: 'user' })
.exec(function(err,post) {
  if (err) throw err; // or something
  res.send(post)
})

Или с обещаниями

Post.findOneAndUpdate(
  {'_id' : id} ,
  {
    $pull: { '_members':  member }
  },
  {'new': true}
)
.populate({path:'_members', select:'_id', model: 'user' })
.then(post => res.send(post))
.catch(err => console.error(err));  // or something

Вместо Model.populate() вместо этого вызовите Model.populate():

Post.findOneAndUpdate(
  {'_id' : id} ,
  {
    $pull: { '_members': member }
  },
  {'new': true}
).exec(function(err,post) {
  if (err) throw err; // or something

  Post.populate(post, {path:'_members', select:'_id', model: 'user' },function(err,post) {
    if (err) throw err; // or something
    res.send(post);   // now populated
  }
})

Или поочередно использовать обещания:

Post.findOneAndUpdate(
  {'_id' : id} ,
  {
    $pull: { '_members': member }
  },
  {'new': true}
)
.then(post => Post.populate(post, {path:'_members', select:'_id', model: 'user' }))
.then(post => res.send(post))      // also populated
.catch(err => console.error(err))    // or whichever

Несколько .populate() том, почему вы хотите вызвать .populate() когда вы только просите вернуть поле _id которое уже включено в документ, но это другой случай. С вариантом model население действительно имеет место здесь.

Как самостоятельная демонстрация:

const async = require('async'),
    mongoose = require('mongoose'),
    Schema = mongoose.Schema;

mongoose.Promise = global.Promise;
mongoose.set('debug',true);
mongoose.connect('mongodb://localhost/test');

const relSchema = new Schema({ c: String });
const testSchema = new Schema({
  a: String,
  b: [{ type: Schema.Types.ObjectId, rel: 'Rel' }]
})

const Test = mongoose.model('Test', testSchema);
const Rel = mongoose.model('Rel', relSchema);

function log(data) {
  console.log(JSON.stringify(data, undefined, 2))
}

async.series(
  [
    (callback) =>
      async.each(mongoose.models,(model,callback) =>
        model.remove({},callback),callback),

    (callback) =>
      async.waterfall(
        [

          (callback) => Rel.create([{ c: 2 },{ c: 3 },{ c: 4 }],callback),


          (rels,callback) => Test.create({ a: 1, b: rels },(err,test) => {
            if (err) callback(err);
            log(test);
            callback(err,test.b.slice(1,3))
          }),

          (rels,calback) =>
            Test.findOneAndUpdate(
              { 'a': 1 },
              { '$pull': { 'b': rels[0] } },
              { 'new': true }
            )
            .populate({ path: 'b', model: 'Rel' })
            .exec((err,test) => {
              if (err) callback(err);
              log(test);
              callback(err);
            })

        ],
        callback
      )

  ],
  (err) => {
    if (err) throw err;
    mongoose.disconnect();
  }
)

И выход:

Mongoose: tests.remove({}, {})
Mongoose: rels.remove({}, {})
Mongoose: rels.insert({ c: '2', _id: ObjectId("595714579afd8860e56d2ec7"), __v: 0 })
Mongoose: rels.insert({ c: '3', _id: ObjectId("595714579afd8860e56d2ec8"), __v: 0 })
Mongoose: rels.insert({ c: '4', _id: ObjectId("595714579afd8860e56d2ec9"), __v: 0 })
Mongoose: tests.insert({ a: '1', _id: ObjectId("595714579afd8860e56d2eca"), b: [ ObjectId("595714579afd8860e56d2ec7"), ObjectId("595714579afd8860e56d2ec8"), ObjectId("595714579afd8860e56d2ec9") ], __v: 0 })
{
  "__v": 0,
  "a": "1",
  "_id": "595714579afd8860e56d2eca",
  "b": [
    "595714579afd8860e56d2ec7",
    "595714579afd8860e56d2ec8",
    "595714579afd8860e56d2ec9"
  ]
}
Mongoose: tests.findAndModify({ a: '1' }, [], { '$pull': { b: ObjectId("595714579afd8860e56d2ec8") } }, { new: true, upsert: false, remove: false, fields: {} })
Mongoose: rels.find({ _id: { '$in': [ ObjectId("595714579afd8860e56d2ec7"), ObjectId("595714579afd8860e56d2ec9") ] } }, { fields: {} })
{
  "_id": "595714579afd8860e56d2eca",
  "a": "1",
  "__v": 0,
  "b": [
    {
      "_id": "595714579afd8860e56d2ec7",
      "c": "2",
      "__v": 0
    },
    {
      "_id": "595714579afd8860e56d2ec9",
      "c": "4",
      "__v": 0
    }
  ]
}

@ user1788781 Вы можете поблагодарить меня, используя ваши недавно приобретенные "привилегии upvote"