Remove unused files and packages leading to deprecated warning

package.json

@@ -73,7 +73,6 @@
     "graphql-ws": "^5.14.2",
     "helmet": "^7.1.0",
     "i18n": "^0.15.1",
-    "image-hash": "^5.3.1",
     "ioredis": "^5.3.2",
     "jsonwebtoken": "^9.0.0",
     "jwt-decode": "^4.0.0",
@@ -82,9 +81,9 @@
     "mongoose": "^5.13.21",
     "mongoose-paginate-v2": "^1.7.0",
     "morgan": "^1.10.0",
-    "nanoid": "^5.0.5",
+    "nanoid": "^5.0.6",
     "nodemailer": "^6.9.12",
-    "pm2": "^5.2.0",
+    "pm2": "^5.3.1",
     "redis": "^4.6.13",
     "rrule": "^2.8.1",
     "typedoc-plugin-markdown": "^3.17.1",

src/resolvers/Mutation/index.ts

@@ -77,7 +77,6 @@ import { removeUserCustomData } from "./removeUserCustomData";
 import { removeUserFamily } from "./removeUserFamily";
 import { removeUserFromGroupChat } from "./removeUserFromGroupChat";
 import { removeUserFromUserFamily } from "./removeUserFromUserFamily";
-import { removeUserImage } from "./removeUserImage";
 import { removeUserTag } from "./removeUserTag";
 import { resetCommunity } from "./resetCommunity";
 import { revokeRefreshTokenForUser } from "./revokeRefreshTokenForUser";
@@ -202,7 +201,6 @@ export const Mutation: MutationResolvers = {
   removePost,
   removeUserCustomData,
   removeUserFromGroupChat,
-  removeUserImage,
   removeUserTag,
   resetCommunity,
   revokeRefreshTokenForUser,

src/resolvers/Mutation/removeOrganizationImage.ts

@@ -5,13 +5,12 @@ import {
 import type { MutationResolvers } from "../../types/generatedGraphQLTypes";
 import { errors, requestContext } from "../../libraries";
 import { Organization } from "../../models";
-import { adminCheck, deleteImage } from "../../utilities";
+import { adminCheck, deletePreviousImage } from "../../utilities";
 import { findOrganizationsInCache } from "../../services/OrganizationCache/findOrganizationsInCache";
 import { cacheOrganizations } from "../../services/OrganizationCache/cacheOrganizations";
 /**
  * This function enables to remove an organization's image.
  * @param _parent - parent of current request
- * @param args - payload provided with the request
  * @param context - context of entire application
  * @remarks The following checks are done:
  * 1. If the user exists.
@@ -34,7 +33,9 @@ export const removeOrganizationImage: MutationResolvers["removeOrganizationImage
         _id: args.organizationId,
       }).lean();
 
-      await cacheOrganizations([organization!]);
+      if (organization) {
+        await cacheOrganizations([organization]);
+      }
     }
 
     // Checks whether organization exists.
@@ -58,9 +59,10 @@ export const removeOrganizationImage: MutationResolvers["removeOrganizationImage
       );
     }
 
-    await deleteImage(organization.image);
-
     // Sets image field of organization to null and returns the updated organization.
+    if (organization?.image) {
+      await deletePreviousImage(organization?.image);
+    }
     const updatedOrganization = await Organization.findOneAndUpdate(
       {
         _id: organization._id,
@@ -79,5 +81,8 @@ export const removeOrganizationImage: MutationResolvers["removeOrganizationImage
       await cacheOrganizations([updatedOrganization]);
     }
 
-    return updatedOrganization!;
+    if (!updatedOrganization) {
+      throw new Error("Organization could not be updated");
+    }
+    return updatedOrganization;
   };

src/utilities/encodedImageStorage/deletePreviousImage.ts

@@ -5,8 +5,11 @@ import { EncodedImage } from "../../models/EncodedImage";
 export const deletePreviousImage = async (
   imageToBeDeletedPath: string,
 ): Promise<void> => {
+  if (!imageToBeDeletedPath) {
+    throw new Error("imageToBeDeletedPath is null or undefined");
+  }
   const imageToBeDeleted = await EncodedImage.findOne({
-    fileName: imageToBeDeletedPath!,
+    fileName: imageToBeDeletedPath,
   });
 
   if (imageToBeDeleted?.numberOfUses === 1) {

src/utilities/index.ts

@@ -1,8 +1,7 @@
 export * from "./adminCheck";
 export * from "./auth";
 export * from "./copyToClipboard";
-export * from "./deleteImage";
 export * from "./mailer";
 export * from "./superAdminCheck";
-export * from "./uploadImage";
-export * from "./isAuthCheck";
+export * from "./encodedImageStorage/deletePreviousImage";
+export * from "./isAuthCheck";