Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Fixes code example in docs #7730

Closed
wants to merge 2 commits into from
Closed

Fixes code example in docs #7730

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
69c9266
Fixes code example in docs
winescout Jun 6, 2014
b214dc5
Update resource.js
winescout Jun 7, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion src/ngResource/resource.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -191,7 +191,9 @@ function shallowClearAndCopy(src, dst) {
* usually the resource is assigned to a model which is then rendered by the view. Having an empty
* object results in no rendering, once the data arrives from the server then the object is
* populated with the data and the view automatically re-renders itself showing the new data. This
* means that in most cases one never has to write a callback function for the action methods.
* means that in most cases one never has to write a callback function for the action methods.
* Callbacks are evaluated in the same context as the initiating action method, providing access
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Using the word "context" makes it seems as if the this of the function will be the resource object, which is not the case.

Maybe this line can have the following added to it: [...] arguments, where the value is the resulting resource instance or collection object?

* to objects returned in the request.
*
* The action methods on the class object or instance object can be invoked with the following
* parameters:
Expand Down