fix statue activity log (#396)

* fix statue activity log * formatting --------- Co-authored-by: Andrei Ioniță <[email protected]>
code4romania · Sep 2, 2024 · 99559ba · 99559ba
1 parent 77d88d5
commit 99559ba
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 4 deletions.
diff --git a/app/Filament/Resources/OrganizationResource/Actions/Tables/Activity/ViewActivityAction.php b/app/Filament/Resources/OrganizationResource/Actions/Tables/Activity/ViewActivityAction.php
@@ -7,6 +7,8 @@
 use App\Filament\Forms\Components\Value;
 use App\Models\Activity;
 use Filament\Tables\Actions\ViewAction as BaseAction;
+use Illuminate\Support\HtmlString;
+use Spatie\MediaLibrary\MediaCollections\Models\Media;
 
 class ViewActivityAction extends BaseAction
 {
@@ -41,6 +43,22 @@ protected function setUp(): void
 
             Value::make('changed_field_new_value')
                 ->label(__('activity.value.new'))
+                ->content(function (Activity $record) {
+                    if ($record->changed_field === 'statute') {
+                        $media = Media::find($record->changed_field_new_value);
+                        if (! $media) {
+                            return '-';
+                        }
+
+                        return new HtmlString(\sprintf(
+                            '<a href="%s" target="_blank">%s</a>',
+                            $media->getTemporaryUrl(now()->addMinutes(30)),
+                            $media->file_name
+                        ));
+                    }
+
+                    return $record->changed_field_new_value;
+                })
                 ->inlineLabel(),
         ]);
 

diff --git a/app/Models/Activity.php b/app/Models/Activity.php
@@ -176,10 +176,6 @@ public function reject(?string $reason): void
      */
     public function getModifiedField(mixed $value): ?string
     {
-        if ($this->description === 'statute') {
-            $value = Media::find($value)?->getUrl();
-        }
-
         if (\gettype($value) == 'boolean') {
             $value = $value ? __('field.boolean.true') : __('field.boolean.false');
         }