Support data retrieval when using ngResource promises that return a object instead of a function.

[lius]

No description provided.

[l-lin]

Sweet! Thanks!

[lius]

Glad to contribute!

[l-lin]

However, for you information, with a promise, it's not possible to reload the data (ie send a request to the server again, and not just re-render the table).
Indeed, the promise, once resolved, does not re-send request to the server.
That's why the option handles a function that returns a promise so that the promise inside can be resolve multiple times.

[lius]

Considering your advice and how ngResource works in recent versions, I think it's not possible pass a function that returns a promise directly from $resource calls, because actually the return is an object that holds the promise as one of it's properties, and not the promise (offering the possibility to bind directly to $scope without callbacks). This is what I was thinking when trying to change the code to accept a object too, since the use of $resource is a fairly common scenario in angular applications. What you think?

[l-lin]

Indeed, I always used $http or just $resource(...).$promise or a third party lib to fetch the data from the server. Didn't notice that angular evolved ngResource in a way we no longer need a callback.

I'm interested on how you manage to make the module works with the "convential" way.
My attempt did not succeed:

var Persons = $resource('data.json');
$scope.dtOptions = DTOptionsBuilder.fromFnPromise(Persons.query())
      .withPaginationType('full_numbers');

[lius]

Using your example, I would change it to pass the promise has a parameter:

var Persons = $resource('data.json');
$scope.dtOptions = DTOptionsBuilder.fromFnPromise(Persons.query().$promise)
      .withPaginationType('full_numbers');

[lius]

By the way, about binding using $resource without a callback (in recent versions):

$scope.people = Person.query();

[l-lin]

In that case, you can always wrap the promise with a function:

var Persons = $resource('data.json');
$scope.dtOptions = DTOptionsBuilder.fromFnPromise(function(){
        return Persons.query().$promise;
    })
    .withPaginationType('full_numbers');

[lius]

Ok, It works. So, in this case, I think accepting objects alongside functions it's a matter of API decision to the library. It's up to you! Thanks