`#[derive(Reflect)]` on enum that wraps another enum does not work

[jenanwise]

Bevy version

Tried: 0.9 and 523072902cecad440b5d2109077e437fcb7580b6

What you did

Deriving Reflect on a simple enum works:

#[derive(Reflect)]
enum Inner {
    Foo,
}

But deriving Reflect on an enum that wraps another enum does not:

#[derive(Reflect)] // <-- the trait `FromReflect` is not implemented for `Inner`
enum Outer {
    Inner(Inner),
}

[jakobhellermann]

You need to derive FromReflect for the inner enum as well. I forgot why it is required, but without it the enum can't be reflected.

[alice-i-cecile]

This should be documented on the Reflect trait via a doc test :)

[MrGVSV]

You need to derive FromReflect for the inner enum as well. I forgot why it is required, but without it the enum can't be reflected.

It's required because of Reflect::apply. Just like with Vec<T: FromReflect> we need to ensure we can create concrete instances of new data. For Vec<T> this means possibly inserting a brand new value. For enums, this means possibly setting it to a different variant.

[MrGVSV]

Should we close this out and open a new issue for the documentation on Reflect (or maybe Enum) to indicate this?

[alice-i-cecile]

Yes please. Ping me when you've made one and I'll do so.

[MrGVSV]

@alice-i-cecile created #6985