JsonRest object store should support use HTTP Headers

[dancrumb]

JsonRest? does not allow users to set HTTP Headers.

This would be a very useful feature, especially for dealing with sotres that require any level of authentication.

A pull request has been sent to the dojo repo on github.

Here is the commit to our fork: ​https://github.com/vodori/dojo/commit/c23ae27888805b1bf6eaa1891bf6310c78af400a

[dancrumb]

ttrenka recommends that this be assigned to kzyp

[Kenneth G. Franqueiro]

I'd note that it looks like this changeset may have missed fixing the remove method.

[dancrumb]

That was on purpose (although the decision may have been incorrect).

I didn't want to modify the interface of this object, and remove only takes an id.

However, in retrospect, it probably makes more sense to add an options parameter to remove than to 'maintain' an interface, especially as adding an options parameter wouldn't break any pre-existing users

[Colin Snover]

I’ve seen several people ask/complain about this. Since we’re still not at feature freeze on 1.8 it would be great to see this, since it’s low-effort & high-impact.

[Colin Snover]

In [28877]:

Enhance dojo/store/JsonRest to allow headers to be provided:

1. for all calls, not just GET calls;
2. directly on the store that are sent automatically with all requests for that store.

Fixes #15406.

Also fixes some doc block whitespace issues, Refs #13101.