value|defaultValue={Symbol|Function} should be ignored, not stringified

[gaearon]

Regression in master from #11534.
Found it thanks to the attribute fixture snapshots.

[nhunzaker]

Just for clarity: This is the correct behavior for Symbols:

With my change, value and defaultValue never pass through the property sanitation layer. React tries to stringify symbols, which throws an error, and functions get turned into "function () {}".

[gaearon]

Now that I looked at this again, the update codepath was already broken. This just made the initial code path match it. So this is actually an improvement. We should still get #11741 to completion to have consistent sane behavior, but IMO this is not as urgent as I thought at first.

[kanlanc]

Is it resolved or can I start working on it?

Sorry new to open source

[gaearon]

I think the issue still exists, it’s just more consistent now

[kanlanc]

just for reference how do you call dibs on a bug