ElyBy-Mirror/accounts-frontend
1
0
Fork 0
mirror of https://github.com/elyby/accounts-frontend.git synced 2024-12-28 16:00:24 +05:30
Code Issues Packages Projects Releases Wiki Activity

#337: properly serialize null values in request.buildQuery

Browse Source
This commit is contained in:
SleepWalker 2017-06-12 22:02:15 +03:00
parent 59000926ab
commit 41661dbea5
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/services/request/request.js
View File

@ -114,7 +114,7 @@ function doFetch(url, options = {}) {
* @return {string} * @return {string}
*/ */
function convertQueryValue(value) { function convertQueryValue(value) {
if (typeof value === 'undefined') { if (typeof value === 'undefined' || value === null) {
return ''; return '';
} }

3
src/services/request/request.test.js
View File

@ -57,12 +57,13 @@ describe('services/request', () => {
it('should build query', () => { it('should build query', () => {
const data = { const data = {
notSet: undefined, notSet: undefined,
notSet2: null,
numeric: 1, numeric: 1,
complexString: 'sdfgs sdfg ', complexString: 'sdfgs sdfg ',
positive: true, positive: true,
negative: false negative: false
}; };
const expectedQs = 'notSet=&numeric=1&complexString=sdfgs%20sdfg%20&positive=1&negative=0'; const expectedQs = 'notSet=&notSet2=&numeric=1&complexString=sdfgs%20sdfg%20&positive=1&negative=0';
expect(request.buildQuery(data), 'to equal', expectedQs); expect(request.buildQuery(data), 'to equal', expectedQs);
}); });