fix for internal/bind/numeric_args.go RewriteQuery function from PR review

Gleb Brozhe · Gleb Brozhe · commit 148de8deea26 · 2024-03-22T11:37:17.000-07:00
diff --git a/internal/bind/numeric_args.go b/internal/bind/numeric_args.go
@@ -56,9 +56,6 @@ func (m NumericArgs) RewriteQuery(sql string, args ...interface{}) (yql string,
 				)
 			}
 			paramIndex := int(p - 1)
-			if newArgs[paramIndex] == nil {
-				return "", nil, xerrors.WithStackTrace(ErrInconsistentArgs)
-			}
 			buffer.WriteString(newArgs[paramIndex].(table.ParameterOption).Name())
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -56,9 +56,6 @@ func (m NumericArgs) RewriteQuery(sql string, args ...interface{}) (yql string,`
`56`	`56`	`)`
`57`	`57`	`}`
`58`	`58`	`paramIndex := int(p - 1)`
`59`		`- if newArgs[paramIndex] == nil {`
`60`		`- return "", nil, xerrors.WithStackTrace(ErrInconsistentArgs)`
`61`		`- }`
`62`	`59`	`buffer.WriteString(newArgs[paramIndex].(table.ParameterOption).Name())`
`63`	`60`	`}`
`64`	`61`	`}`