Made attachment datatable sortable by internal filename and external url

Author: jbtronics

migrations/Version20250220215048.php

@@ -6,11 +6,7 @@
 
 use App\Migration\AbstractMultiPlatformMigration;
 use Doctrine\DBAL\Schema\Schema;
-use Doctrine\Migrations\AbstractMigration;
 
-/**
- * Auto-generated Migration: Please modify to your needs!
- */
 final class Version20250220215048 extends AbstractMultiPlatformMigration
 {
     public function getDescription(): string

src/DataTables/AttachmentDataTable.php

@@ -119,6 +119,7 @@ public function configure(DataTable $dataTable, array $options): void
         $dataTable->add('internal_link', TextColumn::class, [
             'label' => 'attachment.table.internal_file',
             'propertyPath' => 'filename',
+            'orderField' => 'NATSORT(attachment.original_filename)',
             'render' => function ($value, Attachment $context) {
                 if ($this->attachmentHelper->isInternalFileExisting($context)) {
                     return sprintf(
@@ -135,12 +136,14 @@ public function configure(DataTable $dataTable, array $options): void
         $dataTable->add('external_link', TextColumn::class, [
             'label' => 'attachment.table.external_link',
             'propertyPath' => 'host',
+            'orderField' => 'attachment.external_path',
             'render' => function ($value, Attachment $context) {
                 if ($context->hasExternal()) {
                     return sprintf(
-                        '<a href="%s" class="link-external">%s</a>',
+                        '<a href="%s" class="link-external" title="%s" target="_blank" rel="noopener">%s</a>',
                         htmlspecialchars((string) $context->getExternalPath()),
-                        htmlspecialchars($value)
+                        htmlspecialchars((string) $context->getExternalPath()),
+                        htmlspecialchars($value),
                     );
                 }