refactor(#86): buildPolygonLayer — dedup _isMixed tak #90

New issue

Closed

opened 2026-04-21 15:06:41 +02:00 by Ghost · 1 comment

Ghost commented

2026-04-21 15:06:41 +02:00

(Migrated from codeberg.org)

Part of #86 (code-audit scope 1). Zie AUDIT.md §3.

Context

buildPolygonLayer in index.html:999-1043 heeft twee takken:

Normale tak (5 regels) — filter + addLayer
_isMixed-tak (8 regels) — filter op mixed-set + addLayer

Ze delen de meeste logica maar zijn gedupliceerd. Subtiel gedragsverschil in de fall-through-return.

Scope

Beide takken samenvoegen tot één 9-regelige filter
_isMixed setup-flag kan dan weg
Geen functionele wijziging

Verificatie

node test.js → 1437+ groen
npx playwright test → 166 groen
LOC-reductie ~12-14

Effort/Impact/Risico

1 / 2 / 1 (quick win)

Part of #86 (code-audit scope 1). Zie [AUDIT.md §3](../blob/dev/AUDIT.md#3-buildpolygonlayer--_ismixed-vs-normale-tak-indexhtml999-1043-45-loc). ## Context `buildPolygonLayer` in `index.html:999-1043` heeft twee takken: - Normale tak (5 regels) — filter + addLayer - `_isMixed`-tak (8 regels) — filter op mixed-set + addLayer Ze delen de meeste logica maar zijn gedupliceerd. Subtiel gedragsverschil in de fall-through-return. ## Scope - Beide takken samenvoegen tot één 9-regelige filter - `_isMixed` setup-flag kan dan weg - Geen functionele wijziging ## Verificatie - [ ] `node test.js` → 1437+ groen - [ ] `npx playwright test` → 166 groen - [ ] LOC-reductie ~12-14 ## Effort/Impact/Risico 1 / 2 / 1 (quick win)

Ghost commented

2026-04-21 16:52:57 +02:00

(Migrated from codeberg.org)

Fixed in v2.18.8 — unified buildPolygonLayer filter; ~14 LOC saved per AUDIT §3.3.