Update patrons now uses request parameter for id

allisonglittle · Nov 30, 2021 · 852442d · 852442d
1 parent b3eb570
commit 852442d
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/library-express/patrons.js b/library-express/patrons.js
@@ -143,7 +143,7 @@ module.exports = function () {
         var mysql = req.app.get('mysql');
         console.log(req.body)
         var sql = "UPDATE Patrons SET firstName = ?, lastName = ?, registerDate = ?, contactPhone = ?, contactEmail = ?, favoriteTitle = ? WHERE memberID = ?";
-        var inserts = [req.body.firstName, req.body.lastName, req.body.registerDate, req.body.contactPhone, req.body.contactEmail, req.body.favoriteTitle, req.body.id];
+        var inserts = [req.body.firstName, req.body.lastName, req.body.registerDate, req.body.contactPhone, req.body.contactEmail, req.body.favoriteTitle, req.params.id];
         sql = mysql.pool.query(sql, inserts, function (error, results, fields) {
             if (error) {
                 console.log(JSON.stringify(error));

diff --git a/library-express/views/update_patrons.handlebars b/library-express/views/update_patrons.handlebars
@@ -1,7 +1,6 @@
 <main>
     <h1>Update Patron</h1>
     <form id="updatePatron" action="/patrons/{{patron.id}}" method="post">
-        <input type="hidden" id="id" name="id" value={{patron.id}}>
         <p><label for="firstName">First Name:</label><br>
             <input type="text" name="firstName" value="{{patron.firstName}}" required>
         </p>