Merge pull request #18 from techenby/amn/item-photo

techenby · web-flow · commit f3533d11de43 · 2026-03-15T11:55:31.000-05:00
Add photos to items
diff --git a/app/Actions/Inventory/CreateItem.php b/app/Actions/Inventory/CreateItem.php
@@ -0,0 +1,34 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Actions\Inventory;
+
+use App\Models\Item;
+use App\Models\Team;
+use Illuminate\Http\UploadedFile;
+use Illuminate\Support\Arr;
+use Illuminate\Support\Str;
+
+class CreateItem
+{
+    /**
+     * @param  array<string, mixed>  $data
+     */
+    public function handle(Team $team, array $data): Item
+    {
+        $photo = Arr::pull($data, 'photo');
+
+        $item = $team->items()->create($data);
+
+        if ($photo instanceof UploadedFile) {
+            $filename = Str::slug($item->name) . '-' . $item->id . '.' . $photo->getClientOriginalExtension();
+
+            $path = $photo->storeAs("teams/{$team->id}/items", $filename);
+
+            $item->update(['photo_path' => $path]);
+        }
+
+        return $item;
+    }
+}
diff --git a/app/Actions/Inventory/UpdateItem.php b/app/Actions/Inventory/UpdateItem.php
@@ -0,0 +1,44 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Actions\Inventory;
+
+use App\Models\Item;
+use Illuminate\Http\UploadedFile;
+use Illuminate\Support\Arr;
+use Illuminate\Support\Facades\Storage;
+use Illuminate\Support\Str;
+
+class UpdateItem
+{
+    /**
+     * @param  array<string, mixed>  $data
+     */
+    public function handle(Item $item, array $data, bool $removePhoto = false): Item
+    {
+        $photo = Arr::pull($data, 'photo');
+
+        $item->update($data);
+
+        if ($removePhoto && ! $photo instanceof UploadedFile) {
+            if ($item->photo_path) {
+                Storage::delete($item->photo_path);
+            }
+
+            $item->update(['photo_path' => null]);
+        } elseif ($photo instanceof UploadedFile) {
+            if ($item->photo_path) {
+                Storage::delete($item->photo_path);
+            }
+
+            $filename = Str::slug($item->name) . '-' . $item->id . '.' . $photo->getClientOriginalExtension();
+
+            $path = $photo->storeAs("teams/{$item->team_id}/items", $filename);
+
+            $item->update(['photo_path' => $path]);
+        }
+
+        return $item;
+    }
+}
diff --git a/app/Livewire/Forms/Inventory/ItemForm.php b/app/Livewire/Forms/Inventory/ItemForm.php
@@ -4,10 +4,14 @@
 
 namespace App\Livewire\Forms\Inventory;
 
+use App\Actions\Inventory\CreateItem;
+use App\Actions\Inventory\UpdateItem;
 use App\Enums\ItemType;
 use App\Models\Item;
 use Illuminate\Support\Facades\Auth;
+use Illuminate\Support\Facades\Storage;
 use Illuminate\Validation\Rule;
+use Livewire\Features\SupportFileUploads\TemporaryUploadedFile;
 use Livewire\Form;
 
 class ItemForm extends Form
@@ -23,6 +27,12 @@ class ItemForm extends Form
     /** @var array<int, array{key: string, value: string}> */
     public array $metadata = [];
 
+    public ?TemporaryUploadedFile $photo = null;
+
+    public ?string $existingPhotoUrl = null;
+
+    public bool $removePhoto = false;
+
     public function load(Item $item): void
     {
         $metadata = collect($item->metadata ?? [])
@@ -36,6 +46,7 @@ public function load(Item $item): void
             'type' => $item->type->value,
             'parent_id' => $item->parent_id,
             'metadata' => $metadata,
+            'existingPhotoUrl' => $item->photo_path ? Storage::temporaryUrl($item->photo_path, now()->addMinutes(30)) : null,
         ]);
     }
 
@@ -60,9 +71,9 @@ public function save(): void
             ->all() ?: null;
 
         if ($this->editingItem) {
-            $this->editingItem->update($data);
+            (new UpdateItem)->handle($this->editingItem, $data, $this->removePhoto);
         } else {
-            Auth::user()->currentTeam->items()->create($data);
+            (new CreateItem)->handle(Auth::user()->currentTeam, $data);
         }
 
         $this->reset();
@@ -78,6 +89,7 @@ protected function rules(): array
             'metadata' => ['nullable', 'array'],
             'metadata.*.key' => ['nullable', 'string', 'max:255', 'distinct'],
             'metadata.*.value' => ['required_with:metadata.*.key', 'nullable', 'string', 'max:255'],
+            'photo' => ['nullable', 'image', 'max:5120'],
         ];
     }
 }
diff --git a/app/Models/Item.php b/app/Models/Item.php
@@ -24,6 +24,7 @@ class Item extends Model
         'parent_id',
         'type',
         'name',
+        'photo_path',
         'metadata',
     ];
 
diff --git a/database/migrations/2026_03_15_043252_add_photo_path_to_items_table.php b/database/migrations/2026_03_15_043252_add_photo_path_to_items_table.php
@@ -0,0 +1,22 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+return new class extends Migration
+{
+    public function up(): void
+    {
+        Schema::table('items', function (Blueprint $table) {
+            $table->string('photo_path')->nullable()->after('name');
+        });
+    }
+
+    public function down(): void
+    {
+        Schema::table('items', function (Blueprint $table) {
+            $table->dropColumn('photo_path');
+        });
+    }
+};
diff --git a/resources/views/pages/inventory/modals/item-form.blade.php b/resources/views/pages/inventory/modals/item-form.blade.php
@@ -4,6 +4,31 @@
 
         <flux:input wire:model="form.name" :label="__('Name')" type="text" required />
 
+        <flux:field>
+            <flux:label>{{ __('Photo') }}</flux:label>
+
+            <flux:file-upload wire:model="form.photo">
+                <flux:file-upload.dropzone :heading="__('Drop photo here or click to browse')" :text="__('JPG, PNG up to 5MB')" inline />
+            </flux:file-upload>
+
+            <div class="mt-4 flex flex-col gap-2">
+            @if ($this->form->photo)
+                <flux:file-item :heading="$this->form->photo->getClientOriginalName()" :image="$this->form->photo->isPreviewable() ? $this->form->photo->temporaryUrl() : null" :size="$this->form->photo->getSize()">
+                    <x-slot name="actions">
+                        <flux:button wire:click="removePhoto" variant="ghost" size="sm" icon="x-mark" />
+                    </x-slot>
+                </flux:file-item>
+            @elseif ($this->form->existingPhotoUrl)
+                <flux:file-item :heading="basename($this->form->editingItem->photo_path)" :image="$this->form->existingPhotoUrl">
+                    <x-slot name="actions">
+                        <flux:button wire:click="removePhoto" variant="ghost" size="sm" icon="x-mark" />
+                    </x-slot>
+                </flux:file-item>
+            @endif
+            </div>
+            <flux:error name="form.photo" />
+        </flux:field>
+
         <flux:select wire:model="form.type" :label="__('Type')" placeholder="Select type" variant="listbox" searchable>
             @foreach (\App\Enums\ItemType::cases() as $type)
                 <flux:select.option :value="$type->value">{{ ucfirst($type->value) }}</flux:select.option>
diff --git a/resources/views/pages/inventory/⚡index/index.php b/resources/views/pages/inventory/⚡index/index.php
@@ -123,6 +123,17 @@ public function removeMetadata(int $index): void
         $this->form->removeMetadata($index);
     }
 
+    public function removePhoto(): void
+    {
+        if ($this->form->photo) {
+            $this->form->photo->delete();
+            $this->form->photo = null;
+        } elseif ($this->form->existingPhotoUrl) {
+            $this->form->existingPhotoUrl = null;
+            $this->form->removePhoto = true;
+        }
+    }
+
     public function save(): void
     {
         $this->form->save();
diff --git a/resources/views/pages/inventory/⚡index/index.test.php b/resources/views/pages/inventory/⚡index/index.test.php
@@ -5,6 +5,7 @@
 use App\Models\User;
 use Illuminate\Database\Eloquent\ModelNotFoundException;
 use Illuminate\Http\UploadedFile;
+use Illuminate\Support\Facades\Storage;
 use Livewire\Livewire;
 
 test('guests are redirected to the login page', function () {
@@ -392,6 +393,92 @@
     });
 });
 
+describe('can manage item photos', function () {
+    test('can create an item with a photo', function () {
+        Storage::fake();
+        $user = User::factory()->withTeam()->create();
+        $photo = UploadedFile::fake()->image('guitar.jpg');
+
+        Livewire::actingAs($user)
+            ->test('pages::inventory.index')
+            ->set('form.name', 'Guitar')
+            ->set('form.type', ItemType::Item->value)
+            ->set('form.photo', $photo)
+            ->call('save')
+            ->assertHasNoErrors();
+
+        $item = Item::where('name', 'Guitar')->first();
+
+        expect($item->photo_path)->not->toBeNull();
+        Storage::assertExists($item->photo_path);
+    });
+
+    test('can create an item without a photo', function () {
+        $user = User::factory()->withTeam()->create();
+
+        Livewire::actingAs($user)
+            ->test('pages::inventory.index')
+            ->set('form.name', 'Screwdriver')
+            ->set('form.type', ItemType::Item->value)
+            ->call('save')
+            ->assertHasNoErrors();
+
+        expect(Item::where('name', 'Screwdriver')->first()->photo_path)->toBeNull();
+    });
+
+    test('can update an item with a new photo', function () {
+        Storage::fake();
+        $user = User::factory()->withTeam()->create();
+        $item = Item::factory()->for($user->currentTeam)->create(['name' => 'Guitar', 'photo_path' => 'teams/1/items/guitar.jpg']);
+        Storage::put('teams/1/items/guitar.jpg', 'old');
+
+        $newPhoto = UploadedFile::fake()->image('new-guitar.png');
+
+        Livewire::actingAs($user)
+            ->test('pages::inventory.index')
+            ->call('edit', $item->id)
+            ->set('form.photo', $newPhoto)
+            ->call('save')
+            ->assertHasNoErrors();
+
+        $item->refresh();
+
+        expect($item->photo_path)->not->toBeNull();
+        Storage::assertExists($item->photo_path);
+        Storage::assertMissing('teams/1/items/guitar.jpg');
+    });
+
+    test('can remove an existing photo', function () {
+        Storage::fake();
+        $user = User::factory()->withTeam()->create();
+        $item = Item::factory()->for($user->currentTeam)->create(['name' => 'Guitar', 'photo_path' => 'teams/1/items/guitar.jpg']);
+        Storage::put('teams/1/items/guitar.jpg', 'content');
+
+        Livewire::actingAs($user)
+            ->test('pages::inventory.index')
+            ->call('edit', $item->id)
+            ->call('removePhoto')
+            ->call('save')
+            ->assertHasNoErrors();
+
+        expect($item->fresh()->photo_path)->toBeNull();
+        Storage::assertMissing('teams/1/items/guitar.jpg');
+    });
+
+    test('photo validation rejects non-image files', function () {
+        $user = User::factory()->withTeam()->create();
+        $file = UploadedFile::fake()->create('document.pdf', 100, 'application/pdf');
+
+        Livewire::actingAs($user)
+            ->test('pages::inventory.index')
+            ->set('form.name', 'Guitar')
+            ->set('form.type', ItemType::Item->value)
+            ->set('form.photo', $file)
+            ->call('save')
+            ->assertHasErrors('form.photo');
+    });
+});
+
 describe('can import items', function () {
     test('can import items from amazon csv', function () {
         $user = User::factory()->withTeam()->create();
