Error in bindConnectorMethod

[brigand]

I'm getting this error in shallowEqual from bindConnectorMethod where it expects currentOptions to be an object, but it's undefined.

It doesn't appear to impair functionality (dragging is a bit glitchy but maybe unrelated), and only happens when inserting a new item.

(Object.keys) Uncaught TypeError: Cannot convert undefined or null to object

Any tips on how to narrow it down to a repro case?

I'd like to add a better error message once I figure it out, assuming it's not an actual bug.

Full stack trace.

Edit:
Caused by this diff: https://gist.github.com/brigand/facd66d909216e6df714
Backend: HTML5Backend

[marudor]

I've got the same Issue in my Project. I worked around it by using the shallowEqual Library instead of the shallowEqual in reactDnD.
Main Problem seems to be the a === b check in the shallowEqual.
a is undefined while b is null.

@gaearon any reason to shallowEqual instead of shallowEqualScalar (or the shallowEqual npm module)
Just changing that worked for me. See my Fork (https://github.com/marudor/react-dnd)

[gaearon]

I'd appreciate a runnable example reproducing this.

[marudor]

For me your latest change (1.1.6) fixed this.
I assume due to your Change in dnd-core.

[gaearon]

Interesting. I'll close for now; please reopen if you have a reproducible example.

[marudor]

Okay, this is still an Issue.
Sadly I still have no real small example to show you. Only able to reproduce it in our company app which I can't share.
It is fixed by my scoped release (I just change your shallowEqual implementation with the shallowequal npm package)

Any reason you're using your own instead of the npm package?
I guess using your shallowEqualScalar method everywhere would also work.

[gaearon]

shallowEqualScalar implements completely different logic from shallowEqual.
It returns false if either value is object, which is the opposite of the effect I want here.

[gaearon]

Should be fixed in 2.0.1.

[gaearon]

Actually you should use 2.0.2, it includes a more correct fix.

[marudor]

Thanks, looks fixed to me. (Although I just quickly checked, I will tell you if I encounter this again)