CHG: changed logic

routes/profileUpdate.js

@@ -8,6 +8,7 @@ const fs = require('fs');
 const router = express.Router();
 const User = require('../Database/models/user');
 const { successResponse, failedResponse } = require("../utils/responseModel");
+const { Types } = require('mongoose'); // Import Mongoose Types
 
 
 const storage = multer.diskStorage({
@@ -191,16 +192,15 @@ router.get("/:userId/addresses", asyncHandler(async (req, res) => {
     res.status(200).json(successResponse("Success!", user.addresses));
 }));
 
-// Update address
 router.put("/:userId/addresses/:addressId", asyncHandler(async (req, res) => {
     try {
-        const addressId = req.params.addressId.trim(); // Trim whitespace
+        const addressId = new Types.ObjectId(req.params.addressId); // Convert to ObjectId
         console.log("Request Address ID:", addressId);
 
         const updatedUser = await User.findOneAndUpdate(
             {
                 unique_id: req.params.userId,
-                "addresses._id": addressId // Compare as string
+                "addresses._id": addressId // Use the converted ObjectId
             },
             {
                 $set: {
@@ -221,12 +221,12 @@ router.put("/:userId/addresses/:addressId", asyncHandler(async (req, res) => {
             console.log(`Address ${index}:`, {
                 _id: a._id,
                 addressId: addressId,
-                match: a._id === addressId
+                match: a._id.equals(addressId) // Use .equals() for comparison
             });
         });
 
-        // Find the updated address by comparing _id as a string
-        const updatedAddress = updatedUser.addresses.find(a => a._id.toString().trim() === addressId.toString().trim());
+        // Find the updated address by comparing _id as ObjectId
+        const updatedAddress = updatedUser.addresses.find(a => a._id.equals(addressId));
 
         if (!updatedAddress) {
             console.log("Addresses Array:", updatedUser.addresses); // Log the addresses array