shouldBackgroundReloadRecord == true appears to ignore the cache

[jerel]

I'm upgrading an app to Ember 1.3.2 and Ember Data 1.3.4. To acknowledge deprecations I added:

shouldReloadAll: function() {
  return false;
},
shouldBackgroundReloadRecord: function() {
  return true;
},

And I'm seeing strange behavior with regards to locally cached models.

The following scenario causes the same records to be fetched from the api many times even though all channels were preloaded by a parent model:

  model: function() {
    return this.store.peekAll('message');
  },

  {{#each model as |message| }}
    {{message.channel.name}}
  {{/each}}

To display a list of messages /api/channels/1 is called ~20 times within the same runloop.

[wecc]

Thanks for reporting!

I think this should be fixed by #3468

[HeroicEric]

@jerel can you confirm this was fixed by #3468 ?

[bmac]

Closing this should have been fixed by #3468.