#4102 upload sponsor endpoint

Author: staysgt

src/backend/src/controllers/finance.controllers.ts

@@ -1,5 +1,6 @@
 import { NextFunction, Request, Response } from 'express';
 import FinanceServices from '../services/finance.services.js';
+import { HttpException } from '../utils/errors.utils.js';
 
 export default class FinanceController {
   static async createSponsor(req: Request, res: Response, next: NextFunction) {
@@ -43,8 +44,7 @@ export default class FinanceController {
         contactPhone,
         contactPosition,
         stockDescription,
-        discountDescription,
-        req.file
+        discountDescription
       );
       res.status(200).json(sponsor);
     } catch (error: unknown) {
@@ -384,8 +384,7 @@ export default class FinanceController {
         contactPhone,
         contactPosition,
         stockDescription,
-        discountDescription,
-        req.file
+        discountDescription
       );
 
       res.status(200).json(updatedSponsor);
@@ -432,4 +431,15 @@ export default class FinanceController {
       next(error);
     }
   }
+
+  static async uploadSponsorLogo(req: Request, res: Response, next: NextFunction) {
+    try {
+      const { sponsorId } = req.params as Record<string, string>;
+      if (!req.file) throw new HttpException(400, 'Invalid or undefined image data');
+      const updatedSponsor = await FinanceServices.uploadSponsorLogo(req.currentUser, req.organization, sponsorId, req.file);
+      res.status(200).json(updatedSponsor);
+    } catch (error: unknown) {
+      next(error);
+    }
+  }
 }

src/backend/src/routes/finance.routes.ts

@@ -17,7 +17,6 @@ const upload = multer({ limits: { fileSize: MAX_FILE_SIZE }, storage: memoryStor
 
 financeRouter.post(
   '/sponsor/create',
-  upload.single('logoImage'),
   nonEmptyString(body('name')),
   body('activeStatus').isBoolean(),
   body('valueTypes').isArray(),
@@ -52,6 +51,8 @@ financeRouter.get('/sponsor/:sponsorId/sponsorTasks', FinanceController.getSpons
 
 financeRouter.post('/sponsor/:sponsorId/delete', FinanceController.deleteSponsor);
 
+financeRouter.post('/sponsor/:sponsorId/uploadLogo', upload.single('logoImage'), FinanceController.uploadSponsorLogo);
+
 financeRouter.post(
   '/sponsorTier/create',
   nonEmptyString(body('name')),
@@ -146,7 +147,6 @@ financeRouter.get('/sponsorTiers', FinanceController.getAllSponsorTiers);
 
 financeRouter.post(
   '/sponsor/:sponsorId/edit',
-  upload.single('logoImage'),
   nonEmptyString(body('name')),
   body('activeStatus').isBoolean(),
   body('valueTypes').isArray(),

src/backend/src/services/finance.services.ts

@@ -81,8 +81,7 @@ export default class FinanceServices {
     contactPhone?: string,
     contactPosition?: string,
     stockDescription?: string,
-    discountDescription?: string,
-    logoImage?: Express.Multer.File
+    discountDescription?: string
   ) {
     if (!(await userHasPermission(submitter.userId, organization.organizationId, isHead)))
       throw new AccessDeniedException('Only heads can create a sponsor');
@@ -106,8 +105,6 @@ export default class FinanceServices {
       data: { name: contactName, email: contactEmail, phone: contactPhone, position: contactPosition }
     });
 
-    const { id: logoImageId } = logoImage ? await uploadFile(logoImage) : { id: undefined };
-
     const sponsor = await prisma.sponsor.create({
       data: {
         name,
@@ -122,7 +119,6 @@ export default class FinanceServices {
         taxExempt,
         discountCode,
         sponsorNotes,
-        logoImageId,
         contactId: contact.sponsorContactId,
         sponsorTasks: {
           create: sponsorTasks.map((task) => ({
@@ -1230,8 +1226,7 @@ export default class FinanceServices {
     contactPhone?: string,
     contactPosition?: string,
     stockDescription?: string,
-    discountDescription?: string,
-    logoImage?: Express.Multer.File
+    discountDescription?: string
   ): Promise<Sponsor> {
     if (!(await userHasPermission(submitter.userId, organization.organizationId, isHead)))
       throw new AccessDeniedException('Only heads can edit sponsors.');
@@ -1329,8 +1324,6 @@ export default class FinanceServices {
       data: { name: contactName, email: contactEmail, phone: contactPhone, position: contactPosition }
     });
 
-    const { id: logoImageId } = logoImage ? await uploadFile(logoImage) : { id: undefined };
-
     const updatedSponsor = await prisma.sponsor.update({
       where: { sponsorId: oldSponsor.sponsorId },
       data: {
@@ -1345,15 +1338,49 @@ export default class FinanceServices {
         tier: sponsorTierId ? { connect: { sponsorTierId } } : { disconnect: true },
         taxExempt,
         discountCode,
-        sponsorNotes,
-        ...(logoImageId && { logoImageId })
+        sponsorNotes
       },
       ...getSponsorQueryArgs(organization.organizationId)
     });
 
     return sponsorTransformer(updatedSponsor);
   }
 
+  /**
+   * Uploads a logo image for a sponsor and stores the resulting image ID.
+   *
+   * @param submitter The user performing the upload
+   * @param organization The organization the sponsor belongs to
+   * @param sponsorId The id of the sponsor
+   * @param logoImage The logo image file to upload
+   * @returns The updated sponsor
+   */
+  static async uploadSponsorLogo(
+    submitter: User,
+    organization: Organization,
+    sponsorId: string,
+    logoImage: Express.Multer.File
+  ): Promise<Sponsor> {
+    if (!(await userHasPermission(submitter.userId, organization.organizationId, isHead)))
+      throw new AccessDeniedException('Only heads can update a sponsor logo');
+
+    const sponsor = await prisma.sponsor.findUnique({
+      where: { sponsorId, organizationId: organization.organizationId }
+    });
+
+    if (!sponsor) throw new NotFoundException('Sponsor', sponsorId);
+
+    const { id } = await uploadFile(logoImage);
+
+    const updatedSponsor = await prisma.sponsor.update({
+      where: { sponsorId },
+      data: { logoImageId: id },
+      ...getSponsorQueryArgs(organization.organizationId)
+    });
+
+    return sponsorTransformer(updatedSponsor);
+  }
+
   /**
    * Toggles the done status of a sponsor task
    * @param submitter The user toggling the task