1635: Increased invoice description length

migrations/Version20240701104400.php

@@ -0,0 +1,31 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DoctrineMigrations;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+/**
+ * Auto-generated Migration: Please modify to your needs!
+ */
+final class Version20240701104400 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return '';
+    }
+
+    public function up(Schema $schema): void
+    {
+        // this up() migration is auto-generated, please modify it to your needs
+        $this->addSql('ALTER TABLE invoice CHANGE description description VARCHAR(500) DEFAULT NULL');
+    }
+
+    public function down(Schema $schema): void
+    {
+        // this down() migration is auto-generated, please modify it to your needs
+        $this->addSql('ALTER TABLE invoice CHANGE description description VARCHAR(255) DEFAULT NULL');
+    }
+}

src/Entity/Invoice.php

@@ -13,10 +13,14 @@
 #[ORM\Entity(repositoryClass: InvoiceRepository::class)]
 class Invoice extends AbstractBaseEntity
 {
+    // Opus allows at most 500 characters (cf. BillingService::exportInvoicesToSpreadsheet).
+    public const DESCRIPTION_MAX_LENGTH = 500;
+
     #[ORM\Column(length: 255)]
     private ?string $name = null;
 
-    #[ORM\Column(length: 255, nullable: true)]
+
+    #[ORM\Column(length: self::DESCRIPTION_MAX_LENGTH, nullable: true)]
     private ?string $description = null;
 
     #[ORM\Column]

src/Service/BillingService.php

@@ -307,7 +307,7 @@ public function exportInvoicesToSpreadsheet(array $invoiceIds): Spreadsheet
             $setCellValue(15, $row, substr('Att: '.$contactName, 0, 35));
             // 16. "Toptekst, yderligere spec i det hvide felt på fakturaen"
             $description = $invoice->getDescription() ?? '';
-            $setCellValue(16, $row, substr($description, 0, 500));
+            $setCellValue(16, $row, substr($description, 0, Invoice::DESCRIPTION_MAX_LENGTH));
             // 17. "Leverandør"
             if ($internal) {
                 $setCellValue(17, $row, str_pad($this->invoiceSupplierAccount, 10, '0', \STR_PAD_LEFT));