view video page

Author: scott grayson

.github/workflows/phpstan.yml

@@ -11,12 +11,12 @@ jobs:
     name: phpstan
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v4
 
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.2'
+          php-version: '8.1'
           coverage: none
 
       - name: Install composer dependencies

database/migrations/2024_01_01_000002_add_external_link_support_to_library_items_table.php

@@ -36,3 +36,5 @@ public function down(): void
         });
     }
 };
+
+

src/Models/LibraryItem.php

@@ -47,12 +47,23 @@ protected static function boot(): void
             if (empty($item->slug)) {
                 $item->slug = static::generateUniqueSlug($item->name, $item->parent_id);
             }
+
+            // Set created_by and updated_by on creation (like Laravel does with timestamps)
+            if (auth()->check()) {
+                $item->created_by = auth()->id();
+                $item->updated_by = auth()->id(); // Set both on creation
+            }
         });
 
         static::updating(function (self $item) {
             if ($item->isDirty('name') && ! $item->isDirty('slug')) {
                 $item->slug = static::generateUniqueSlug($item->name, $item->parent_id, $item->id);
             }
+
+            // Set updated_by on updates
+            if (auth()->check()) {
+                $item->updated_by = auth()->id();
+            }
         });
     }
 

src/Resources/Pages/CreateLink.php

@@ -21,3 +21,5 @@ protected function mutateFormDataBeforeCreate(array $data): array
         return $data;
     }
 }
+
+

src/Resources/Pages/ViewLibraryItem.php

@@ -23,6 +23,12 @@ class ViewLibraryItem extends ViewRecord
     public function getTitle(): string
     {
         $record = $this->getRecord();
+
+        // For external links, show the item name instead of "View External Link"
+        if ($record->type === 'link') {
+            return $record->name;
+        }
+
         $type = match ($record->type) {
             'folder' => 'Folder',
             'file' => 'File',
@@ -52,6 +58,7 @@ protected function getHeaderActions(): array
         if ($this->getRecord()->type === 'link' && $this->getRecord()->external_url) {
             $actions[] = Action::make('visit_link')
                 ->label('Visit Link')
+                ->color('gray')
                 ->icon('heroicon-o-arrow-top-right-on-square')
                 ->url(fn () => $this->getRecord()->external_url)
                 ->openUrlInNewTab();
@@ -115,18 +122,16 @@ public function infolist(Schema $schema): Schema
 
         return $schema
             ->components([
-                // Video section for external links that are videos
-                Section::make('Video')
-                    ->schema([
-                        VideoEmbed::make('external_url')
-                            ->visible(fn () => $record->type === 'link' && $record->isVideoUrl()),
-                    ])
+                // Video for external links that are videos (no card wrapper)
+                VideoEmbed::make('external_url')
+                    ->hiddenLabel()
                     ->visible(fn () => $record->type === 'link' && $record->isVideoUrl())
                     ->columnSpanFull(),
 
                 // Item details section
-                Section::make('Item Details')
+                Section::make()
                     ->schema([
+                        // Row 1: Name, Type
                         Grid::make(2)
                             ->schema([
                                 TextEntry::make('name')
@@ -140,26 +145,24 @@ public function infolist(Schema $schema): Schema
                                         default => $state,
                                     }),
                             ]),
-                        TextEntry::make('external_url')
-                            ->label('URL')
-                            ->visible(fn () => $record->type === 'link'),
-                        TextEntry::make('link_description')
-                            ->label('Description')
-                            ->visible(fn () => $record->type === 'link' && $record->link_description),
+
+                        // Row 2: Created At, Created By
                         Grid::make(2)
                             ->schema([
-                                TextEntry::make('creator.name')
-                                    ->label('Created By'),
                                 TextEntry::make('created_at')
                                     ->label('Created At')
                                     ->dateTime(),
-                                TextEntry::make('updater.name')
-                                    ->label('Modified By'),
-                                TextEntry::make('updated_at')
-                                    ->label('Modified At')
-                                    ->dateTime(),
+                                TextEntry::make('creator.name')
+                                    ->label('Created By'),
                             ]),
-                    ]),
+
+                        // Row 3: Description (full width)
+                        TextEntry::make('link_description')
+                            ->label('Description')
+                            ->visible(fn () => $record->type === 'link' && $record->link_description)
+                            ->columnSpanFull(),
+                    ])
+                    ->columnSpanFull(),
 
                 // Media section for files
                 Section::make('Media')

src/Traits/HasLibraryAccess.php

@@ -73,3 +73,5 @@ public function getAccessibleRootLibraryItems()
         return LibraryItem::whereNull('parent_id')->forUser($this)->get();
     }
 }
+
+

src/Traits/HasParentFolder.php

@@ -25,3 +25,4 @@ protected function getParentId(): ?int
         return $this->parentId;
     }
 }
+