opravit implementace boolean query parametrů
V dokumentaci uvádíme příklady použití boolean parametrů jako ?onlyMonitored=true
. IMHO je lepší testovat query parametr oproti === "true"
jako tomu je u:
Je totiž pro mne nelogické použít ?onlyMonitored=false
a získat přesto pouze vyfiltrované dle pravidla (req.query.PARAMETER || false
se kvůli typu string vyhodnotí vždy jako true
, jako false
jen pokud je undefined
, tedy nepřítomen), děje se tomu tak u jiných routerů viz příklady (parametry includeRoute
, a další, resp. includePositions
, resp. aggregate
):
https://gitlab.com/operator-ict/golemio/code/output-gateway/-/blob/development/src/resources/gtfs/GTFSRouter.ts#L130 https://gitlab.com/operator-ict/golemio/code/output-gateway/-/blob/development/src/resources/vehiclepositions/VehiclePositionsRouter.ts#L34 https://gitlab.com/operator-ict/golemio/code/output-gateway/-/blob/c0eef87e15861240a3c6213e8fe34470fbe57c92/src/resources/bicyclecounters/BicycleCountersRouter.ts#L78