Merge pull request #3531 from Northeastern-Electric-Racing/lead-manager-changes

added lead and manager defaults for wps

Author: chpy04

src/backend/src/services/work-packages.services.ts

@@ -34,7 +34,6 @@ import {
 import { getBlockingWorkPackages, validateBlockedBys } from '../utils/work-packages.utils';
 import { getDescriptionBulletQueryArgs } from '../prisma-query-args/description-bullets.query-args';
 import { userHasPermission } from '../utils/users.utils';
-import ProjectsService from './projects.services';
 
 /** Service layer containing logic for work package controller functions. */
 export default class WorkPackagesService {
@@ -173,13 +172,37 @@ export default class WorkPackagesService {
     // and what number work package this should be
     const { carNumber, projectNumber } = projectWbsNum;
 
-    const project = await ProjectsService.getSingleProject(projectWbsNum, organization);
+    const project = await prisma.project.findFirst({
+      where: {
+        wbsElement: {
+          carNumber,
+          projectNumber,
+          organizationId: organization.organizationId,
+          dateDeleted: null
+        }
+      },
+      include: {
+        workPackages: {
+          where: { wbsElement: { dateDeleted: null } },
+          include: {
+            wbsElement: true
+          }
+        },
+        wbsElement: true
+      }
+    });
+
+    if (!project) {
+      throw new NotFoundException('Project', `${carNumber}.${projectNumber}.0`);
+    }
+
+    const { projectId } = project;
 
-    const { id: projectId } = project;
+    const { leadId, managerId } = project.wbsElement;
 
     const newWorkPackageNumber: number =
       project.workPackages
-        .map((element) => element.wbsNum.workPackageNumber)
+        .map((element) => element.wbsElement.workPackageNumber)
         .reduce((prev, curr) => Math.max(prev, curr), 0) + 1;
 
     // make the date object but add 12 hours so that the time isn't 00:00 to avoid timezone problems
@@ -208,7 +231,9 @@ export default class WorkPackagesService {
             changes: {
               createMany: { data: changesToCreate }
             },
-            organizationId: organization.organizationId
+            organizationId: organization.organizationId,
+            leadId,
+            managerId
           }
         },
         stage,
@@ -233,9 +258,9 @@ export default class WorkPackagesService {
       [],
       blockedByElements,
       null,
+      leadId,
       null,
-      null,
-      null,
+      managerId,
       [],
       descriptionBullets,
       crId,

src/backend/src/utils/projects.utils.ts

@@ -70,6 +70,10 @@ export const updateProjectAndCreateChanges = async (
           links: { where: { dateDeleted: null }, ...getLinkQueryArgs(organizationId) },
           descriptionBullets: { where: { dateDeleted: null }, ...getDescriptionBulletQueryArgs(organizationId) }
         }
+      },
+      workPackages: {
+        where: { wbsElement: { dateDeleted: null } },
+        include: { wbsElement: true }
       }
     }
   });
@@ -79,7 +83,7 @@ export const updateProjectAndCreateChanges = async (
   if (originalProject.wbsElement.dateDeleted) throw new DeletedException('Project', projectId);
   if (originalProject.wbsElement.organizationId !== organizationId) throw new InvalidOrganizationException('Project');
 
-  const { wbsElementId } = originalProject;
+  const { wbsElementId, workPackages: originalWorkPackages } = originalProject;
 
   const nameChangeJson = createChange(
     'name',
@@ -158,6 +162,48 @@ export const updateProjectAndCreateChanges = async (
 
   changesJson = changesJson.concat(descriptionBulletChanges.changes).concat(linkChanges.changes);
 
+  // if the project has no managerId and a managerId is provided, we need to update the work packages
+  // that do not have a managerId and create changes for them
+  // this is so that project manager is the default manager for all work packages
+  if (!originalProject.wbsElement.managerId && managerId) {
+    const wpToUpdate = originalWorkPackages.filter((wp) => !wp.wbsElement.managerId);
+
+    const wpChanges = (
+      await Promise.all(
+        wpToUpdate.map(async (wp) =>
+          createChange('manager', null, await getUserFullName(managerId), crId, implementerId, wp.wbsElementId, null, null)
+        )
+      )
+    ).filter((change) => change !== undefined);
+
+    changesJson = changesJson.concat(wpChanges);
+
+    await prisma.wBS_Element.updateMany({
+      where: { wbsElementId: { in: wpToUpdate.map((wp) => wp.wbsElementId) } },
+      data: { managerId }
+    });
+  }
+
+  // same goes for the project lead
+  if (!originalProject.wbsElement.leadId && leadId) {
+    const wpToUpdate = originalWorkPackages.filter((wp) => !wp.wbsElement.leadId);
+
+    const wpChanges = (
+      await Promise.all(
+        wpToUpdate.map(async (wp) =>
+          createChange('lead', null, await getUserFullName(leadId), crId, implementerId, wp.wbsElementId, null, null)
+        )
+      )
+    ).filter((change) => change !== undefined);
+
+    changesJson = changesJson.concat(wpChanges);
+
+    await prisma.wBS_Element.updateMany({
+      where: { wbsElementId: { in: wpToUpdate.map((wp) => wp.wbsElementId) } },
+      data: { leadId }
+    });
+  }
+
   // update the project with the input fields
   const updatedProject = await prisma.project.update({
     where: {