That's what I think too. Not sure if we need to add some special condition
for custom data, but I agree that we should do the validation.


On Mon, Feb 17, 2014 at 6:59 PM, Les Hazlewood <notifications@github.com>wrote:

> In lib/stormpath-sdk/data_store.rb:
>
> > @@ -149,7 +152,7 @@ def execute_request(http_method, href, body=nil, query=nil)
> >    end
> >
> >    def cache_walk(resource)
> > -    assert_not_nil resource['href'], "resource must have 'href' property"
> > +    # assert_not_nil resource['href'], "resource must have 'href' property"
>
> I'm a bit confused: if you're 'walking' the cache, you need an href to
> store the cached data. We should always assert that any resource being
> saved to the cache has an href property.
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/stormpath/stormpath-sdk-ruby/pull/53/files#r9811234>
> .
>