When checking the checkbox with a /POST method to the server, it passes the item's ID back to the script.

Then I try to call the findByIdAndRemove() function but it doesn't seem to actually delete the document.

I'm checking with Mongosh (MongoDB new Shell).

NODE:

app.post('/delete', function(req, res){

  const checkedItem = req.body.checked;

  Item.findByIdAndRemove(checked);
  
});

EJS:

<% newListItems.forEach(function(item){ %>

      <form action="/delete" method="post">

        <div class="item">
          <input type="checkbox" onChange="this.form.submit()" name='checked' value='<%= item._id %>'>
          <p> <%=item.name%> </p>
        </div>

      </form>


    <% }); %>

'Test' should be deleted when checked: .png

But 'Test' is still intact: .png

With the most recent Mongoose upgrade to version 7.0, they removed callbacks and some methods stoped working the way they did.

I've tried reading their docs about async and await, using .then and .catch and found no help on the open web since the update is recent.

When checking the checkbox with a /POST method to the server, it passes the item's ID back to the script.

Then I try to call the findByIdAndRemove() function but it doesn't seem to actually delete the document.

I'm checking with Mongosh (MongoDB new Shell).

NODE:

app.post('/delete', function(req, res){

  const checkedItem = req.body.checked;

  Item.findByIdAndRemove(checked);
  
});

EJS:

<% newListItems.forEach(function(item){ %>

      <form action="/delete" method="post">

        <div class="item">
          <input type="checkbox" onChange="this.form.submit()" name='checked' value='<%= item._id %>'>
          <p> <%=item.name%> </p>
        </div>

      </form>


    <% }); %>

'Test' should be deleted when checked: https://i.sstatic/ujL7Z.png

But 'Test' is still intact: https://i.sstatic/vomkW.png

With the most recent Mongoose upgrade to version 7.0, they removed callbacks and some methods stoped working the way they did.

I've tried reading their docs about async and await, using .then and .catch and found no help on the open web since the update is recent.

• javascript
• node.js
• mongodb
• mongoose
• ejs

• Please include code in the question as formatted text, not as images. meta.stackoverflow./q/285551/2282634 – Joe Commented Mar 22, 2023 at 22:44
• You should use findOneAndDelete() unless you have a good reason not to – Phil Commented Mar 22, 2023 at 22:55
• 1) You should wait for the delete operation to plete. 2) Have you debugged the req.body.checked value to make sure it's correct? – Phil Commented Mar 22, 2023 at 22:56
• Thanks Joe and Phil for the tips. Yes Phil, I have debugged the query, it works fine. This post: stackoverflow./questions/75586474/… helped me, using try, catch and await and async the right way. – Henry Commented Mar 23, 2023 at 16:46

3 Answers 3

Check your mongoose version:

npm list mongoose

As of version 8 there is no more .findByIdAndRemove() in its place you will have to use .findByIdAndDelete()

https://mongoosejs./docs/api/model.html

I'm also facing the same problem so, I put everything under async function inside post request.

app.js

app.post("/delete", function(req,res){

async function myDelete(){
  const checkedItemId = req.body.checkbox;

  const del = await Item.findByIdAndRemove(checkedItemId);

};
myDelete();
res.redirect("/");
});

Use the following snippet of code instead

await Prompt.findByIdAndDelete({ _id: params.id });