From 0c64fe55d7f9bf54ecb3859607238f1708ce1499 Mon Sep 17 00:00:00 2001
From: vk <vk>
Date: Thu, 12 Feb 2026 22:53:56 +0100
Subject: [PATCH] Kleine Bilder anzeigen (vibe-kanban 9bcfc693)
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Die Ladezeiten der Bilder ist zu hoch, weil die Original geladen werden.
Bei /association/project-support lade in der Übersicht und in der Einzel-Ansicht /association/project-support/badgebox-for-nostr-manage-your-badges nur die Conversions der Bilder, also die kleinere Versionen.
---
 app/Models/ProjectProposal.php                | 10 +-
 .../views/components/project-card.blade.php   |  4 +-
 .../project-support/show.blade.php            |  2 +-
 routes/web.php                                | 10 +-
 tests/Feature/MediaSignedRouteTest.php        | 95 +++++++++++++++++++
 5 files changed, 115 insertions(+), 6 deletions(-)
 create mode 100644 tests/Feature/MediaSignedRouteTest.php

diff --git a/app/Models/ProjectProposal.php b/app/Models/ProjectProposal.php
index 20f3f1c..f358e0b 100644
--- a/app/Models/ProjectProposal.php
+++ b/app/Models/ProjectProposal.php
@@ -78,14 +78,20 @@ class ProjectProposal extends Model implements HasMedia
             ->useFallbackUrl(asset('einundzwanzig-alpha.jpg'));
     }
 
-    public function getSignedMediaUrl(string $collection = 'main', int $expireMinutes = 60): string
+    public function getSignedMediaUrl(string $collection = 'main', int $expireMinutes = 60, ?string $conversion = null): string
     {
         $media = $this->getFirstMedia($collection);
         if (! $media) {
             return asset('einundzwanzig-alpha.jpg');
         }
 
-        return url()->temporarySignedRoute('media.signed', now()->addMinutes($expireMinutes), ['media' => $media]);
+        $parameters = ['media' => $media];
+
+        if ($conversion && $media->hasGeneratedConversion($conversion)) {
+            $parameters['conversion'] = $conversion;
+        }
+
+        return url()->temporarySignedRoute('media.signed', now()->addMinutes($expireMinutes), $parameters);
     }
 
     public function einundzwanzigPleb(): BelongsTo
diff --git a/resources/views/components/project-card.blade.php b/resources/views/components/project-card.blade.php
index 4438fdc..26fb675 100644
--- a/resources/views/components/project-card.blade.php
+++ b/resources/views/components/project-card.blade.php
@@ -24,7 +24,7 @@
             <a class="relative block w-full h-48 sm:w-56 sm:h-auto xl:sidebar-expanded:w-40 2xl:sidebar-expanded:w-56 shrink-0 sm:shrink-0"
                href="{{ route('association.projectSupport.item', ['projectProposal' => $project]) }}">
                 <img class="absolute object-cover object-center w-full h-full"
-                     src="{{ $project->getSignedMediaUrl('main') }}" alt="Meetup 01">
+                     src="{{ $project->getSignedMediaUrl('main', 60, 'preview') }}" alt="Meetup 01">
                 <button class="absolute top-0 right-0 mt-4 mr-4">
                     <img class="rounded-full h-8 w-8"
                          src="{{ $project->einundzwanzigPleb->profile?->picture }}"
@@ -36,7 +36,7 @@
             <a class="relative block w-full h-48 sm:w-56 sm:h-auto xl:sidebar-expanded:w-40 2xl:sidebar-expanded:w-56 shrink-0 sm:shrink-0"
                href="{{ route('association.projectSupport.item', ['projectProposal' => $project]) }}">
                 <img class="absolute object-cover object-center w-full h-full"
-                     src="{{ $project->getSignedMediaUrl('main') }}" alt="Meetup 01">
+                     src="{{ $project->getSignedMediaUrl('main', 60, 'preview') }}" alt="Meetup 01">
                 <button class="absolute top-0 right-0 mt-4 mr-4">
                     <img class="rounded-full h-8 w-8"
                          src="{{ $project->einundzwanzigPleb->profile?->picture }}"
diff --git a/resources/views/livewire/association/project-support/show.blade.php b/resources/views/livewire/association/project-support/show.blade.php
index b16c305..7819279 100644
--- a/resources/views/livewire/association/project-support/show.blade.php
+++ b/resources/views/livewire/association/project-support/show.blade.php
@@ -168,7 +168,7 @@ new class extends Component {
                 </div>
 
                 <figure class="mb-6">
-                    <img class="rounded-sm h-48" src="{{ $projectProposal->getSignedMediaUrl('main') }}"
+                    <img class="rounded-sm h-48" src="{{ $projectProposal->getSignedMediaUrl('main', 60, 'preview') }}"
                          alt="Picture">
                 </figure>
 
diff --git a/routes/web.php b/routes/web.php
index c9153ef..7850a11 100644
--- a/routes/web.php
+++ b/routes/web.php
@@ -19,8 +19,16 @@ Route::get('dl/{media}', function (Media $media, Request $request) {
     ->middleware('signed');
 
 Route::get('media/{media}', function (Media $media, Request $request) {
+    $conversion = $request->query('conversion');
+
+    if ($conversion && $media->hasGeneratedConversion($conversion)) {
+        $path = $media->getPathRelativeToRoot($conversion);
+    } else {
+        $path = $media->getPathRelativeToRoot();
+    }
+
     return Storage::disk($media->disk)->response(
-        $media->getPathRelativeToRoot(),
+        $path,
         $media->file_name,
         [
             'Content-Type' => $media->mime_type,
diff --git a/tests/Feature/MediaSignedRouteTest.php b/tests/Feature/MediaSignedRouteTest.php
new file mode 100644
index 0000000..de6be98
--- /dev/null
+++ b/tests/Feature/MediaSignedRouteTest.php
@@ -0,0 +1,95 @@
+<?php
+
+use App\Models\ProjectProposal;
+use Illuminate\Support\Facades\Storage;
+
+it('serves original media via signed route', function () {
+    Storage::fake('private');
+
+    $project = ProjectProposal::factory()->create();
+
+    $project->addMedia(
+        \Illuminate\Http\UploadedFile::fake()->image('test.jpg', 100, 100)
+    )->toMediaCollection('main');
+
+    $media = $project->getFirstMedia('main');
+
+    $url = url()->temporarySignedRoute('media.signed', now()->addMinutes(60), ['media' => $media]);
+
+    $this->get($url)->assertSuccessful();
+});
+
+it('serves conversion media via signed route when conversion parameter is provided', function () {
+    Storage::fake('private');
+
+    $project = ProjectProposal::factory()->create();
+
+    $project->addMedia(
+        \Illuminate\Http\UploadedFile::fake()->image('test.jpg', 500, 500)
+    )->toMediaCollection('main');
+
+    $media = $project->getFirstMedia('main');
+
+    $url = url()->temporarySignedRoute('media.signed', now()->addMinutes(60), [
+        'media' => $media,
+        'conversion' => 'preview',
+    ]);
+
+    $this->get($url)->assertSuccessful();
+});
+
+it('falls back to original when conversion does not exist', function () {
+    Storage::fake('private');
+
+    $project = ProjectProposal::factory()->create();
+
+    $project->addMedia(
+        \Illuminate\Http\UploadedFile::fake()->image('test.jpg', 100, 100)
+    )->toMediaCollection('main');
+
+    $media = $project->getFirstMedia('main');
+
+    $url = url()->temporarySignedRoute('media.signed', now()->addMinutes(60), [
+        'media' => $media,
+        'conversion' => 'nonexistent',
+    ]);
+
+    $this->get($url)->assertSuccessful();
+});
+
+it('rejects unsigned media requests', function () {
+    Storage::fake('private');
+
+    $project = ProjectProposal::factory()->create();
+
+    $project->addMedia(
+        \Illuminate\Http\UploadedFile::fake()->image('test.jpg', 100, 100)
+    )->toMediaCollection('main');
+
+    $media = $project->getFirstMedia('main');
+
+    $this->get("/media/{$media->id}")->assertForbidden();
+});
+
+it('generates signed url with conversion parameter', function () {
+    Storage::fake('private');
+
+    $project = ProjectProposal::factory()->create();
+
+    $project->addMedia(
+        \Illuminate\Http\UploadedFile::fake()->image('test.jpg', 500, 500)
+    )->toMediaCollection('main');
+
+    $urlWithoutConversion = $project->getSignedMediaUrl('main');
+    $urlWithConversion = $project->getSignedMediaUrl('main', 60, 'preview');
+
+    expect($urlWithoutConversion)->not->toContain('conversion=');
+});
+
+it('returns fallback url when no media exists', function () {
+    $project = ProjectProposal::factory()->create();
+
+    $url = $project->getSignedMediaUrl('main', 60, 'preview');
+
+    expect($url)->toContain('einundzwanzig-alpha.jpg');
+});