#9253 Require User when handling announcement image upload

pkp · Nov 8, 2023 · f90977a · f90977a
1 parent 0f04c23
commit f90977a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/classes/announcement/Repository.php b/classes/announcement/Repository.php
@@ -230,7 +230,7 @@ protected function handleImageUpload(Announcement $announcement): void
             $temporaryFileManager = new TemporaryFileManager();
             $temporaryFile = $temporaryFileManager->getFile((int) $image['temporaryFileId'], $user?->getId());
             $filePath = $this->getImageSubdirectory() . '/' . $this->getImageFilename($announcement, $temporaryFile);
-            if ($this->storeTemporaryFile($temporaryFile, $filePath, $user?->getId(), $announcement)) {
+            if ($this->storeTemporaryFile($temporaryFile, $filePath, $user->getId(), $announcement)) {
                 $announcement->setImage(
                     $this->getImageData($announcement, $temporaryFile)
                 );
@@ -248,7 +248,7 @@ protected function handleImageUpload(Announcement $announcement): void
      * @param string $newPath The new filename with the path relative to the public files directoruy
      * @return bool Whether or not the operation was successful
      */
-    protected function storeTemporaryFile(TemporaryFile $temporaryFile, string $newPath, ?int $userId, Announcement $announcement): bool
+    protected function storeTemporaryFile(TemporaryFile $temporaryFile, string $newPath, int $userId, Announcement $announcement): bool
     {
         $publicFileManager = new PublicFileManager();
         $temporaryFileManager = new TemporaryFileManager();