From aed29200a73f292dbdb220ab509c336b588c0193 Mon Sep 17 00:00:00 2001
From: James Collins <james.collins@outlook.com.au>
Date: Tue, 23 Apr 2024 10:21:14 +1000
Subject: [PATCH] fix incrementing and automatic titling

---
 app/Http/Controllers/MediaController.php | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/app/Http/Controllers/MediaController.php b/app/Http/Controllers/MediaController.php
index 06a3271..a71179c 100644
--- a/app/Http/Controllers/MediaController.php
+++ b/app/Http/Controllers/MediaController.php
@@ -149,16 +149,18 @@ class MediaController extends Controller
 
         $file = $request->file('file');
 
-        $name = $file->getClientOriginalName();
+        $fileName = $file->getClientOriginalName();
+        $name = pathinfo($fileName, PATHINFO_FILENAME);
+        $extension = pathinfo($fileName, PATHINFO_EXTENSION);
         $name = Helpers::cleanFileName($name);
 
-        if(Media::find($name) !== null) {
+        if(Media::find($name . '.' . $extension) !== null) {
             $increment = 2;
-            while(Media::find($name . '-' . $increment) !== null) {
+            while(Media::find($name . '-' . $increment . '.' . $extension) !== null) {
                 $increment++;
             }
 
-            $name = $name . '-' . $increment;
+            $fileName = $name . '-' . $increment . '.' . $extension;
         }
 
         $hash = hash_file('sha256', $file->path());
@@ -180,9 +182,9 @@ class MediaController extends Controller
         }
 
         $media = Media::Create([
-            'title' => $request->get('title', $name),
+            'title' => $request->get('title', Helpers::filenameToTitle($fileName)),
             'user_id' => auth()->id(),
-            'name' => $name,
+            'name' => $fileName,
             'size' => $file->getSize(),
             'mime_type' => $file->getMimeType(),
             'hash' => $hash