Patches en Upstream Issues¶

Doel: Track bekende bugs in Drupal core en contrib modules die invloed hebben op dit project.

🔴 Actieve Issues (wachten op upstream fix)¶

Issue: #3505182 - Navigation:title component crasht met NULL entity label
Affects: Drupal 11.2.x (onze versie: 11.2.10)
Status: RTBC (Ready To Be Committed) - MR 10235 voor 11.x-dev branch
Impact op ons project: - Factuur detailpagina (/user/{uid}/invoices/{id}) geeft SDC component error - Error: InvalidComponentDataException: Unable to render component "navigation:title" - Gebeurt omdat draft invoices geen label hebben (invoice_number = NULL)

Patch beschikbaar?
✅ Ja, maar alleen voor 11.x-dev branch (development)
❌ Past niet op 11.2.10 (stable release)
URL: https://git.drupalcode.org/project/drupal/-/merge_requests/10235.diff

Onze workaround: - Link niet naar factuur detailpagina - Gebruik direct PDF download link via Entity Print - Geen SDC component = geen crash

Wanneer fixen: - Wacht tot Drupal 11.3.0 of 12.0.0 met deze fix - Of: upgrade naar 11.x-dev (niet aan te raden voor productie) - Verwachting: Q1/Q2 2026 (volgende minor release)

2. Commerce Invoice - Entity Label Fallback¶

Issue: #3532643 - Provide label fallback for draft invoices
Affects: Commerce Invoice 2.2.x (onze versie: 2.2.0)
Status: Needs review - MR 11 voor 2.x-dev branch
Impact op ons project: - Draft/validation invoices hebben NULL als label - Triggert de core Navigation bug (zie #1) - Invoice->label() zou fallback moeten geven: "Invoice #123" of "Draft Invoice"

Patch beschikbaar?
✅ Ja, maar alleen voor 2.x-dev branch (development)
❌ Past niet op 2.2.0 (stable release)
URL: https://git.drupalcode.org/project/commerce_invoice/-/merge_requests/11.diff

Onze workaround: - Zelfde als #1: gebruik PDF links, niet detailpagina - Invoice nummering werkt prima (alleen label ontbreekt)

Wanneer fixen: - Wacht tot Commerce Invoice 2.3.0 of 3.0.0 met deze fix - Verwachting: onbekend (contrib module, geen vaste release cycle)

✅ Actieve Patches (toegepast via composer-patches)¶

1. Commerce Invoice - File Validator Drupal 10+ Fix¶

Locatie: patches/commerce_invoice-file_validator_drupal10.patch
Reden: Commerce Invoice 2.2.0 gebruikt oude file validator API
Status: ✅ Toegepast, werkt op 11.2.10
Composer entry:

{
  "drupal/commerce_invoice": {
    "Fix file validator for Drupal 10+": "patches/commerce_invoice-file_validator_drupal10.patch"
  }
}

Opmerking: Dit is een lokale patch (niet van drupal.org), getest en werkend.

🔧 Patch Management Lessen¶

Waarom MR patches niet werkten¶

Probleem: GitLab Merge Request URLs (.diff) zijn voor development branches: - https://git.drupalcode.org/project/drupal/-/merge_requests/10235.diff → 11.x-dev - https://git.drupalcode.org/project/commerce_invoice/-/merge_requests/11.diff → 2.x-dev

Onze installatie: Stable releases (11.2.10, 2.2.0)

Error bij toepassen:

No available patcher was able to apply patch https://git.drupalcode.org/...

Best Practices voor Patches¶

Check target branch - MR patches zijn vaak voor HEAD/dev branch
Gebruik lokale patches voor stable releases (copy patch uit issue, rebase indien nodig)
Test altijd - composer reinstall {package} om patch te applyen
Security risk - MR URL patches kunnen veranderen (malicious code injection mogelijk)
Lock patches - gebruik patches.lock.json (composer-patches v2) voor checksums

Wanneer WEL patches gebruiken¶

✅ Productie-ready scenarios: - Patch is getagd met specifieke Drupal versie (bijv. 11.2.x-dev) - Patch bestand staat op https://www.drupal.org/files/issues/... (static URL) - Lokale patch bestanden in /patches/ directory - Issue is RTBC en wacht alleen op commit

❌ NIET gebruiken: - MR .diff URLs direct in composer.json (kunnen veranderen) - Patches voor verkeerde major/minor versie - Patches zonder checksum verification

📋 Monitoring Plan¶

Check quarterly (elk kwartaal): 1. Drupal release notes voor fix van #3505182 2. Commerce Invoice changelog voor fix van #3532643 3. Update dit document met nieuwe status

Signalen voor actie: - Drupal 11.3.0 release announcement - Commerce Invoice 2.3.0 release - Security advisory die een van deze issues noemt

Bij upstream fix: 1. Update Drupal core / Commerce Invoice naar versie met fix 2. Test factuur detailpagina (/user/{uid}/invoices/{invoice_id}) 3. Verwijder workaround (PDF links kunnen blijven, zijn betere UX) 4. Update dit document met "✅ Opgelost in versie X.Y.Z"

🔗 Referenties¶

Drupal.org Issues: - Core #3505182: https://www.drupal.org/project/drupal/issues/3505182 - Commerce Invoice #3532643: https://www.drupal.org/project/commerce_invoice/issues/3532643

Related: - Entity Print module: https://www.drupal.org/project/entity_print - Composer Patches docs: https://github.com/cweagans/composer-patches

Ons project: - Sessie verslag: Notities/20251216-fase2-sessie-verslag.md - Commerce setup: Notities/commerce-setup-overzicht.md

Laatst bijgewerkt: 2025-12-16
Volgende check: 2026-03-01 (Q1 2026)