database/gdb: fix #2119 #3559
database/gdb: fix #2119 #3559
Conversation
os/gstructs/gstructs_type.go
Outdated
| if t.Kind() != reflect.Struct { | ||
| return []string{} | ||
| } | ||
| return getStructFields(t.Type) |
There was a problem hiding this comment.
这里不能直接改这个基础方法，根据issue描述，似乎是这里没有获取到匿名结构体的Field。这里有一个RecursiveOption的控制你可以研究一下呢？
There was a problem hiding this comment.
@gqcn 这里获取的是最外层结构体，
所以那里是无法获取到Status结构体的字段
There was a problem hiding this comment.
这里不能直接改这个基础方法，根据
issue描述，似乎是这里没有获取到匿名结构体的Field。这里有一个
倒是可以使用gstructs包的其他方法来解析结构体获取所有的字段，当时我看那个方法只有这里使用了，所以就做了改动。
There was a problem hiding this comment.
@gqcn 这里获取的是最外层结构体，
所以那里是无法获取到Status结构体的字段
那截图红框部分没有达到预期，预期是如果内嵌的结构体在没有tag时嵌套结构体的属性会作为Fields返回。可以研究下这个方法为什么失败了呢？
os/gstructs/gstructs_type.go
Outdated
| if t.Kind() != reflect.Struct { | ||
| return []string{} | ||
| } | ||
| return getStructFields(t.Type) |
There was a problem hiding this comment.
@gqcn 这里获取的是最外层结构体，
所以那里是无法获取到Status结构体的字段
那截图红框部分没有达到预期，预期是如果内嵌的结构体在没有tag时嵌套结构体的属性会作为Fields返回。可以研究下这个方法为什么失败了呢？
2.使用gstructs.Fields获取结构体的字段
| @@ -347,21 +347,6 @@ func Test_StructTypeBySlice(t *testing.T) { | |||
| }) | |||
| } | |||
|
|
|||
| func TestType_FieldKeys(t *testing.T) { | |||
|
这个PR漏掉了，我新开了分支合并了这个PR内容并做了一些完善 #4011 |
This PR was missed, I opened a new branch to merge the content of this PR and made some improvements #4011 |
gstructs.Type.FieldKeys() 无法识别匿名结构体的字段，导致带有With的查询时，查询的字段不符合期望 #2119