GrottoCenter / grottocenter-api / 25119995935

86%

develop: 87%

Build Type

Pull #1542

github

Committed by ClemRz

Commit Message

fix(organization): coerce empty-string coordinates to null before DB update

Empty latitude/longitude strings from the frontend caused a PostgreSQL
22P02 error (invalid input syntax for type numeric) on PUT
/api/v1/organizations/:id.

Add coerceToNumeric utility (mirrors coerceToInt) that converts empty
and whitespace-only strings to null. Apply it in
GrottoService.getConvertedDataFromClientRequest, GrottoService.createGrotto,
CaveService.getConvertedDataFromClient, and the cave update controller.

Remove the ad-hoc delete-on-empty-string hack from createGrotto.

Pull Request Pull Request #1542: fix(organization): coerce empty-string coordinates to null

Coverage Stats

2877 of 3474 branches covered (82.82%)

Branch coverage included in aggregate %.

12 of 14 new or added lines in 4 files covered. (85.71%)

7 existing lines in 3 files now uncovered.

5972 of 6778 relevant lines covered (88.11%)

95.22 hits per line