diff options
author | Andrew Gerrand <adg@golang.org> | 2011-09-19 11:54:29 +1000 |
---|---|---|
committer | Andrew Gerrand <adg@golang.org> | 2011-09-19 11:54:29 +1000 |
commit | 40bfc6421ead71d20bfc9968e6bdd6d849991ec5 (patch) | |
tree | 4e2eb0207ce14de92ad8e9a115ba45195ee8382e | |
parent | 6ecd7af254cc32a87f0a784367adc86d50e3dea3 (diff) | |
download | go-40bfc6421ead71d20bfc9968e6bdd6d849991ec5.tar.gz |
[release-branch.r60] json: fix decode bug with struct tag names with ,opts being ignored
??? CL 4965049 / f8e4df3c4048
json: fix decode bug with struct tag names with ,opts being ignored
When the encoder was updated to respect the ",omitempty"
struct tag options, the decoder half was never updated to know
about the new struct tag format. (the format is now an optional
name, followed by zero or more ",option" strings)
This only affected people who used ",omitempty" along with
a field name. In that case, the serialized JSON wouldn't
decode to the original value.
R=golang-dev, dvyukov
CC=golang-dev
http://codereview.appspot.com/4965049
???
R=dsymonds
CC=golang-dev
http://codereview.appspot.com/5029043
-rw-r--r-- | src/pkg/json/decode.go | 12 | ||||
-rw-r--r-- | src/pkg/json/decode_test.go | 6 |
2 files changed, 16 insertions, 2 deletions
diff --git a/src/pkg/json/decode.go b/src/pkg/json/decode.go index 4f6562bd5..6782c76c4 100644 --- a/src/pkg/json/decode.go +++ b/src/pkg/json/decode.go @@ -486,7 +486,7 @@ func (d *decodeState) object(v reflect.Value) { if isValidTag(key) { for i := 0; i < sv.NumField(); i++ { f = st.Field(i) - if f.Tag.Get("json") == key { + if tagName(f.Tag.Get("json")) == key { ok = true break } @@ -918,3 +918,13 @@ func unquoteBytes(s []byte) (t []byte, ok bool) { } return b[0:w], true } + +// tagName extracts the field name part out of the "json" struct tag +// value. The json struct tag format is an optional name, followed by +// zero or more ",option" values. +func tagName(v string) string { + if idx := strings.Index(v, ","); idx != -1 { + return v[:idx] + } + return v +} diff --git a/src/pkg/json/decode_test.go b/src/pkg/json/decode_test.go index a855d6048..4c179de5d 100644 --- a/src/pkg/json/decode_test.go +++ b/src/pkg/json/decode_test.go @@ -262,7 +262,8 @@ type All struct { Float32 float32 Float64 float64 - Foo string `json:"bar"` + Foo string `json:"bar"` + Foo2 string `json:"bar2,dummyopt"` PBool *bool PInt *int @@ -331,6 +332,7 @@ var allValue = All{ Float32: 14.1, Float64: 15.1, Foo: "foo", + Foo2: "foo2", String: "16", Map: map[string]Small{ "17": {Tag: "tag17"}, @@ -391,6 +393,7 @@ var allValueIndent = `{ "Float32": 14.1, "Float64": 15.1, "bar": "foo", + "bar2": "foo2", "PBool": null, "PInt": null, "PInt8": null, @@ -481,6 +484,7 @@ var pallValueIndent = `{ "Float32": 0, "Float64": 0, "bar": "", + "bar2": "", "PBool": true, "PInt": 2, "PInt8": 3, |