batmanjs/batman — pull request #910 by nsimmons (2013-11-25 00:35:18+00:00)
Title: Should not cache properties whose keypath contains an accessor that is not cacheable

While digging through how properties and accessor work, I came across this bug.

I don't think it makes sense to cache properties if one of the accessors in the property's keypath is explicitly set to cache: false However this was done because of #380. The root problem is #403 so I will see if I can fix that first.

Here is the code causing this issue: https://github.com/batmanjs/batman/blob/master/src/observable/keypath.coffee#L12-L13

I have added a broken regression test for this.

/cc @pushrax @kristianpd @nickjs