Field.Set: use of reflect.Indirect about structs

[fatih]

Hm but reflect.Indirect is returning the value itself if it's not a pointer. And I need the value to set the field. See: http://golang.org/src/pkg/reflect/value.go?s=66983:67011#L2249

Here are the tests fro set https://github.com/fatih/structs/blob/master/field_test.go#L46

Could you add a test case that invalidates this test so I can be sure what the real problem is?

[pikanezi]

I've forked your repository and added a test that fails.
https://github.com/pikanezi/structs

Basically:

type Bar struct {
    A string
}

type Foo struct {
    B *Bar
}


foo := &Foo{
    B: nil,
}
structInfo := structs.New(foo)
bar := &Bar {
    A: "helloWorld",
}

err := structInfo.Field("B").Set(bar); err != nil {
    fmt.Println(err) // Prints "field is not settable"
}

[fatih]

Thanks! I'll look it into asap.

[fatih]

Thanks for the case. I've added it to the test cases and fix the problem. As you said earlier there is no need to take the address as that would disallow to set it. 👍

[pikanezi]

Glad I could help :)