1

Error que me da la consola al querer hacer la petición Post agregando un auto a la colección de autos que se encuentra dentro del modelo de cliente:

(node:12176) UnhandledPromiseRejectionWarning: ValidationError: Car validation failed: client: Cast to ObjectId failed for value "[ { nombre: 'Ford', apellido: 'Fiat' } ]" at path "client"

Modelo auto

 const { Schema, model } = require('mongoose');
    
    let CarSchema = new Schema({
      model: { type: String, required: [true, "the model field is required"] },
      colour: { type: String, required: [true, "the colour field is required"] },
    
      client: {
        type: Schema.Types.ObjectId,
        ref: "Client",
        required: true
      },
    });
    
    CarSchema.methods.toJSON = function () {
      const { __v, ...data } = this.toObject();
      return data;
    };

Modelo cliente

module.exports = model( 'Car', CarSchema );
const { Schema, model } = require('mongoose');
    
    let ClientSchema = new Schema({
        
        nombre: {
            type: String,
            required: [true, 'El nombre es obligatorio']
        },
        apellido: {
            type: String,
            required: [true, 'El correo es obligatorio']
        },
        car: [
                { type: Schema.Types.ObjectId, ref: 'car' ,
                required: [true, 'El auto es obligatorio']
            }],
    });
    
    ClientSchema.methods.toJSON = function() {
        const { __v, ...client  } = this.toObject();
        client.id = _id;
        return client;
    }
    
    module.exports = model( 'Client', ClientSchema );

ControladorCar

let Car = require('../models/car');
const mongoose = require('mongoose');

    exports.car_create_post = async function(req, res) {
        const car = new Car(req.body);
        delete car._id;
        await car.save();
        
        const client = await Client.findById({_id: car.client})
        client.car.push(car);
        await client.save();
        res.status(200).json({success:true, data: car })
    };
1
  • 2
    Pues si que puedes (debes) cambiar algunas cosas en tu implementación. En cuanto pueda te doy una respuesta. Saludos Commented el 27 mar. 2021 a las 8:55

1 respuesta 1

3

Basándome en esta respuesta, estás guardando el objeto completo en vez del ObjectId de car:

client.car.push(car);

En su lugar, deberías guardar el ObjectId del nuevo car. Recuperar el objeto nuevo retornado en la función .save() podría ser un método para obtener su _id:

await car.save()
         .then(newCar => {
            const client = await Client.findById({_id: car.client});
            client.car.push(newCar._id); //<--------
            await client.save();
            return res.status(200).json({success:true, data: car });
          });

(asumiendo que te encargarás de validar los errores con un try-catch, por ejemplo) :)

2
  • yo quiero guardar el objeto completo dentro del array, como seria eso?
    – Fede
    Commented el 27 mar. 2021 a las 3:52
  • 2
    En ese caso, estarías rompiendo lo que definiste en el schema de Client. Por cierto, en ClientSchema.car estás referenciando al modelo en minúscula: es decir: ref: 'car', cuando debería ser ref: 'Car'.
    – padaleiana
    Commented el 27 mar. 2021 a las 10:43

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.