Skip to content

content API update and some other features #113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
whoisterencelee wants to merge 8 commits into from
Closed

content API update and some other features #113

Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
e6f361c
added repo.req access point to allow making custom request without se…
whoisterencelee Mar 24, 2014
8f3b016
added repo.req access point to allow making custom request withou…
whoisterencelee Mar 24, 2014
20653f7
Add option to limit number of commits to list with getCommits
whoisterencelee Mar 24, 2014
6e4c5a6
Add status===0 in request to allow offline use of xhr
whoisterencelee Mar 25, 2014
3280a97
fix request header to default to json and allow custom Accept headers
whoisterencelee Mar 30, 2014
187b4a3
added datatype option to contents calls
whoisterencelee Mar 30, 2014
c558c6b
further fix to req response datatype handling
whoisterencelee Mar 30, 2014
fea7913
added contents_update function
whoisterencelee Mar 31, 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
30 changes: 23 additions & 7 deletions github.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,28 +33,28 @@
//
// I'm not proud of this and neither should you be if you were responsible for the XMLHttpRequest spec.

function _request(method, path, data, cb, raw, sync) {
function _request(method, path, data, cb, datatype, sync) {
function getURL() {
var url = path.indexOf('//') >= 0 ? path : API_URL + path;
return url + ((/\?/).test(url) ? "&" : "?") + (new Date()).getTime();
}

var xhr = new XMLHttpRequest();
if (!raw) {xhr.dataType = "json";}
if (!datatype)datatype="json";

xhr.open(method, getURL(), !sync);
if (!sync) {
xhr.onreadystatechange = function () {
if (this.readyState == 4) {
if (this.status >= 200 && this.status < 300 || this.status === 304) {
cb(null, raw ? this.responseText : this.responseText ? JSON.parse(this.responseText) : true, this);
if (this.status >= 200 && this.status < 300 || this.status === 304 || this.status === 0) {
cb(null, /^(?!json)/.test(datatype) ? this.responseText : this.responseText ? JSON.parse(this.responseText) : true, this);
} else {
cb({path: path, request: this, error: this.status});
}
}
}
};
xhr.setRequestHeader('Accept','application/vnd.github.raw+json');
xhr.setRequestHeader('Accept','application/vnd.github.'+datatype);
xhr.setRequestHeader('Content-Type','application/json;charset=UTF-8');
if ((options.token) || (options.username && options.password)) {
xhr.setRequestHeader('Authorization', options.token
Expand Down Expand Up @@ -444,10 +444,19 @@
// Get contents
// --------

this.contents = function(branch, path, cb, sync) {
return _request("GET", repoPath + "/contents?ref=" + branch + (path ? "&path=" + path : ""), null, cb, 'raw', sync);
this.contents = function(branch, path, cb, sync, datatype) {
if(!datatype)datatype='raw';// use 'json' if you want to get sha of content as well, it's need for update contents
return _request("GET", repoPath + "/contents/" + (path ? path : "") + "?ref=" + branch, null, cb, datatype, sync);
};

// Update contents
// --------

this.contents_update = function(branch, path, message, content, previous_sha, cb, sync) {
return _request("PUT", repoPath + "/contents/" + (path ? path : "") + "?ref=" + branch,
{ message:message, content:Base64.encode(content), sha:previous_sha }, cb, null, sync);
}

// Fork repository
// -------

Expand Down Expand Up @@ -612,6 +621,7 @@
// List commits on a repository. Takes an object of optional paramaters:
// sha: SHA or branch to start listing commits from
// path: Only commits containing this file path will be returned
// per_page: number of commits to list
// since: ISO 8601 date - only commits after this date will be returned
// until: ISO 8601 date - only commits before this date will be returned
// -------
Expand All @@ -626,6 +636,9 @@
if (options.path) {
params.push("path=" + encodeURIComponent(options.path));
}
if (options.per_page) {
params.push("per_page="+ encodeURIComponent(options.per_page));
}
if (options.since) {
var since = options.since;
if (since.constructor === Date) {
Expand All @@ -645,6 +658,9 @@
}
_request("GET", url, null, cb);
};

this.req=_request;

};

// Gists API
Expand Down