fix: correct _ratings→_projects, add NotFoundError, fix imports/types in project-controller

Agent-Logs-Url: https://github.com/hackaburg/tilt/sessions/b49199e5-4de9-427d-ab30-cee6555e2094

Co-authored-by: sezanzeb <28510156+sezanzeb@users.noreply.github.com>

Author: Copilot

backend/src/controllers/project-controller.ts

@@ -1,7 +1,10 @@
-import { Authorized, Delete, JsonController, NotFoundError } from "routing-controllers";
+import { Authorized, Delete, JsonController, NotFoundError, Put, Param, Body, CurrentUser } from "routing-controllers";
 import { Inject } from "typedi";
 import { UserRole } from "../entities/user-role";
 import { IProjectService, ProjectServiceToken } from "../services/project-service";
+import { ProjectDTO, convertBetweenEntityAndDTO } from "./dto";
+import { Project } from "../entities/project";
+import { User } from "../entities/user";
 
 // TODO for every team, add a new project automatically with the correct teamId
 
@@ -19,13 +22,17 @@ export class ProjectController {
   public async updateProject(
     @Param("id") projectId: number,
     @Body() { data: projectDTO }: { data: ProjectDTO },
-  ): Promise<TeamDTO> {
+    @CurrentUser() user: User,
+  ): Promise<ProjectDTO> {
     // TODO ProjectUpdateDTO?
-    const project = convertBetweenEntityAndDTO(projectDTO, Project);
+    const existing = await this._projects.getProjectByID(projectId);
 
-    // TODO how to make actual not found errors for incorrect ids?
+    if (existing == null) {
+      throw new NotFoundError();
+    }
 
-    const updateProject = await this._ratings.updateProject(project, user);
-    return convertBetweenEntityAndDTO(updateProject, ProjectDTO);
+    const project = convertBetweenEntityAndDTO(projectDTO, Project);
+    const updatedProject = await this._projects.updateProject(project, user);
+    return convertBetweenEntityAndDTO(updatedProject, ProjectDTO);
   }
 }