refactor: hernoem statnaam → name in provincie_2023.geojson #71

New issue

Closed

opened 2026-04-12 16:10:02 +02:00 by Ghost · 1 comment

Ghost commented

2026-04-12 16:10:02 +02:00

(Migrated from codeberg.org)

Probleem

Provincies gebruiken statnaam als GeoJSON-property, alle andere types (water, country, gewest) gebruiken name. Dit vereist een nameKey-parameter in het polygonTypes-register (#67) en maakt de code iets complexer dan nodig.

Oplossing

Hernoem de statnaam-property naar name in provincie_2023.geojson:

# Eenmalige sed op het bestand
sed -i 's/"statnaam"/"name"/g' provincie_2023.geojson

Dan kan nameKey uit het polygonTypes-register:

// Vóór
province: { nameKey: 'statnaam', ... }
// Na
// nameKey is overal 'name', geen speciale property nodig

Impact

~5 regels vereenvoudigd in het register
Eenmalige GeoJSON-bewerking

Risico

Laag. Controleer na de wijziging dat buildProvinceLayer() nog steeds de juiste provincienamen matcht. Unit tests voor set 54 zijn de regressiecheck.

Afhankelijkheid

Werkt na #67 (polygonTypes-register). Doe dit als laatste van de refactorings — pas als het register stabiel is.

Zie REFACTORING.md voorstel #4

## Probleem Provincies gebruiken `statnaam` als GeoJSON-property, alle andere types (`water`, `country`, `gewest`) gebruiken `name`. Dit vereist een `nameKey`-parameter in het polygonTypes-register (#67) en maakt de code iets complexer dan nodig. ## Oplossing Hernoem de `statnaam`-property naar `name` in `provincie_2023.geojson`: ```bash # Eenmalige sed op het bestand sed -i 's/"statnaam"/"name"/g' provincie_2023.geojson ``` Dan kan `nameKey` uit het polygonTypes-register: ```js // Vóór province: { nameKey: 'statnaam', ... } // Na // nameKey is overal 'name', geen speciale property nodig ``` ## Impact - ~5 regels vereenvoudigd in het register - Eenmalige GeoJSON-bewerking ## Risico Laag. Controleer na de wijziging dat `buildProvinceLayer()` nog steeds de juiste provincienamen matcht. Unit tests voor set 54 zijn de regressiecheck. ## Afhankelijkheid Werkt na #67 (polygonTypes-register). Doe dit als laatste van de refactorings — pas als het register stabiel is. _Zie REFACTORING.md voorstel #4_

Ghost commented

2026-04-12 21:20:55 +02:00

(Migrated from codeberg.org)

Geïmplementeerd in refactor-commit 96e1a18.

Geïmplementeerd in refactor-commit 96e1a18.