From 41ed443d54da93b7745204e91323cca6825b4df5 Mon Sep 17 00:00:00 2001 From: Eli Ribble Date: Thu, 21 May 2026 13:56:48 +0000 Subject: [PATCH] lint: log errors when adding errors from flyover uploads --- platform/csv/flyover.go | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/platform/csv/flyover.go b/platform/csv/flyover.go index f29032f1..7ac3b31c 100644 --- a/platform/csv/flyover.go +++ b/platform/csv/flyover.go @@ -19,6 +19,7 @@ import ( "source.gleipnir.technology/Gleipnir/nidus-sync/db/enums" "source.gleipnir.technology/Gleipnir/nidus-sync/db/models" "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/geom" "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 { setter.Condition = omit.From(condition) } 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 } case headerFlyoverLatitude: lat, err = strconv.ParseFloat(value, 64) 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 } case headerFlyoverLongitude: lng, err = strconv.ParseFloat(value, 64) 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 } }