tablet/gorm
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: Add unused argument Introduced compatibility issues

Browse Source 
Fix: Add unused argument (#4871)
Breaking the previous compatibility. when idx=0, vars seems not to need. 
b8f33a42a4
This commit is contained in:
xiaoyiyu 2023-08-28 18:12:08 +08:00 committed by GitHub
parent e57e5d8884
commit 55d012f179
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
clause/expression.go
View File

@ -68,7 +68,7 @@ func (expr Expr) Build(builder Builder) {
} }
} }
if idx < len(expr.Vars) { if idx > 0 && idx < len(expr.Vars) {
for _, v := range expr.Vars[idx:] { for _, v := range expr.Vars[idx:] {
builder.AddVar(builder, sql.NamedArg{Value: v}) builder.AddVar(builder, sql.NamedArg{Value: v})
} }