fix: use createdBy along with redactedBy to allow user to edit a report

betagouv · Sep 23, 2024 · c236b43 · c236b43
1 parent 334dce3
commit c236b43
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/packages/frontend/src/features/ReportActions.tsx b/packages/frontend/src/features/ReportActions.tsx
@@ -32,8 +32,11 @@ export const ReportActions = forwardRef<HTMLDivElement, { report: ReportWithUser
       data: {
         ...payload,
         id: `report-${v4()}`,
-        title: `${report.title} - copie`,
+        title: `${report.title ?? "Sans titre"} - copie`,
         createdAt: new Date(),
+        redactedBy: user.name,
+        redactedById: user.id,
+        createdBy: user.id,
         pdf: undefined,
       },
     });

diff --git a/packages/frontend/src/routes/edit.$reportId.tsx b/packages/frontend/src/routes/edit.$reportId.tsx
@@ -74,13 +74,14 @@ const WithReport = ({ report }: { report: Report }) => {
 
   const user = useUser()!;
   const isOwner = report.redactedById === user.id;
+  const isCreator = report.createdBy === user.id;
 
   const userDelegations = useLiveQuery(
     db.delegation.liveFirst({ where: { createdBy: report.createdBy, delegatedTo: user.id } }),
   );
 
   const hasDelegation = !!userDelegations.results;
-  const canEdit = isOwner || hasDelegation;
+  const canEdit = isOwner || isCreator || hasDelegation;
 
   const navigate = useNavigate();