Unsetting a store with Grid.set("store", null) is broken in 0.3.9

[kilink]

Before 0.3.9, using Grid.set("store" null) worked for disconnecting a store from the Grid (useful when adding/removing lots of objects from the store).

This now fails on https://github.com/SitePen/dgrid/blob/master/_StoreMixin.js#L110 ,
because the code is trying to read the "notify" property of null. This was changed as part of a fix for #677.

I think simply changing that expression to the following will fix the issue:

if (store && typeof store.notify === "function"){

Alternately, if this isn't how one should unset/disconnect a store from a Grid, there doesn't appear to be an API for that. For what it's worth, I seem to recall this being the way to do it with the old dojox grids (calling setStore(null)).

[kfranqueiro]

Thanks for reporting this. I tried to be careful to poke for regressions like this before tagging 0.3.9, but looks like I missed one. :)

You're right though, there's not really any better way to disconnect a store, particularly since Observable has no way of pausing observe handlers.