add refresh method to models

[tchak]

fix #51

next step : refresh method on ModelArray

[tchak]

I am not sure it make sens to have refresh on a generic ModelArray, feedback from @tomdale or @wycats is more then welcome

[wycats]

Agreed. What is the use-case for this feature?

[tchak]

The most useful is to refresh findQuery results

For example a searchController could be implemented as

App.searchController = Em.ArrayProxy.create({
  content: App.store.findQuery(App.Item, {q:''}),

  search: function(text) {
    var content = this.get('content');
    content.set('query', {q: text});
    content.refresh();
  }
})

[tomdale]

👎 from me on this in its current state.

My reservations:

1. This does not seem like the correct solution to Refreshing partially loaded records #51. In the scenario described there, we have models that are partially loaded during queries. I'd consider it a hack that you have to refresh the model to finish populating the record. I would be very confused reading code that calls refresh() on a record in order to load extended details--I'd think that we would be refreshing it to get the latest value. Partially-loaded records are common and I think we need the right concepts to handle them.
2. While refreshing a model is important, I don't think this implementation has thought through all of the possible cases. It basically just blindly resets the record and starts loading again. For example, right now if I call refresh() on a model that has been updated but not yet saved, those changes will be lost and the model will eventually return to the loaded state.
3. The callAdapter() refactor is very confusing to me. Reading through the code, I see callAdapter() often but it is hard to reason about what it is actually doing, since "call adapter" is very vague. I'm on board with the refactor since it DRYs up the code but I think you should consider renaming this something like findFromAdapter or something else that makes its behavior explicit.

[tchak]

I agree about partial loading.

What about refresh on AdapterPopulatedModelArray ? If I remove for now refresh on model and fix findFromAdapter would it be ok for you?

[tchak]

closing for now
Need full rewrite against master and need to take in to account possible overload of dirty records