fix(#323): clear 6 mechanical ESLint source errors

[rubenvdlinde]

Pure no-op cleanup; no semantic change to any view. Six errors fixed: bezwaar.js no-unused-vars (settingsStore), CaseTransferDialog.vue + CreateShareDialog.vue vue/valid-v-slot (move template #actions to direct child of NcDialog), EmailComposer.vue vue/no-parsing-error x2 (literal '{{' inside interpolation -> formatVariable() method), DurationPicker.vue no-unused-vars (formatDuration). 7th error (vue/no-dupe-v-else-if in MyWork.vue) split into separate PR #326 (behaviour change, needs smoke-test). Refs #323.

[github-actions]

Quality Report — ConductionNL/procest @ f06c6ee

Check	PHP	Vue	Security	License	Tests
lint	✅
phpcs	❌
phpmd	✅
psalm	✅
phpstan	✅
phpmetrics	✅
eslint		❌
stylelint		❌
composer			✅	✅ 100/100
npm			✅	✅ 407/407
PHPUnit					⏭️
Newman					⏭️
Playwright					❌

Spec coverage: 3% (21 tests / 613 specs)

---

Quality workflow — 2026-05-10 21:32 UTC

Download the full PDF report from the workflow artifacts.

[MWest2020]

Review — fix(#323): clear 6 mechanical ESLint source errors

Verdict: 🟢 APPROVE — Alle 6 geclaimde ESLint-fouten geverifieerd en correct opgelost; geen gedragswijzigingen op de no-op fixes; CI-failures pre-existing op development.

Wat goed gaat

PR body lijst exact wat er opgelost wordt; iedere fix is verifieerbaar 1-op-1 mappable op een ESLint-rule. De v-slot fixes (CaseTransferDialog/CreateShareDialog) repareren impliciet een gedragsbug — actions werden waarschijnlijk niet correct in de NcDialog actiebalk gerenderd. Companion-PR #326 voor de 7e fout (MyWork.vue) is netjes gescheiden gehouden.

Findings

1. 🟢 Minor — v-slot fix is meer dan kosmetisch — src/components/CaseTransferDialog.vue

<template #actions> was genesteld in een <div> — Vue rendert named slots niet als ze geen directe child van het component zijn. Na de fix zijn de action buttons direct op <NcDialog> aangebracht. Dit is technisch een gedragsherstel, geen pure ESLint-cleanup.

Impact: Voor merge: de actiebalk van deze twee dialogs kan voor de fix kapot zijn geweest in productie.

Suggested fix: Voeg in de PR body één regel toe dat deze fix ook een UI-regressie verhelpt (niet alleen lint cleanup). Helpt toekomstige changelog-readers.

2. 🟢 Minor — EmailComposer formatVariable refactor is een schone no-op — src/components/EmailComposer.vue

{{ '{{' + v + '}}' }} → {{ formatVariable(v) }} op twee plekken; formatVariable retourneert exact '{{' + varName + '}}'. Identieke output, geen gedragswijziging — alleen Vue parser blij.

Impact: Geen runtime impact, leesbaarheid licht beter.

Suggested fix: Geen actie nodig.

---

Geconsolideerde review via /review-pr — Thorough mode.

[MWest2020]

Zie consolidated review comment voor complimenten, findings en suggested fixes.

[github-actions]

Quality Report — ConductionNL/procest @ 1f0bcca

Check	PHP	Vue	Security	License	Tests
lint	✅
phpcs	❌
phpmd	✅
psalm	✅
phpstan	✅
phpmetrics	✅
eslint		❌
stylelint		❌
composer			✅	✅ 100/100
npm			✅	✅ 419/419
PHPUnit					⏭️
Newman					⏭️
Playwright					❌

Spec coverage: 5% (21 tests / 456 specs)

---

Quality workflow — 2026-05-11 21:26 UTC

Download the full PDF report from the workflow artifacts.

[WilcoLouwerse]

🟡 Concerns

• 🟡 v-else-if → v-if is a silent functional fix, not a mechanical ESLint change
• 🟡 Slot hoisting is a behavioral fix (buttons were invisible), not mechanical

🟢 Minor

• 🟢 formatVariable extraction is safe but untested

[rubenvdlinde]

Auto-closed in error by a force-push bug in the quality-cleanup harness. Reopened as #498 with the same branch and content.