[Bug] Het faalbeleid van de presentatie-import wordt gezet maar nooit gelezen

brenno commented

2026-07-24 18:16:22 +00:00

Owner

Bij het overzetten van de presentatie-import (#772) is SlideFailurePolicy meegekomen, maar het bijbehorende gedrag niet. Het resultaat is bedrading die niets doet.

Wat er staat. lib/services/import/models/slide_failure_policy.dart declareert bestEffort, skip en rasterize. ProblemSlide.suggestedPolicy draagt zo'n waarde, en deck_builder.dart (~regel 585) vult hem netjes in: rasterize als de dia een afbeelding heeft, anders bestEffort.

Wat er niet staat. Niets leest dat veld ooit uit. De import doet altijd best-effort + een 'niet overgenomen'-notitiedia; skip en rasterize zijn onbereikbaar. De problemSlides uit PresentationImportResult worden alleen geteld voor de melding ('N dia's vragen aandacht').

Waarom dit meer is dan een ongebruikt veld. Het losse bronproject had hier een echte functie: bij een dia die niet schoon converteert kon de gebruiker per dia kiezen — best-effort met notitie, rasteriseren als afbeelding, overslaan, of de hele import afbreken — met een 'auto-doorgaan'-instelling om die vraag over te slaan. Die keuze is er nu niet; de enum wekt de indruk van wel.

De dode-code-poort ving dit niet, en dat is geen fout van de poort: die toetst of een bestand bereikbaar is vanuit main(), niet of een enumwaarde ooit gelezen wordt. Het bestand is bereikbaar, dus groen.

Twee wegen, en de keuze is de eigenlijke vraag:

Bouwen wat het belooft — een beslismoment per probleemdia (of een keuze vooraf: 'vraag het me' / 'altijd best-effort'). ProblemSlide draagt al alles wat zo'n dialoog nodig heeft: bronnummer, titel, de beschrijvingen van wat er misgaat, en of er een afbeelding is. Let op de wisselwerking met de bulk-wachtrij: tien bestanden × per-dia-vragen is snel onwerkbaar, dus daar hoort een 'niet meer vragen'-uitweg bij.
Weghalen — de enum en suggestedPolicy schrappen. Best-effort + notitiedia is een verdedigbare vaste keuze, en dan is er ook niets dat anders suggereert.

Let op dat rasterize in het bronproject leunde op een aanroep van LibreOffice (soffice) als extern proces. Dat komt hier niet terug: geen subproces, geen externe afhankelijkheid. Wordt het optie 1, dan moet 'rasteriseren' iets anders betekenen (bijvoorbeeld: alleen de afbeeldingen van de dia houden en de rest laten vallen — wat de bron-code feitelijk ook deed als er al een afbeelding was) of vervallen.

Herkomst: gevonden bij de eindcontrole van #772, op de vraag of werkelijk alles uit het bronproject was overgezet. Antwoord: de conversiemotor wel, dit beleid niet.

Bij het overzetten van de presentatie-import (#772) is `SlideFailurePolicy` meegekomen, maar het bijbehorende gedrag niet. Het resultaat is bedrading die niets doet. **Wat er staat.** `lib/services/import/models/slide_failure_policy.dart` declareert `bestEffort`, `skip` en `rasterize`. `ProblemSlide.suggestedPolicy` draagt zo'n waarde, en `deck_builder.dart` (~regel 585) vult hem netjes in: `rasterize` als de dia een afbeelding heeft, anders `bestEffort`. **Wat er niet staat.** Niets leest dat veld ooit uit. De import doet altijd best-effort + een 'niet overgenomen'-notitiedia; `skip` en `rasterize` zijn onbereikbaar. De `problemSlides` uit `PresentationImportResult` worden alleen geteld voor de melding ('N dia's vragen aandacht'). **Waarom dit meer is dan een ongebruikt veld.** Het losse bronproject had hier een echte functie: bij een dia die niet schoon converteert kon de gebruiker per dia kiezen — best-effort met notitie, rasteriseren als afbeelding, overslaan, of de hele import afbreken — met een 'auto-doorgaan'-instelling om die vraag over te slaan. Die keuze is er nu niet; de enum wekt de indruk van wel. **De dode-code-poort ving dit niet**, en dat is geen fout van de poort: die toetst of een *bestand* bereikbaar is vanuit `main()`, niet of een enumwaarde ooit gelezen wordt. Het bestand is bereikbaar, dus groen. **Twee wegen, en de keuze is de eigenlijke vraag:** 1. **Bouwen wat het belooft** — een beslismoment per probleemdia (of een keuze vooraf: 'vraag het me' / 'altijd best-effort'). `ProblemSlide` draagt al alles wat zo'n dialoog nodig heeft: bronnummer, titel, de beschrijvingen van wat er misgaat, en of er een afbeelding is. Let op de wisselwerking met de bulk-wachtrij: tien bestanden × per-dia-vragen is snel onwerkbaar, dus daar hoort een 'niet meer vragen'-uitweg bij. 2. **Weghalen** — de enum en `suggestedPolicy` schrappen. Best-effort + notitiedia is een verdedigbare vaste keuze, en dan is er ook niets dat anders suggereert. Let op dat `rasterize` in het bronproject leunde op een aanroep van LibreOffice (`soffice`) als extern proces. Dat komt hier **niet** terug: geen subproces, geen externe afhankelijkheid. Wordt het optie 1, dan moet 'rasteriseren' iets anders betekenen (bijvoorbeeld: alleen de afbeeldingen van de dia houden en de rest laten vallen — wat de bron-code feitelijk ook deed als er al een afbeelding was) of vervallen. **Herkomst:** gevonden bij de eindcontrole van #772, op de vraag of werkelijk alles uit het bronproject was overgezet. Antwoord: de conversiemotor wel, dit beleid niet.

brenno added the

bug

label

2026-07-24 18:16:22 +00:00

brenno added the

in-progress

label

2026-07-24 19:25:04 +00:00

brenno commented

2026-07-24 19:25:04 +00:00

Author

Owner

Opgepakt, optie 1: het beslismoment bouwen. Tak: fix/import-faalbeleid-812.

Reikwijdte: deck_builder.dart gaat het beleid werkelijk toepassen (per bron-dia), de import-service krijgt een tweefase-vorm zodat de vraag tússen classificeren en bouwen past (niet twee keer parsen), en er komt een beslisdialoog voor de enkelvoudige import. De bulk-wachtrij vraagt bewust níéts — tien bestanden × per-dia-vragen is geen route — die blijft best-effort, en dat komt in de tekst te staan.

Eén naamswijziging vooraf: rasterize wordt imageOnly. Rasteriseren impliceert renderen, en dat kan hier niet (geen subproces). Wat de bron-code feitelijk deed was 'houd alleen de afbeeldingen van deze dia', en zo gaat het ook heten.

Opgepakt, optie 1: het beslismoment bouwen. Tak: `fix/import-faalbeleid-812`. Reikwijdte: `deck_builder.dart` gaat het beleid werkelijk toepassen (per bron-dia), de import-service krijgt een tweefase-vorm zodat de vraag tússen classificeren en bouwen past (niet twee keer parsen), en er komt een beslisdialoog voor de enkelvoudige import. De bulk-wachtrij vraagt bewust níéts — tien bestanden × per-dia-vragen is geen route — die blijft best-effort, en dat komt in de tekst te staan. Eén naamswijziging vooraf: `rasterize` wordt `imageOnly`. Rasteriseren impliceert renderen, en dat kan hier niet (geen subproces). Wat de bron-code feitelijk deed was 'houd alleen de afbeeldingen van deze dia', en zo gaat het ook heten.

brenno referenced this issue

2026-07-24 20:12:50 +00:00

fix(import): het faalbeleid per dia werkt echt (#812) #819