# HG changeset patch # User Sascha L. Teichmann # Date 1558080187 -7200 # Node ID 5932f957449386a5cb516984ae7c38098c4c7974 # Parent ad27721d804d2489b0017edce88ba67f5e22dd71 Follow-up for 3350:e640f51b5a4e (Fix reporting of removed reference water levels). Be more precise with error handling: added forgotten .Err() after looping over rows. diff -r ad27721d804d -r 5932f9574493 pkg/imports/wg.go --- a/pkg/imports/wg.go Fri May 17 09:51:28 2019 +0200 +++ b/pkg/imports/wg.go Fri May 17 10:03:07 2019 +0200 @@ -410,24 +410,27 @@ string(*dr.Reflevel2code), string(*dr.Reflevel3code), }) - rwls, err2 := tx.StmtContext(ctx, + rwls, err := tx.StmtContext(ctx, deleteReferenceWaterLevelsStmt).QueryContext(ctx, ic.code.String(), &validity, &currLevels, ) - if err2 != nil { - return nil, err2 + if err != nil { + return nil, err } defer rwls.Close() for rwls.Next() { var delRef string - if err2 = rwls.Scan(&delRef); err2 != nil { - return nil, err2 + if err := rwls.Scan(&delRef); err != nil { + return nil, err } feedback.Warn("Removed reference water level %s from %s", delRef, ic.code) } + if err := rwls.Err(); err != nil { + return nil, err + } case err != nil: feedback.Warn(handleError(err).Error()) if err2 := tx.Rollback(); err2 != nil {