LibreKAT/Ocideck

Watch

Fork 1

Document opslaan/exporteren verdubbelt extensies met hoofdletters #1659

New issue

Closed

opened 2026-08-21 12:00:46 +00:00 by brenno · 2 comments

brenno commented

2026-08-21 12:00:46 +00:00

Owner

Probleem

De extensiecontrole is hoofdlettergevoelig. Een gebruiker die NAME.MD kiest bij Opslaan als krijgt NAME.MD.md; Report.PDF bij export wordt Report.PDF.pdf. Dit is zichtbaar op case-preserving platformen en kan verwarring of verkeerde bestandsassociaties geven.

Reproductie

Kies Opslaan als en voer Document.MD in.
Exporteer daarnaast naar Rapport.PDF.
Controleer de uiteindelijke namen.

Verwacht

Bekende extensies worden hoofdletterongevoelig herkend en niet dubbel toegevoegd.

Technische aanwijzing

lib/services/file/file_service_open.dart:_writeDocumentToPicked en lib/widgets/document_editor_screen.dart:_pickDocumentExportPath gebruiken case-sensitive endsWith.

Gevonden bij audit van commit d439638c6b.

## Probleem De extensiecontrole is hoofdlettergevoelig. Een gebruiker die NAME.MD kiest bij Opslaan als krijgt NAME.MD.md; Report.PDF bij export wordt Report.PDF.pdf. Dit is zichtbaar op case-preserving platformen en kan verwarring of verkeerde bestandsassociaties geven. ## Reproductie 1. Kies Opslaan als en voer Document.MD in. 2. Exporteer daarnaast naar Rapport.PDF. 3. Controleer de uiteindelijke namen. ## Verwacht Bekende extensies worden hoofdletterongevoelig herkend en niet dubbel toegevoegd. ## Technische aanwijzing lib/services/file/file_service_open.dart:_writeDocumentToPicked en lib/widgets/document_editor_screen.dart:_pickDocumentExportPath gebruiken case-sensitive endsWith. Gevonden bij audit van commit d439638c6bd1b519d68d87680cb022b7d5eddc85.

brenno added the

bug

triage

labels

2026-08-21 12:00:46 +00:00

brenno added

accepted

and removed

triage

labels

2026-08-21 22:52:25 +00:00

brenno commented

2026-08-21 22:52:25 +00:00

Author

Owner

Triage: accepted

Bevestigd tegen main (e93ef205c).

lib/services/file/file_service_open.dart — dest.endsWith('.md') ? dest : '$dest.md'.
lib/widgets/document_editor_screen.dart — dest.endsWith('.$ext') ? dest : '$dest.$ext'.

Beide hoofdlettergevoelig, dus Document.MD wordt Document.MD.md en Rapport.PDF wordt Rapport.PDF.pdf.

Oplossingsrichting

Op beide plekken vergelijken via p.extension(dest).toLowerCase() in plaats van endsWith. Dat vangt en passant een tweede rafel in dezelfde regel: een gekozen pad zónder extensie maar mét een punt in een mapnaam wordt nu ook verkeerd gelezen.

Doe in dezelfde pas de deck-kant mee — saveDeckAs/pakketexport gebruiken hetzelfde patroon; als daar hetzelfde staat, hoort het in één wijziging, niet in een tweede issue dat over een half jaar nog open staat.

Regressietest (verplicht)

test/document_new_and_save_as_test.dart (of de dichtstbijzijnde file-service-test) met een kiezer die Document.MD teruggeeft → pad ongewijzigd; idem Rapport.PDF langs _pickDocumentExportPath.

Kosten

Klein: twee regels plus tests. Geen l10n.

Prioriteit

Laag-middel. Geen dataverlies, wel verwarring en verkeerde bestandskoppelingen op case-preserving systemen.

## Triage: accepted **Bevestigd tegen `main` (e93ef205c).** - `lib/services/file/file_service_open.dart` — `dest.endsWith('.md') ? dest : '$dest.md'`. - `lib/widgets/document_editor_screen.dart` — `dest.endsWith('.$ext') ? dest : '$dest.$ext'`. Beide hoofdlettergevoelig, dus `Document.MD` wordt `Document.MD.md` en `Rapport.PDF` wordt `Rapport.PDF.pdf`. ## Oplossingsrichting Op beide plekken vergelijken via `p.extension(dest).toLowerCase()` in plaats van `endsWith`. Dat vangt en passant een tweede rafel in dezelfde regel: een gekozen pad zónder extensie maar mét een punt in een mapnaam wordt nu ook verkeerd gelezen. Doe in dezelfde pas de deck-kant mee — `saveDeckAs`/pakketexport gebruiken hetzelfde patroon; als daar hetzelfde staat, hoort het in één wijziging, niet in een tweede issue dat over een half jaar nog open staat. ## Regressietest (verplicht) `test/document_new_and_save_as_test.dart` (of de dichtstbijzijnde file-service-test) met een kiezer die `Document.MD` teruggeeft → pad ongewijzigd; idem `Rapport.PDF` langs `_pickDocumentExportPath`. ## Kosten Klein: twee regels plus tests. Geen l10n. ## Prioriteit Laag-middel. Geen dataverlies, wel verwarring en verkeerde bestandskoppelingen op case-preserving systemen.

brenno referenced this issue

2026-08-21 22:56:34 +00:00

Extensiecontrole is hoofdlettergevoelig op de deck-, stijlprofiel- en zegelroute #1700

brenno commented

2026-08-21 22:56:50 +00:00

Author

Owner

De drie plekken buiten de documentmodus staan nu apart: #1700 (deck Opslaan als, stijlprofielexport, zegel-tijdstempelverzoek). Voorstel daar is één gedeelde withExtension-helper voor alle vijf de plekken; doe dit issue en #1700 in één pas.

De drie plekken buiten de documentmodus staan nu apart: #1700 (deck Opslaan als, stijlprofielexport, zegel-tijdstempelverzoek). Voorstel daar is één gedeelde `withExtension`-helper voor alle vijf de plekken; doe dit issue en #1700 in één pas.

brenno added the

in-progress

label

2026-08-21 23:36:48 +00:00

brenno referenced this issue from a commit

2026-08-22 00:18:57 +00:00

fix: eerste batch accepted issues (#1700, #1659, #1675, #1679, #1658, #1657, #1665, #1667, #1649, #1648, #1651, #1685)

brenno referenced this issue

2026-08-22 00:19:14 +00:00

fix: alle accepted issues in één batch (17 issues) #1706

brenno referenced this issue from a commit

2026-08-22 00:31:52 +00:00