2

我正在开发一个猫鼬/node.js/express 应用程序。

在我的路线中，我使用express 的 app.param()方法将我的模型实例放入请求中 - 而不是在每个控制器操作中获取它。

我在我的控制器中得到了show和create动作 - 但是我被困在执行update动作中。

这是我的相关控制器代码：

// mymodel-controller.js
var mongoose = require('mongoose')
var MyModel = mongoose.model('MyModel');
var utils = require('../../lib/utils');

"use strict";

exports.update = function (req, res, next) {

  var mymodel = req.mymodel;
  mymodel.set(req.body);
  mymodel.slug = utils.convertToSlug(req.body.title);

  mymodel.save(function(err, doc) {
    if (!err) {
      console.log('update successful');
       // here i see the correctly updated model in the console, however the db is not updated
      console.dir(doc); 
      return res.redirect('/mymodels/' + mymodel.slug);
    } else {
      console.error('update error', err);
      return res.render('mymodels/edit', {
        title: 'Edit Model',
        model: mymodel,
        errors: err.errors
      });
    }
  });
}

奇怪的是，猫鼬保存成功，我没有收到任何错误。我看到“更新成功”并且控制台上打印了正确更新的模型，但没有保留在数据库中。我也尝试在更新和保存模型之前手动获取模型，而不是使用 app.param(..) 但我有同样的效果。知道我在这里缺少什么吗？

更新

这是我设置req.mymodel较长路由文件的 - 部分的代码。例如，在我的表演动作中，我还使用 req.mymodel 来显示它，到目前为止它工作正常。

/**
 * mymodel ROUTES
 */
app.param('mymodel', function(req, res, next, id){
  MyModel.findOne({ 'slug': id }, function(err, mymodel) {
    if (err) {
      next(err);
    } else if (mymodel) {
      req.mymodel = mymodel;
      next();
    } else {
      res.status(404).render('404', {title: "Not found", errorMessage: "The requested mymodel was not found."});
    }
  });
});
app.get('/mymodels',                 mymodelsController.index);
app.get('/mymodels/new',             mymodelsController.new);
app.post('/mymodels',                mymodelsController.create);
app.get('/mymodels/:mymodel',        mymodelsController.show);
app.get('/mymodels/:mymodel/edit',   mymodelsController.edit);
app.put('/mymodels/:mymodel',        mymodelsController.update); // that's the one not working with the code above
app.del('/mymodels/:mymodel',        mymodelsController.destroy);

更新2

另一方面，这段代码在我的控制器中工作并更新数据库（虽然我更喜欢使用 instance.save()）

exports.update = function (req, res, next) {

  var mymodel = req.param('mymodel');


  var query = { slug: mymodel };

  var update = {};
  update.title = req.param('title');
  update.body = req.param('body');
  update.points = req.param('points');
  update.location = req.param('location');
  update.slug = utils.convertToSlug(req.param('title'));

  MyModel.update(query, update, function (err, numAffected) {
    if (err) return next(err);

    if (0 === numAffected) {
      return next(new Error('no model to modify'), null);
    }

    res.redirect('/mymodels/' + update.slug);
  });
}
4

1 回答 1

2

问题确实是我req.param在路线中设置值的方式。

而不是在我的路线文件中这样做：

app.param('mymodel', function(req, res, next, id){
  MyModel.findOne({ 'slug': id }, function(err, mymodel) {
    if (err) {
      next(err);
    } else if (mymodel) {
      req.mymodel = mymodel;
      next();
    } else {
      res.status(404).render('404', {title: "Not found", errorMessage: "The requested mymodel was not found."});
    }
  });
});

我必须声明一个每次加载实例的控制器操作，比如

app.param('mymodel', mymodelController.load);

在这个例子中看起来很像。

于 2013-10-31T09:55:54.040 回答