blackteay/gin
3
0
Fork 0
mirror of https://github.com/gin-gonic/gin.git synced 2025-10-18 23:12:17 +08:00
Code Issues Projects Releases Wiki Activity

fix bug that tag form:",default=1" are not effective in some cases, e.g: http://host/list?page_no=&page_size=

Browse Source
This commit is contained in:
Jerry 2020-05-06 10:25:23 +08:00
parent 235898e642
commit 9883d05806
2 changed files with 15 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
binding/form_mapping.go
View File

@ -163,8 +163,10 @@ func setByForm(value reflect.Value, field reflect.StructField, form map[string][
val = opt.defaultValue
}
if len(vs) > 0 {
if len(vs) > 0 && vs[0] != "" {
val = vs[0]
} else {
val = opt.defaultValue
}
return true, setWithProperType(val, value, field)
}

10
binding/form_mapping_test.go
View File

@ -63,13 +63,21 @@ func TestMappingBaseTypes(t *testing.T) {
func TestMappingDefault(t *testing.T) {
var s struct {
Int int `form:",default=9"`
PageNo int `form:"page_no,default=9"`
String string `form:"string,default=9"`
Slice []int `form:",default=9"`
Array [1]int `form:",default=9"`
}
err := mappingByPtr(&s, formSource{}, "form")
form := map[string][]string{
"page_no": []string{""},
"string": []string{""},
}
err := mappingByPtr(&s, formSource(form), "form")
assert.NoError(t, err)
assert.Equal(t, 9, s.Int)
assert.Equal(t, 9, s.PageNo)
assert.Equal(t, "9", s.String)
assert.Equal(t, []int{9}, s.Slice)
assert.Equal(t, [1]int{9}, s.Array)
}