Bladeren bron

Change order of ingredient routes.
Change order of functions in ingredientData to reflect routes.

Lee Morgan 5 jaren geleden
bovenliggende
commit
0421782003
2 gewijzigde bestanden met toevoegingen van 31 en 31 verwijderingen
  1. 30 30
      controllers/ingredientData.js
  2. 1 1
      routes.js

+ 30 - 30
controllers/ingredientData.js

@@ -170,35 +170,6 @@ module.exports = {
             });
     },
 
-    //DELETE - Removes an ingredient from the merchant's inventory
-    removeIngredient: function(req, res){
-        if(!req.session.user){
-            req.session.error = "MUST BE LOGGED IN TO DO THAT";
-            return res.redirect("/");
-        }
-
-        Merchant.findOne({_id: req.session.user})
-            .then((merchant)=>{
-                for(let i = 0; i < merchant.inventory.length; i++){
-                    if(req.params.id === merchant.inventory[i].ingredient._id.toString()){
-                        merchant.inventory.splice(i, 1);
-                        break;
-                    }
-                }
-
-                return merchant.save()
-            })
-            .then((merchant)=>{
-                return Ingredient.deleteOne({_id: req.params.id});
-            })
-            .then((ingredient)=>{
-                return res.json({});
-            })
-            .catch((err)=>{
-                return res.json("ERROR: UNABLE TO RETRIEVE USER DATA");
-            });
-    },
-
     createFromSpreadsheet: function(req, res){
         if(!req.session.user){
             req.session.error = "MUST BE LOGGED IN TO DO THAT";
@@ -305,5 +276,34 @@ module.exports = {
             .catch((err)=>{
                 return "ERROR: UNABLE TO CREATE YOUR INGREDIENTS";
             });
-    }
+    },
+
+    //DELETE - Removes an ingredient from the merchant's inventory
+    removeIngredient: function(req, res){
+        if(!req.session.user){
+            req.session.error = "MUST BE LOGGED IN TO DO THAT";
+            return res.redirect("/");
+        }
+
+        Merchant.findOne({_id: req.session.user})
+            .then((merchant)=>{
+                for(let i = 0; i < merchant.inventory.length; i++){
+                    if(req.params.id === merchant.inventory[i].ingredient._id.toString()){
+                        merchant.inventory.splice(i, 1);
+                        break;
+                    }
+                }
+
+                return merchant.save()
+            })
+            .then((merchant)=>{
+                return Ingredient.deleteOne({_id: req.params.id});
+            })
+            .then((ingredient)=>{
+                return res.json({});
+            })
+            .catch((err)=>{
+                return res.json("ERROR: UNABLE TO RETRIEVE USER DATA");
+            });
+    },
 }

+ 1 - 1
routes.js

@@ -29,8 +29,8 @@ module.exports = function(app){
     app.get("/ingredients", ingredientData.getIngredients);
     app.post("/ingredients/create", ingredientData.createIngredient);  //also adds to merchant
     app.put("/ingredients/update", ingredientData.updateIngredient);
-    app.delete("/ingredients/remove/:id", ingredientData.removeIngredient);
     app.post("/ingredients/create/spreadsheet", upload.single("ingredients"), ingredientData.createFromSpreadsheet);
+    app.delete("/ingredients/remove/:id", ingredientData.removeIngredient);
     
 
     //Recipes