[Invoices]: Download the PDF for an invoice from the invoice list

[nielsdrost7]

Context

The invoices list has a "Download PDF" action stub but it currently does nothing. Users must navigate to the invoice edit view to obtain the PDF. Completing this action makes PDF retrieval a one-click operation.

Related: #190 covers the same action from within the EditInvoice page header. Both issues share the same underlying InvoiceService::generatePdf() implementation.

---

User story

As a user, I want to click Download PDF in the invoice row dropdown so I can download the invoice PDF without opening the edit form.

---

Acceptance criteria

• The existing Download PDF stub action is implemented and triggers a file download.
• The downloaded file is the rendered invoice PDF.
• The action is visible for all invoice statuses.
• If PDF generation fails, the user sees an error notification — no silent failure.

---

Arrange · Act · Assert

// Modules/Invoices/Tests/Feature/InvoicesTest.php

#[Test]
public function it_downloads_pdf_from_invoice_list(): void
{
    /* Arrange */
    $invoice = Invoice::factory()->for($this->company)->create([
        'invoice_number' => 'INV-2026-001',
    ]);

    /* Act */
    $component = Livewire::actingAs($this->user)
        ->test(ListInvoices::class, ['tenant' => Str::lower($this->company->search_code)])
        ->callTableAction('download-pdf', $invoice);

    /* Assert */
    $component->assertSuccessful();
    $component->assertFileDownloaded('INV-2026-001.pdf');
}

[coderabbitai]

📝 CodeRabbit Plan Mode

Generate an implementation plan and prompts that you can use with your favorite coding agent.

• Create Plan

Examples

• Example 1
• Example 2

---

🔗 Similar Issues

Related Issues

• [Quotes]: Download the PDF for a quote from the quote list #196
• [Clients]: Create a quote for a specific client from the client list #191
• [Clients]: Create an invoice for a specific client from the client list #192
• [Invoices]: Download PDF for an invoice from the invoice view #190
• [Quotes]: Send a quote by email from the quote list #199

👤 Suggested Assignees

• nielsdrost7

---

🧪 Issue enrichment is currently in open beta.

You can configure auto-planning by selecting labels in the issue_enrichment configuration.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!

[nielsdrost7]

@coderabbitai make pull-request with AAA test Scenario's: Arrange, Act, Assert. And also make sure you test edge cases

[coderabbitai]

Rate Limit Exceeded

@nielsdrost7 have exceeded the limit for the number of chat messages per hour. Please wait 0 minutes and 10 seconds before sending another message.

[nielsdrost7]

Reopening: the list-row "Download PDF" action is still a stub on develop (InvoicesTable.php:190-197, empty ->action(function (Invoice $record): void {}) with a "todo" modal) — the working implementation exists only on unmerged PR #608; keep open until that wiring lands on develop.

Verified with a failing test on branch chore/verify-pdf-list-stub-and-wkhtmltopdf-bug (commit 036559c): InvoiceListActionsTest::it_fails_to_generate_a_pdf_when_the_download_pdf_action_is_triggered_from_the_invoice_list asserts InvoiceService::generatePdf() is called when the row action fires, and fails with should be called exactly 1 times but called 0 times — the action closure never calls it.

---

The invoices list row action group shows a Download PDF button, but its action closure is empty — clicking it does nothing. InvoiceService::generatePdf() already works (it powers the same action on EditInvoice), so this is pure wiring. PR #608 (feature/130-report-builder) contains an implementation via PdfGenerationService::invoicePdf(); this issue stays open until that (or equivalent wiring on develop) is merged.

Specific

Replace the stub action in Modules/Invoices/Filament/Company/Resources/Invoices/Tables/InvoicesTable.php (currently ->action(function (Invoice $record): void {}) plus a leftover "todo" modalDescription) with a real streamed PDF download of the row's invoice, reusing the existing service path (InvoiceService::generatePdf() on develop today; PdfGenerationService::invoicePdf() once PR #608 merges). Remove the modal — it exists only because the stub had nothing to do.

Measurable

• Clicking Download PDF on any invoice row downloads <invoice_number>.pdf (draft fallback: invoice-draft-<id>.pdf).
• No empty ->action(function (Invoice $record): void {}) stub remains for this action.
• Existing PDF content (items, totals, footer) matches what EditInvoice's working download produces for the same invoice.
• Generation failure shows a danger notification instead of a silent no-op.

Achievable

Direct reuse of InvoiceService::generatePdf() — the same call already wired on EditInvoice.php:96-99. No new service code needed; only the table action closure and removing the leftover modal.

Relevant

This was closed as done in the past but the UI is verifiably dead on develop — a customer-facing regression sitting in a "closed" issue.

Time-Bound

Immediate — this is wiring, not new development. Should not wait for PR #608 unless that PR is merging within the same sprint.

Arrange

Invoice with invoice_number = 'INV-2026-001' and items, in AbstractCompanyPanelTestCase.

Act

Livewire::test(ListInvoices::class, [...])->callAction(TestAction::make('download pdf')->table($invoice)).

Assert

• The action triggers InvoiceService::generatePdf() for the row's invoice (verified via mock expectation in the linked branch's test).
• Response is a StreamedResponse with Content-Type: application/pdf.
• No "todo" modal remains on the action.
• Draft invoices still download successfully (fallback filename).