Merge pull request #2059 from OneCommunityGlobal/SaiKrishna_AddEditDeleteInConsumables

SaiKrishna add Edit Delete Inventory Type Backend

Author: one-community

src/controllers/bmdashboard/bmInventoryTypeController.js

@@ -388,6 +388,145 @@ function bmInventoryTypeController(InvType, MatType, ConsType, ReusType, ToolTyp
       res.status(500).send(error);
     }
   };
+
+  // PUT - Update any inventory type by ID (generic)
+  const updateInventoryType = async (req, res) => {
+    try {
+      const { invtypeId } = req.params;
+      const updateData = req.body;
+
+      // Remove fields that shouldn't be updated directly
+      delete updateData._id;
+      delete updateData.__t;
+      delete updateData.__v;
+
+      const updatedInvType = await InvType.findByIdAndUpdate(invtypeId, updateData, {
+        new: true,
+        runValidators: true,
+      });
+
+      if (!updatedInvType) {
+        return res.status(404).json({ error: 'Inventory type not found' });
+      }
+
+      res.status(200).json(updatedInvType);
+    } catch (error) {
+      res.status(500).json({ error: error.message });
+    }
+  };
+
+  // DELETE - Delete any inventory type by ID
+  const deleteInventoryType = async (req, res) => {
+    try {
+      const { invtypeId } = req.params;
+
+      const deletedInvType = await InvType.findByIdAndDelete(invtypeId);
+
+      if (!deletedInvType) {
+        return res.status(404).json({ error: 'Inventory type not found' });
+      }
+
+      res.status(200).json({ message: 'Inventory type deleted successfully' });
+    } catch (error) {
+      res.status(500).json({ error: error.message });
+    }
+  };
+
+  // POST - Add a new unit to the JSON file
+  const addInventoryUnit = async (req, res) => {
+    try {
+      const { unit, category } = req.body;
+
+      if (!unit) {
+        return res.status(400).json({ error: 'Unit is required' });
+      }
+
+      readFile(filepath, 'utf8', (err, data) => {
+        if (err) {
+          console.error('Error reading file:', err);
+          return res.status(500).json({ error: 'Error reading units file' });
+        }
+
+        try {
+          const jsonData = JSON.parse(data);
+
+          // Check if unit already exists
+          const exists = jsonData.some(
+            (item) => item.unit.toLowerCase() === unit.toLowerCase(),
+          );
+          if (exists) {
+            return res.status(409).json({ error: 'Unit already exists' });
+          }
+
+          // Add new unit
+          const newUnit = { unit, category: category || 'Material' };
+          jsonData.push(newUnit);
+
+          writeFile(filepath, JSON.stringify(jsonData, null, 2), 'utf8', (writeErr) => {
+            if (writeErr) {
+              console.error('Error writing to file:', writeErr);
+              return res.status(500).json({ error: 'Error saving unit' });
+            }
+            res.status(201).json(newUnit);
+          });
+        } catch (parseError) {
+          console.error('Error parsing JSON:', parseError);
+          res.status(500).json({ error: 'Error parsing units file' });
+        }
+      });
+    } catch (error) {
+      res.status(500).json({ error: error.message });
+    }
+  };
+
+  // DELETE - Remove a unit from the JSON file by unit name
+  const deleteInventoryUnit = async (req, res) => {
+    try {
+      const { unitName } = req.params;
+
+      if (!unitName) {
+        return res.status(400).json({ error: 'Unit name is required' });
+      }
+
+      readFile(filepath, 'utf8', (err, data) => {
+        if (err) {
+          console.error('Error reading file:', err);
+          return res.status(500).json({ error: 'Error reading units file' });
+        }
+
+        try {
+          const jsonData = JSON.parse(data);
+
+          // Find index of unit to delete
+          const decodedUnitName = decodeURIComponent(unitName);
+          const unitIndex = jsonData.findIndex(
+            (item) => item.unit.toLowerCase() === decodedUnitName.toLowerCase(),
+          );
+
+          if (unitIndex === -1) {
+            return res.status(404).json({ error: 'Unit not found' });
+          }
+
+          // Remove the unit
+          jsonData.splice(unitIndex, 1);
+
+          writeFile(filepath, JSON.stringify(jsonData, null, 2), 'utf8', (writeErr) => {
+            if (writeErr) {
+              console.error('Error writing to file:', writeErr);
+              return res.status(500).json({ error: 'Error deleting unit' });
+            }
+            res.status(200).json({ message: 'Unit deleted successfully' });
+          });
+        } catch (parseError) {
+          console.error('Error parsing JSON:', parseError);
+          res.status(500).json({ error: 'Error parsing units file' });
+        }
+      });
+    } catch (error) {
+      res.status(500).json({ error: error.message });
+    }
+  };
+
   return {
     fetchMaterialTypes,
     fetchConsumableTypes,
@@ -402,6 +541,10 @@ function bmInventoryTypeController(InvType, MatType, ConsType, ReusType, ToolTyp
     addToolType,
     fetchInvUnitsFromJson,
     fetchInventoryByType,
+    updateInventoryType,
+    deleteInventoryType,
+    addInventoryUnit,
+    deleteInventoryUnit,
   };
 }
 

src/routes/bmdashboard/bmInventoryTypeRouter.js

@@ -30,16 +30,33 @@ const routes = function (baseInvType, matType, consType, reusType, toolType, equ
 
   inventoryTypeRouter.route('/invtypes/consumables').get(controller.fetchConsumableTypes);
 
-  // Route for fetching types by selected type
-  inventoryTypeRouter.route('/invtypes/:type').get(controller.fetchInventoryByType);
-
   // Combined routes for getting a single inventory type and updating its name and unit of measurement
   inventoryTypeRouter
     .route('/invtypes/material/:invtypeId')
     .get(controller.fetchSingleInventoryType)
     .put(controller.updateNameAndUnit);
 
-  inventoryTypeRouter.route('/inventoryUnits').get(controller.fetchInvUnitsFromJson);
+  // Generic route for updating/deleting any inventory type by ID
+  // Using regex to match MongoDB ObjectId format (24 hex characters) - must come BEFORE :type route
+  inventoryTypeRouter
+    .route('/invtypes/:invtypeId([0-9a-fA-F]{24})')
+    .get(controller.fetchSingleInventoryType)
+    .put(controller.updateInventoryType)
+    .delete(controller.deleteInventoryType);
+
+  // Route for fetching types by selected type (Materials, Consumables, etc.)
+  // This comes AFTER the ObjectId route so it only matches non-ObjectId strings
+  inventoryTypeRouter.route('/invtypes/:type').get(controller.fetchInventoryByType);
+
+  // Routes for inventory units (JSON file based)
+  inventoryTypeRouter
+    .route('/inventoryUnits')
+    .get(controller.fetchInvUnitsFromJson)
+    .post(controller.addInventoryUnit);
+
+  inventoryTypeRouter
+    .route('/inventoryUnits/:unitName')
+    .delete(controller.deleteInventoryUnit);
 
   return inventoryTypeRouter;
 };