Skip to content
This repository has been archived by the owner on Jan 18, 2021. It is now read-only.

Query options passed to .rest() are only parsed when 'queryCols' exists #3

Closed
gitbrent opened this issue Oct 13, 2017 · 1 comment
Closed

Query options passed to .rest() are only parsed when 'queryCols' exists #3

gitbrent opened this issue Oct 13, 2017 · 1 comment
Assignees
@gitbrent
Labels
bug
Milestone
1.2.1

Comments

@gitbrent
Copy link
Owner

The .rest() method is ignoring options (not including them in REST queries) when queryCols is not included:

CODE:

sprLib.rest({ url:'/sites/dev/_api/web/lists', queryFilter:'ID eq 99' })

RequestURL:
/sites/dev/_api/web/lists?_=1507783353329

CODE:

sprLib.rest({ url:'/sites/dev/_api/web/lists', queryCols:'ID', queryFilter:'ID eq 99' })

RequestURL:
/sites/dev/_api/web/lists?$select=ID&$top=1000&$filter=ID%20eq%2099&_=1507783353336

@gitbrent gitbrent self-assigned this Oct 13, 2017
@gitbrent gitbrent added the bug label Oct 13, 2017
gitbrent pushed a commit that referenced this issue Oct 13, 2017
Added new section for REST parsing tests
9f4aeb0 
Issue #3
@gitbrent gitbrent added this to the 1.2.1 milestone Oct 13, 2017
gitbrent pushed a commit that referenced this issue Oct 13, 2017
Updated rest param parsing, fix for Issue #3
8288b45
@gitbrent
Copy link
Owner Author

Fixed. Lots of new QUnit tests added for sprLib.rest() param parsing now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@gitbrent gitbrent

Labels
bug
Projects
None yet
Milestone
1.2.1
Development

No branches or pull requests
1 participant
@gitbrent