OpenSourceArk/Ghost
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #6240 from halfdan/6239-ghost-api

Browse Source 
Fix non-idempotent Ghost API helper
This commit is contained in:
Hannah Wolfe 2015-12-20 14:30:34 +00:00
commit fe91944d43
2 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
core/shared/ghost-url.js
View File

@ -30,7 +30,8 @@
url = {
api: function () {
var args = Array.prototype.slice.call(arguments),
queryOptions;
queryOptions,
requestUrl = apiUrl;
if (args.length && typeof args[args.length - 1] === 'object') {
queryOptions = args.pop();
@ -43,11 +44,11 @@
if (args.length) {
args.forEach(function (el) {
apiUrl += el.replace(/^\/|\/$/g, '') + '/';
requestUrl += el.replace(/^\/|\/$/g, '') + '/';
});
}
return apiUrl + generateQueryString(queryOptions);
return requestUrl + generateQueryString(queryOptions);
}
};

16
core/test/unit/ghost_url_spec.js
View File

@ -127,4 +127,20 @@ describe('Ghost Ajax Helper', function () {
rendered.should.match(/include=tags%2Ctests/);
rendered.should.match(/page=2/);
});
it('should be idempotent', function () {
configUtils.set({
url: 'https://testblog.com/blog/'
});
ghostUrl.init({
clientId: 'ghost-frontend',
clientSecret: 'notasecret',
url: configUtils.config.apiUrl()
});
var rendered = ghostUrl.url.api('posts', {limit: 3}),
rendered2 = ghostUrl.url.api('posts', {limit: 3});
rendered.should.equal(rendered2);
});
});