Erreur Mongoose CastError: Cast to ObjectId failed for value "55ec9651cf81ea131fc82a7de" at path "_id" for mod

Version imprimable

Bonjour,
En essayant de supprimer une ressource en base (mongoDb), j'ai cette erreur :

Citation:

CastError: Cast to ObjectId failed for value "55ec9651cf81ea131fc82a7de" at path "_id" for model "Voiture"

La methode est la suivante :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 exports.deleteVoiture = catchAsync(async (req, res, next) => { const voiture = await Voiture.findByIdAndDelete(req.params.id); if (!voiture) { return next(new AppError('No voiture found with that ID', 404)); } res.status(204).json({ status: 'success', data: null }); });
URL sur Postman : localhost:3000/api/v1/voitures/55ec9651cf81ea131fc82a7de

Help please

29/05/2020, 13h04
djedjethai
Bonjour,

Vu le message d'err on dirait que le cast de l'id, qui est un Objet plante. Essaies de le faire manuellement
Code:

1 2 var mongoose = require('mongoose'); var id = mongoose.Types.ObjectId('req.params.id');
puis passe l'id ds la req

Peut etre peux tu essayer aussi de changer de req method, sait on jamais....

Code:

.findByIdAndRemove(id);
30/05/2020, 19h38
toutemes1

En suivant tes instructions, j'ai cette nouvelle erreur :

Citation:

Error: Argument passed in must be a single String of 12 bytes or a string of 24 hex characters

at new ObjectID (D:\Workspace-Dev\natours\node_modules\bson\lib\bson\objectid.js:59:11)
31/05/2020, 08h48
djedjethai

Citation:

Envoyé par toutemes1

En suivant tes instructions, j'ai cette nouvelle erreur :

Ummm... ok... normalement mongoose accepte les _id de type string (contrairement a mongo), donc ta query d origine est correct. Mais il devrait aussi accepter les _id de type objet (puisque mongoose n est qu un wrapper de mongo). Cependant chez toi mongoose veut un _id de type string....

As tu essaye ?

Code:

.findByIdAndRemove.(req.params.id)

Peux tu faire voir le Schema de Voiture ?

Quel est ta version de mongoose ?

Bonjour,
Voici le schéma de Voiture :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
 
const mongoose = require('mongoose');
 
const voitureSchema = new mongoose.Schema({
    name:{
      type: String,
      required: [true, 'A voiture must have a Name'],
      unique: true,
      trim: true
    },
    duration:{
     type: Number,
     required: [true, 'A voiture must have a duration']
    },
    maxGroupSize:{
      type: Number,
      required: [true, 'A voiture must have a Group size']
    },
    difficulty:{
       type: String,
       required: [true, 'A voiture must have a difficulty']
    },
    ratingsAverage: {
      type: Number,
      default: 4.5
    },
    ratingsQuantity:{
        type: Number,
        default: 0
    },
    price:{
      type: Number,
      required: [true, 'A voiture must have a price']
    },
    priceDiscount: Number,
    summary:{
        type: String,
        trim: true,
        required: [true, 'A voiture must have a summary']
    },
    description:{
        type: String,
        trim: true
    },
    imageCover:{
        type: String,
        required: [true, 'A voiture must have a cover image']
    },
    images: [String],
    createdAt:{
        type: Date,
        default: Date.now
    },
    startDates: [Date]
  });
 
  const Voiture = mongoose.model('Voiture', voitureSchema);
 
  module.exports = Voiture;

La version mongoose est la suivante dans package.json :

Code:

1
2
3
4
5
6
7
"dependencies": {
    "dotenv": "^7.0.0",
    "express": "^4.16.4",
    "mongodb": "^3.5.7",
    "mongoose": "5",
    "morgan": "^1.9.1"
  },

31/05/2020, 16h05
djedjethai

Ok,

Quel est ta version de node ?

Ds ton premier rapport d err, l'id que mongoose rapport comme ne pouvant pas caster possede 25 char.

Ds le second rapport d err (en passant l id en objet), mongo precise que l id doit posseder 24 char .... n y aurait il pas un char en plus qui aurait ete ajoute par erreur en amont.....
31/05/2020, 20h25
toutemes1

Ma version de node est la suivante : v12.15.0

Bonjour,
Voici ma nouvelle méthode deleteVoiture :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
exports.voitureTour = catchAsync(async (req, res, next) => {
 
  const voitureId = req.params.id;
 
   Voiture.findByIdAndDelete(voitureId)
  .then(() =>{
    res.status(200).json({
         status: 'success',
         data: null
       }); 
  }).catch(err =>console.log(err));
 
 
});

ça fonctionne

01/06/2020, 00h49
djedjethai

Ok, interressant.... retire aussi le 'async' il ne sert plus a rien.