fix(pakket): grafiekdata uit een pakket wordt gelezen zoals overal elders #420

Merged

brenno merged 1 commit from fix/pakket-grafiekdata-json into main 2026-07-20 16:03:47 +00:00

Conversation 0 Commits 1 Files changed 4 +144 -2

brenno commented 2026-07-20 16:03:40 +00:00

Owner

Copy link

Deze fix lag als losse commit in de hoofdwerkmap, op een lokale main die een
eigen lijn bleek te zijn. Via cherry-pick op origin/main gezet; inhoud
ongewijzigd.

Grafiekdata uit een .ocideck werd als CSV gelezen. Sinds grafiekdata naar
een los bestand verhuisde munt OciDeck altijd .json, maar het uitpakken las elk
databestand als CSV. Op JSON levert dat geen fout maar onzin: de eerste regel {
werd de kopregel, dus geen enkele reeks, en het resultaat was een grafiek zonder
cijfers. De enige test die dit pad dekte gebruikte een .csv en bleef daarom
groen — het soort dekking dat een gat juist verbergt.

En een grafiek zonder cijfers bleef stil. Twee paden meldden het niet: het
bestandskiezer-pad gooide de waarschuwing weg, en het bytes-pad (web, drag-drop,
URL-import) kende er geen. Dat laatste heeft geen projectmap en kán een
data/…-verwijzing dus niet oplossen — een grens van het pad, geen fout, maar
wel iets om te zeggen in plaats van te verzwijgen.

Toetsing

make check groen: 4189 tests, dekking 79,5%. CI blijft pending — er is geen
runner.

De cherry-pick voegde schoon samen met de herstructurering van
tabs_provider.dart uit #417 (TabInfo/TabsState naar een eigen part): deze
wijziging zit in het openpad, niet in het verplaatste deel. De twee aanroepen van
_warnUnfilledChartData zijn losse plekken, geen dubbeling.

🤖 Generated with Claude Code

Deze fix lag als losse commit in de hoofdwerkmap, op een lokale `main` die een eigen lijn bleek te zijn. Via cherry-pick op `origin/main` gezet; inhoud ongewijzigd. **Grafiekdata uit een `.ocideck` werd als CSV gelezen.** Sinds grafiekdata naar een los bestand verhuisde munt OciDeck altijd `.json`, maar het uitpakken las elk databestand als CSV. Op JSON levert dat geen fout maar onzin: de eerste regel `{` werd de kopregel, dus geen enkele reeks, en het resultaat was een grafiek zonder cijfers. De enige test die dit pad dekte gebruikte een `.csv` en bleef daarom groen — het soort dekking dat een gat juist verbergt. **En een grafiek zonder cijfers bleef stil.** Twee paden meldden het niet: het bestandskiezer-pad gooide de waarschuwing weg, en het bytes-pad (web, drag-drop, URL-import) kende er geen. Dat laatste heeft geen projectmap en kán een `data/…`-verwijzing dus niet oplossen — een grens van het pad, geen fout, maar wel iets om te zeggen in plaats van te verzwijgen. ## Toetsing `make check` groen: 4189 tests, dekking 79,5%. CI blijft pending — er is geen runner. De cherry-pick voegde schoon samen met de herstructurering van `tabs_provider.dart` uit #417 (`TabInfo`/`TabsState` naar een eigen part): deze wijziging zit in het openpad, niet in het verplaatste deel. De twee aanroepen van `_warnUnfilledChartData` zijn losse plekken, geen dubbeling. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

brenno added 1 commit 2026-07-20 16:03:40 +00:00

fix(pakket): grafiekdata uit een pakket wordt gelezen zoals overal elders ... 3fa27bb264

Sinds grafiekdata naar een los bestand verhuisde munt OciDeck altijd `.json`,
maar het uitpakken van een `.ocideck` las elk databestand als CSV. Op JSON
levert dat geen fout maar onzin: de eerste regel `{` werd de kopregel, dus geen
enkele reeks, en het resultaat was een grafiek zonder cijfers. De enige test
die dit pad dekte gebruikte een `.csv` en bleef daarom groen.

Daarnaast bleef een grafiek zonder cijfers stil. Twee paden meldden het niet:
het bestandskiezer-pad gooide de waarschuwing weg, en het bytes-pad (web,
drag-drop, URL-import) kende er geen. Dat laatste heeft geen projectmap en kán
een `data/…`-verwijzing dus niet oplossen — een grens van het pad, geen fout,
maar wel iets om te zeggen in plaats van te verzwijgen.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

brenno merged commit 6de4c30366 into main 2026-07-20 16:03:47 +00:00

brenno referenced this pull request from a commit 2026-07-20 16:03:49 +00:00

Merge pull request 'fix(pakket): grafiekdata uit een pakket wordt gelezen zoals overal elders' (#420) from fix/pakket-grafiekdata-json into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

accepted

Assessed and will be built

  

bug

Something is broken

  

declined

Assessed and will not be built

  

docs

Documentation only

  

duplicate

Already tracked elsewhere

  

enhancement

New feature or improvement request

  

good first issue

Small, self-contained, and a good place to start

  

in-progress

Someone is working on this right now. Do not start it in parallel.

  

needs-info

Waiting for the reporter

  

privacy

Personal data can leak. Handled before anything else.

  

security

A weakness someone could use. Handled before anything else.

  

triage

Awaiting assessment

No labels

accepted

bug

declined

docs

duplicate

enhancement

good first issue

in-progress

needs-info

privacy

security

triage

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

LibreKAT/Ocideck!420

No description provided.