Merge pull request #408 from onelrdm/master

taowen · web-flow · commit 908eaed15109 · 2019-10-12T21:05:04.000+08:00
skip - tag before spliting parts
diff --git a/reflect_extension.go b/reflect_extension.go
@@ -341,10 +341,10 @@ func describeStruct(ctx *ctx, typ reflect2.Type) *StructDescriptor {
 		if ctx.onlyTaggedField && !hastag && !field.Anonymous() {
 			continue
 		}
-		tagParts := strings.Split(tag, ",")
 		if tag == "-" {
 			continue
 		}
+		tagParts := strings.Split(tag, ",")
 		if field.Anonymous() && (tag == "" || tagParts[0] == "") {
 			if field.Type().Kind() == reflect.Struct {
 				structDescriptor := describeStruct(ctx, field.Type())

Original file line number	Diff line number	Diff line change
`@@ -341,10 +341,10 @@ func describeStruct(ctx ctx, typ reflect2.Type) StructDescriptor {`
`341`	`341`	`if ctx.onlyTaggedField && !hastag && !field.Anonymous() {`
`342`	`342`	`continue`
`343`	`343`	`}`
`344`		`- tagParts := strings.Split(tag, ",")`
`345`	`344`	`if tag == "-" {`
`346`	`345`	`continue`
`347`	`346`	`}`
	`347`	`+ tagParts := strings.Split(tag, ",")`
`348`	`348`	`if field.Anonymous() && (tag == "" \|\| tagParts[0] == "") {`
`349`	`349`	`if field.Type().Kind() == reflect.Struct {`
`350`	`350`	`structDescriptor := describeStruct(ctx, field.Type())`