Discussion :

[toutemes1]

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

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

La methode est la suivante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

.findByIdAndRemove(id);

[toutemes1]

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

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)

[djedjethai]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

.findByIdAndRemove.(req.params.id)

Peux tu faire voir le Schema de Voiture ?

Quel est ta version de mongoose ?

[toutemes1]

Bonjour,
Voici le schéma de Voiture :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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"
  },

[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.....

[toutemes1]

Ma version de node est la suivante : v12.15.0

[toutemes1]

Bonjour,
Voici ma nouvelle méthode deleteVoiture :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[djedjethai]

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