Gleipnir/nidus-sync
4
0
Fork 0
Code Issues 18 Pull requests Projects Releases Packages Wiki Activity Actions

lint: log errors when adding errors from flyover uploads

Browse source
This commit is contained in:
Eli Ribble 2026-05-21 13:56:48 +00:00
parent d120ed05f2
commit 41ed443d54
No known key found for this signature in database
1 changed files with 10 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

13
platform/csv/flyover.go
View file

@ -19,6 +19,7 @@ import (
"source.gleipnir.technology/Gleipnir/nidus-sync/db/enums" "source.gleipnir.technology/Gleipnir/nidus-sync/db/enums"
"source.gleipnir.technology/Gleipnir/nidus-sync/db/models" "source.gleipnir.technology/Gleipnir/nidus-sync/db/models"
"source.gleipnir.technology/Gleipnir/nidus-sync/h3utils" "source.gleipnir.technology/Gleipnir/nidus-sync/h3utils"
"source.gleipnir.technology/Gleipnir/nidus-sync/lint"
"source.gleipnir.technology/Gleipnir/nidus-sync/platform/file" "source.gleipnir.technology/Gleipnir/nidus-sync/platform/file"
"source.gleipnir.technology/Gleipnir/nidus-sync/platform/geom" "source.gleipnir.technology/Gleipnir/nidus-sync/platform/geom"
"source.gleipnir.technology/Gleipnir/nidus-sync/platform/types" "source.gleipnir.technology/Gleipnir/nidus-sync/platform/types"
@ -191,19 +192,25 @@ func insertFlyover(ctx context.Context, txn bob.Tx, file *models.FileuploadFile,
if err == nil { if err == nil {
setter.Condition = omit.From(condition) setter.Condition = omit.From(condition)
} else { } else {
addError(ctx, txn, c, int32(line_number), int32(i), fmt.Sprintf("'%s' is not a pool condition that we recognize. It should be one of %s", value, poolConditionValidValues())) lint.LogOnErr(func() error {
return addError(ctx, txn, c, int32(line_number), int32(i), fmt.Sprintf("'%s' is not a pool condition that we recognize. It should be one of %s", value, poolConditionValidValues()))
}, "add flyover comment error")
continue continue
} }
case headerFlyoverLatitude: case headerFlyoverLatitude:
lat, err = strconv.ParseFloat(value, 64) lat, err = strconv.ParseFloat(value, 64)
if err != nil { if err != nil {
addError(ctx, txn, c, int32(line_number), int32(i), fmt.Sprintf("'%s' is not decimal value", value)) lint.LogOnErr(func() error {
return addError(ctx, txn, c, int32(line_number), int32(i), fmt.Sprintf("'%s' is not decimal value", value))
}, "add flyover latitude error")
continue continue
} }
case headerFlyoverLongitude: case headerFlyoverLongitude:
lng, err = strconv.ParseFloat(value, 64) lng, err = strconv.ParseFloat(value, 64)
if err != nil { if err != nil {
addError(ctx, txn, c, int32(line_number), int32(i), fmt.Sprintf("'%s' is not decimal value", value)) lint.LogOnErr(func() error {
return addError(ctx, txn, c, int32(line_number), int32(i), fmt.Sprintf("'%s' is not decimal value", value))
}, "add flyover longitude error")
continue continue
} }
} }