Tienes varios detalles de implementación. Vamos a ver como los solucionamos uno por uno.
PRIMERO
El modelo de datos debe tener un nombre coherente con la entidad que representan. Por ello, mi sugenrencia es que uses un nombre que represente realmente dicha entidad.
Por ejemplo:
const mongoose = require('mongoose');
const MenuSchema = new mongoose.Schema({
nombremenu: {
type: String,
required: true
},
descripcion: {
type: String,
required: true
},
precio: {
type: Number,
required:true
},
precioenvio: {
type: Number,
required: true
} ,
date: {
type: Date,
required: true
}
}, { _id: false }); // con _id: false, evito crear un campo _id en el subdocumento
const CartaSchema = new mongoose.Schema({
nombrecate: {
type: String,
required: true
},
menu: {
type: [ MenuSchema ]
}
});
module.exports = mongoose.model('Carta', CartaSchema)
Como puedes observar he separado la definición de Schema del Menú fuera de la Carta, de esta forma es mucho más sencillo mantener el código. Al hacerlo así, debo decidir si al crear o agregar un objeto al Array de menu
, se crearé un valor de _id
o no. En este caso he dicho que no, pasando como opción a la creación del Schema el valor { _id: false }
. Para más información puedes leer la documentación sobre tipo Schema
Otra cosa que he hecho es usar PascalCase (o UpperCamelCase) para nombrar tanto mis esquemas como mi modelo de datos.
SEGUNDO
Para crear una nueva carta tienes varias opciones, pero voy a resaltar el problema que tienes y cómo solucionarlo, para luego escribir una alternativa a tu código que sea más legible.
En tu caso, el problema es que estás llamando al método updateOne
de forma incorrecta. En realidad está incompleta la llamada.
El método updateOne
puede recibir una función callback
(en versiones de Mongoose 6.x o anteriores) o puede ser usada con await
o con then
.
Tu no has hecho nada de ello:
//...
modelo.updateOne({'_id': newcarta._id}, {
$push:{
'menu': {
nombremenu: req.body.nombremenu,
descripcion: req.body.descripcion,
precio: req.body.precio,
precioenvio: req.body.precioenvio
}
}
});
// ...
Entonces lo que está pasando es que estás creando un Objeto Query que por sí solo no actualiza la base de datos a menos que pases la función callback
o hagas uso de await
y llames a exec()
, o uses then()
.
Usando una función callback
Si usas Mongoose 6.11.5 o anterior, la forma de hacer la llamada será:
//...
modelo.updateOne({'_id': newcarta._id}, {
$push:{
'menu': {
nombremenu: req.body.nombremenu,
descripcion: req.body.descripcion,
precio: req.body.precio,
precioenvio: req.body.precioenvio
}
}
}, (error, carta) => {
// se ha ejecutado la actualización
if(error) {
console.error(error);
return res.redirect('/error');
}
console.log(carta);
return res.redirect('/carta');
});
//...
Como se observa, la lógica de res.redirect
se ejecuta dentro de la función callback
que he pasado como último argumento a updateOne
.
Usando then
Debes tomar en cuenta que aunque podemos usar await
y then
sobre un Objeto Query, éste no es una Promesa
Una forma de usar then
en tu código actual sería:
//...
modelo.updateOne({'_id': newcarta._id}, {
$push:{
'menu': {
nombremenu: req.body.nombremenu,
descripcion: req.body.descripcion,
precio: req.body.precio,
precioenvio: req.body.precioenvio
}
}
}).then((carta) => {
// se ha ejecutado la actualización
console.log(carta);
return res.redirect('/carta');}
);
// ...
Puedes ver que he encadenado then
a mi llamada a updateOne
y es dentro de la función que recibe then
en donde voy a implementar la lógica para res.redirect
.
Usando await
Podemos usar await
delante de la llamada a updateOne
de esta forma no necesitamos usar una función callback
ni usar then
. Pero si realmente queremos que updateOne
devuelva una Promesa que pueda resolver o rechazar, debemos convertir el Objeto Query en una Promesa usando el método exec()
encadenado a la llamada a updateOne
, por ejemplo:
//...
await modelo.updateOne({'_id': newcarta._id}, {
$push:{
'menu': {
nombremenu: req.body.nombremenu,
descripcion: req.body.descripcion,
precio: req.body.precio,
precioenvio: req.body.precioenvio
}
}
}).exec(); // <= ejecuta la actualización y devuelve una Promesa
// ...
Mejora
La siguiente es una mejora al código que muestras en tu pregunta, relacionado con la forma de crear un nuevo Menú en una Carta.
Supongamos que escribimos un nuevo menú y lo enviamos al Servidor para ser almacenado en la colección de Cartas. Tienes 2 posibles escenarios:
- La carta no existe (y por supuesto el menú tampoco)
- La carta existe, pero el menú no.
Ante esta incertidumbre, mi solución es usar el método findOneAndUpdate
pero usando la opción upsert
establecida a true
, que significa: si el documento existe, lo actualizo, si no existe, lo creo.
Mi código quedaría de la siguiente forma:
router.post('/home/menu-enviado', async (req, res) => {
const { body } = req;
if(!body) {
return res.redirect('/error');
}
const { nombrecate } = body;
if(!nombrecate) {
return res.redirect('/error');
}
const newMenu = {
nombremenu: body.nombremenu,
descripcion: body.descripcion,
precio: body.precio,
precioenvio: body.precioenvio
}
const update = {
$push: {
menu: newMenu
},
$setOnInsert: {
menu: [ newMenu ]
}
}
try {
const carta = await CartaModel.findOneAndUpdate(
{ nombrecate: nombrecate },
update,
{ upsert: true, new: true }
).exec();
console.log(carta);
return res.redirect('/carta');
}
catch(e) {
console.error(e);
return res.redirect('/error');
}
});
Ahora el código se ve un poco más limpio, además, al estar trabajando con Promesas, estoy usando un bloque try ... catch
para poder manejar el caso en que la Promesa sea rechazada.
También te podrás dar cuenta que el objeto llamado update
dispone de 2 propiedades: $push
y $setOnInsert
. Esto es porque estoy usando la opción upsert
establecida a true
. Por lo tanto, si el documento de Carta no existe, no habrá ningún Array de menú al que hacerle $push
, por ello uso:
$setOnInsert: {
menu: [ newMenu ]
}
haciendo que al insertar una nueva carta, el valor del campo menu
sea un Array cuyo único elemento es el nuevo menú creado a partir de la solicitud.
Espero que esto te ayude a resolver el problema y aclare tus dudas.
modelo
? Dónde lo defines? Si modelo es el objeto creado a partir del esquema de carta, mejor llamarlecartaModel
o algo relacionado con la carta. Porqué haces unsave
primero y luego unupdate
? Si estás creando un documento decarta
entonces o haces elsave
considerando todos los datos (así no tienes que actualizar) o simplemente usainsertOne
.