When I add the line "{ upsert: true }", I got this error:
TypeError: callback.apply is not a function
// on routes that end in /users/petitorAnalysisTextData
// ----------------------------------------------------
router
.route('/users/petitorAnalysisTextData/:userName')
// update the user info (accessed at PUT http://localhost:8080/api/users/petitorAnalysisTextData)
.post(function (req, res) {
// use our user model to find the user we want
User.findOne({userName: req.params.userName}, function (err, user) {
if (err) res.send(err);
console.log(
'userpetitorAnalysis.firstObservation: %@',
userpetitorAnalysis.firstObservation,
);
// Got the user name
var userName = user.userName;
// update the text data
console.log('Baobao is here!');
user.update(
{
userName: userName,
},
{
$set: {
'petitorAnalysis.firstObservation': req.body.firstObservation,
'petitorAnalysis.secondObservation': req.body.secondObservation,
'petitorAnalysis.thirdObservation': req.body.thirdObservation,
'petitorAnalysis.brandName': req.body.brandName,
'petitorAnalysis.productCategory': req.body.productCategory,
},
},
{upsert: true},
);
// save the user
user.save(function (err) {
if (err) return res.send(err);
return res.json({message: 'User updated!'});
});
});
});
Without this line, there is no error. I'm new to nodejs, not very sure where the problem is.
Update
No error message now, but this part of the database is not updated with new data. The embedded document is still empty.
// on routes that end in /users/petitorAnalysisTextData
// ----------------------------------------------------
router
.route('/users/petitorAnalysisTextData/:userName')
// update the user info (accessed at PUT http://localhost:8080/api/users/petitorAnalysisTextData)
.post(function (req, res) {
console.log('1');
// Just give instruction to mongodb to find document, change it;
// then finally after mongodb is done, return the result/error as callback.
User.findOneAndUpdate(
{userName: req.params.userName},
{
$set: {
'petitorAnalysis.firstObservation': req.body.firstObservation,
'petitorAnalysis.secondObservation': req.body.secondObservation,
'petitorAnalysis.thirdObservation': req.body.thirdObservation,
'petitorAnalysis.brandName': req.body.brandName,
'petitorAnalysis.productCategory': req.body.productCategory,
},
},
{upsert: true},
function (err, user) {
// after mongodb is done updating, you are receiving the updated file as callback
console.log('2');
// now you can send the error or updated file to client
if (err) return res.send(err);
return res.json({message: 'User updated!'});
},
);
});
When I add the line "{ upsert: true }", I got this error:
TypeError: callback.apply is not a function
// on routes that end in /users/petitorAnalysisTextData
// ----------------------------------------------------
router
.route('/users/petitorAnalysisTextData/:userName')
// update the user info (accessed at PUT http://localhost:8080/api/users/petitorAnalysisTextData)
.post(function (req, res) {
// use our user model to find the user we want
User.findOne({userName: req.params.userName}, function (err, user) {
if (err) res.send(err);
console.log(
'user.petitorAnalysis.firstObservation: %@',
user.petitorAnalysis.firstObservation,
);
// Got the user name
var userName = user.userName;
// update the text data
console.log('Baobao is here!');
user.update(
{
userName: userName,
},
{
$set: {
'petitorAnalysis.firstObservation': req.body.firstObservation,
'petitorAnalysis.secondObservation': req.body.secondObservation,
'petitorAnalysis.thirdObservation': req.body.thirdObservation,
'petitorAnalysis.brandName': req.body.brandName,
'petitorAnalysis.productCategory': req.body.productCategory,
},
},
{upsert: true},
);
// save the user
user.save(function (err) {
if (err) return res.send(err);
return res.json({message: 'User updated!'});
});
});
});
Without this line, there is no error. I'm new to nodejs, not very sure where the problem is.
Update
No error message now, but this part of the database is not updated with new data. The embedded document is still empty.
// on routes that end in /users/petitorAnalysisTextData
// ----------------------------------------------------
router
.route('/users/petitorAnalysisTextData/:userName')
// update the user info (accessed at PUT http://localhost:8080/api/users/petitorAnalysisTextData)
.post(function (req, res) {
console.log('1');
// Just give instruction to mongodb to find document, change it;
// then finally after mongodb is done, return the result/error as callback.
User.findOneAndUpdate(
{userName: req.params.userName},
{
$set: {
'petitorAnalysis.firstObservation': req.body.firstObservation,
'petitorAnalysis.secondObservation': req.body.secondObservation,
'petitorAnalysis.thirdObservation': req.body.thirdObservation,
'petitorAnalysis.brandName': req.body.brandName,
'petitorAnalysis.productCategory': req.body.productCategory,
},
},
{upsert: true},
function (err, user) {
// after mongodb is done updating, you are receiving the updated file as callback
console.log('2');
// now you can send the error or updated file to client
if (err) return res.send(err);
return res.json({message: 'User updated!'});
},
);
});
2 Answers
Reset to default 11There are 2 ways to update documents in mongodb:
find the document, bring it to server, change it, then save it back to mongodb.
just give instruction to mongodb to find document, change it; then finally after mongodb is done, return the result/error as callback.
In your code, you are mixing both methods.
with user.save(), first you search the database with user.findOne, and pull it to server(nodejs), now it lives in your puter memory. then you can manually change the data and finally save it to mongodb with user.save()
User.findOne({ userName: req.params.userName}, function(err, user) { if (err) res.send(err); //this user now lives in your memory, you can manually edit it user.username = "somename"; user.petitorAnalysis.firstObservation = "somethingelse"; // after you finish editing, you can save it to database or send it to client user.save(function(err) { if (err) return res.send(err); return res.json({ message: 'User updated!' }); });the second one is to use User.findOneAndUpdate().. This is preferred, instead of user.findOne() then user.update(); because those basically searching the database twice. first to findOne(), and search again to update()
Anyway,the second method is telling mongodb to update the data without first bringing to server, Next, only after mongodb finish with its action, you will receive the updated-file (or error) as callback
User.findOneAndUpdate({ userName: req.params.userName},
{
$set: { "petitorAnalysis.firstObservation" : req.body.firstObservation,
"petitorAnalysis.secondObservation" : req.body.secondObservation,
"petitorAnalysis.thirdObservation" : req.body.thirdObservation,
"petitorAnalysis.brandName" : req.body.brandName,
"petitorAnalysis.productCategory" : req.body.productCategory
} },
{ upsert: true },
function(err, user) {
//after mongodb is done updating, you are receiving the updated file as callback
// now you can send the error or updated file to client
if (err)
res.send(err);
return res.json({ message: 'User updated!' });
});
You forgot to pass a callback to the update method
user.update(
{
$set: {
'petitorAnalysis.firstObservation': req.body.firstObservation,
'petitorAnalysis.secondObservation': req.body.secondObservation,
'petitorAnalysis.thirdObservation': req.body.thirdObservation,
'petitorAnalysis.brandName': req.body.brandName,
'petitorAnalysis.productCategory': req.body.productCategory,
},
},
{upsert: true},
function (err, result) {},
);
update method expects 3 arguments.
- document update
- options
- callback