NodeJS i MongoDB - obsługa usuwania elementu

Question

Chcę usunąć element z mongodb. Pojawia się następujący błąd i nie mam pojęcia co może być nie tak:

router.delete('/library/:id', (req, res) => {
  console.log(req.params.id);
  Book.deleteOne({ _id: req.params.id }, (err) => {
    if(err) console.log(err);
  });
  res.redirect('library');
});

<form action="/library/<%= book._id %>" method="DELETE" class="delete-btn-form">
                <div class="delete-btn"></div>
</form>

Id jest poprawnie wyświetlane w konsoli w 2 inii.

Comments

Wstaw proszę wyraźniejszy screen lub pokaż te błędy w formie tekstu.

---

GET http://localhost:5000/library/5ed9f56036f81d215c4ea5cd? 404 (Not Found).

Dwa poniższe błędy odnoszą się do innego zagadnienia (ładowania zdjęcia).

---

Czy serwer na pewno nasłuchuje na porcie 5000 i jest uruchomiony? Jeśli korzystasz z expressa, to wydaje mi się, że metoda delete nasłuchuje na request właśnie DELETE, a nie GET.

Answer 1

Jeśli router nasłuchuje na metodę DELETE, to nie obsłuży formularza wysłanego metodą GET (bo ta metoda jest używana, jeśli formularz ma atrybut [method] o nieznanej wartości). NIestety, z poziomu przeglądarki takie żądanie da się wysłać tylko Ajaksem, formularze nie mają takiej możliwości.

Comments

Jak mam to zrobi używając metody post? Zrobiłem coś takiego i w konsoli poprawnie wyświetla się "Delete route", a strona przeładowuje się bez błędów. Problem w tym że dokument nie jest usuwany z bazy.

router.delete('/delete/:id', (req, res) => {
  console.log('Delete route');
  Book.findOneAndDelete({ _id: req.params.id }, (err) => {
    if (err) console.warn(err);
  });
    res.redirect('/library');
});

<form action="/delete/<%= book._id %>?_method=DELETE" method="POST" class="delete-btn-form">
                <div class="delete-btn"></div>
</form>

 

---

Hmm, tak szczerze to nie powinno się wyświetlać "delete route", bo router nie powinien nawet przechwytywać tego żądania.

Dodatkowo robisz przekierowanie zanim książka zostanie usunięta z bazy. Powinieneś robić przekierowanie wewnątrz callbacku:

router.delete('/delete/:id', (req, res) => {
  console.log('Delete route');
  Book.findOneAndDelete({ _id: req.params.id }, (err) => {
    if (err) console.warn(err);
    res.redirect('/library');
  });
});

 

---

Faktycznie nie zauważyłem przekierowania w złym miejscu, jednak nadal to nie pomaga. Dlaczego router nie powinien przechwycić żądania? Używam method-override i rozumiem, że podczas wykonywania metody post z tagu form, zostaje ona nadpisana i express obsługuje delete.

---

Faktycznie, jest taki mechanizm.

Więc jedyne, co mi przychodzi do głowy, to złe id przekazywane w URL-u.

---

Id wypisane do konsoli jest identyczne jak w bazie danych. No nic będę dalej szukał błędu. Dzięki za poświęcony czas.