Rename upload request item

resources/js/store/modules/fileFunctions.js

@@ -120,12 +120,14 @@ const actions = {
     },
     renameItem: ({ commit, getters, dispatch }, data) => {
         // Updated name in favourites panel
-        if (getters.permission === 'master' && data.type === 'folder') commit('UPDATE_NAME_IN_FAVOURITES', data)
+        if (getters.permission === 'master' && data.type === 'folder')
+            commit('UPDATE_NAME_IN_FAVOURITES', data)
 
         // Get route
-        let route = getters.sharedDetail
+        let route = {
-            ? `/api/editor/rename/${data.id}/${router.currentRoute.params.token}`
+            RequestUpload: `/api/upload-request/${router.currentRoute.params.token}/rename/${data.id}`,
-            : `/api/rename/${data.id}`
+            Public: `/api/editor/rename/${data.id}/${router.currentRoute.params.token}`,
+        }[router.currentRoute.name] || `/api/rename/${data.id}`
 
         axios
             .post(route, {
@@ -135,9 +137,9 @@ const actions = {
             .then((response) => {
                 commit('CHANGE_ITEM_NAME', response.data)
 
-                if (data.type === 'folder' && !Vue.prototype.$isThisRoute(router.currentRoute, ['Public']))
+                if (data.type === 'folder' && router.currentRoute.name !== 'Public')
                     dispatch('getAppData')
-                if (data.type === 'folder' && Vue.prototype.$isThisRoute(router.currentRoute, ['Public']))
+                if (data.type === 'folder' && router.currentRoute.name === 'Public')
                     dispatch('getFolderTree')
             })
             .catch(() => Vue.prototype.$isSomethingWrong())

routes/upload-request.php

@@ -1,5 +1,6 @@
 <?php
 
+use Tests\Domain\UploadRequest\RenameFileOrFolderController;
 use Domain\UploadRequest\Controllers\GetUploadRequestController;
 use Domain\UploadRequest\Controllers\CreateUploadRequestController;
 use Domain\UploadRequest\Controllers\SetUploadRequestAsFilledController;
@@ -9,6 +10,9 @@ Route::get('/{uploadRequest}', GetUploadRequestController::class);
 Route::delete('/{uploadRequest}', SetUploadRequestAsFilledController::class);
 Route::post('/{uploadRequest}/upload', UploadFilesForUploadRequestController::class);
 
+// Edit
+Route::patch('/{uploadRequest}/rename/{id}', RenameFileOrFolderController::class);
+
 Route::group(['middleware' => ['auth:sanctum']], function () {
     Route::post('/', CreateUploadRequestController::class);
 });

tests/Domain/UploadRequest/RenameFileOrFolderController.php

@@ -0,0 +1,59 @@
+<?php
+namespace Tests\Domain\UploadRequest;
+
+use Illuminate\Support\Arr;
+use Domain\Folders\Models\Folder;
+use App\Http\Controllers\Controller;
+use Domain\Files\Resources\FileResource;
+use Domain\Folders\Resources\FolderResource;
+use Domain\Items\Requests\RenameItemRequest;
+use Domain\UploadRequest\Models\UploadRequest;
+use Domain\Folders\Actions\UpdateFolderPropertyAction;
+use Support\Demo\Actions\FakeRenameFileOrFolderAction;
+
+class RenameFileOrFolderController extends Controller
+{
+    public function __construct(
+        public UpdateFolderPropertyAction $updateFolderProperty,
+        public FakeRenameFileOrFolderAction $fakeRenameFileOrFolder,
+    ) {
+    }
+
+    public function __invoke(UploadRequest $uploadRequest, string $id, RenameItemRequest $request)
+    {
+        // Check if upload request is active
+        if ($uploadRequest->status !== 'active') {
+            return response('Gone', 410);
+        }
+
+        // Get item
+        $item = get_item($request->input('type'), $id);
+
+        // Get folders within upload request
+        $folderWithinIds = Folder::with('folders:id,parent_id')
+            ->where('parent_id', $uploadRequest->id)
+            ->get(['id']);
+
+        // Then get all accessible folders within
+        $accessibleParentIds = Arr::flatten([filter_folders_ids($folderWithinIds), $uploadRequest->id]);
+
+        // Check privileges
+        if (! in_array($item->parent_id, $accessibleParentIds)) {
+            return response('Access Denied', 403);
+        }
+
+        // If request contain icon or color, then change it
+        if ($request->input('type') === 'folder' && $request->hasAny(['emoji', 'color'])) {
+            ($this->updateFolderProperty)($request, $id);
+        }
+
+        // Update item
+        $item->update(['name' => $request->input('name')]);
+
+        if ($request->input('type') === 'folder') {
+            return new FolderResource($item);
+        }
+
+        return new FileResource($item);
+    }
+}

tests/Domain/UploadRequest/UploadRequestEditingTest.php

@@ -0,0 +1,84 @@
+<?php
+namespace Tests\Domain\UploadRequest;
+
+use Tests\TestCase;
+use App\Users\Models\User;
+use Domain\Files\Models\File;
+use Domain\Folders\Models\Folder;
+use Domain\UploadRequest\Models\UploadRequest;
+
+class UploadRequestEditingTest extends TestCase
+{
+    /**
+     * @test
+     */
+    public function it_rename_folder_item()
+    {
+        $user = User::factory()
+            ->hasSettings()
+            ->create();
+
+        $uploadRequest = UploadRequest::factory()
+            ->create([
+                'status'  => 'active',
+                'user_id' => $user->id,
+            ]);
+
+        $folder = Folder::factory()
+            ->create([
+                'parent_id' => $uploadRequest->id,
+                'user_id'   => $user->id,
+            ]);
+
+        $this
+            ->actingAs($user)
+            ->postJson("/api/upload-request/$uploadRequest->id/rename/$folder->id", [
+                'name' => 'Renamed Folder',
+                'type' => 'folder',
+            ])
+            ->assertStatus(200)
+            ->assertJsonFragment([
+                'name' => 'Renamed Folder',
+            ]);
+
+        $this->assertDatabaseHas('folders', [
+            'name' => 'Renamed Folder',
+        ]);
+    }
+    /**
+     * @test
+     */
+    public function it_rename_file_item()
+    {
+        $user = User::factory()
+            ->hasSettings()
+            ->create();
+
+        $uploadRequest = UploadRequest::factory()
+            ->create([
+                'status'  => 'active',
+                'user_id' => $user->id,
+            ]);
+
+        $file = File::factory()
+            ->create([
+                'parent_id' => $uploadRequest->id,
+                'user_id'   => $user->id,
+            ]);
+
+        $this
+            ->actingAs($user)
+            ->postJson("/api/upload-request/$uploadRequest->id/rename/$file->id", [
+                'name' => 'Renamed File',
+                'type' => 'file',
+            ])
+            ->assertStatus(200)
+            ->assertJsonFragment([
+                'name' => 'Renamed File',
+            ]);
+
+        $this->assertDatabaseHas('files', [
+            'name' => 'Renamed File',
+        ]);
+    }
+}

tests/Domain/UploadRequest/UploadRequestTest.php

@@ -1,5 +1,4 @@
 <?php
-
 namespace Tests\Domain\UploadRequest;
 
 use Storage;