From c18c8e0a6bc0d6b433bc41128d4cf12a2ff41880 Mon Sep 17 00:00:00 2001
From: Levi DeHaan <levi@dehaandev.com>
Date: Wed, 9 Mar 2011 13:40:44 -0800
Subject: [PATCH 01/10] Made some changes to get it working with new api
 requirements added some things, modified some things, I am using it in a
 current project that is running locally and pulling data from github from
 local://

---
 github.js | 285 ++++++++++++++++++++++++++++++++++++------------------
 1 file changed, 192 insertions(+), 93 deletions(-)

diff --git a/github.js b/github.js
index 2509ec0..bda05d3 100644
--- a/github.js
+++ b/github.js
@@ -43,12 +43,37 @@
 
         url += prefix + "callback=" + encodeURIComponent("gh.__jsonp_callbacks[" + id + "]");
         if (authUsername && authToken) {
-            url += "&login=" + authUsername + "&authToken=" + authToken;
+            url += "&login=" + authUsername + "&token=" + authToken;
         }
         script.setAttribute("src", apiRoot + url);
 
         document.getElementsByTagName('head')[0].appendChild(script);
     },
+    
+    jsonp2 = function (url, callback, context) {
+        apiRoot2 = 'https://gist.github.com/'
+        var id = +new Date,
+        script = document.createElement("script");
+
+        while (gh.__jsonp_callbacks[id] !== undefined)
+            id += Math.random(); // Avoid slight possibility of id clashes.
+
+        gh.__jsonp_callbacks[id] = function () {
+            delete gh.__jsonp_callbacks[id];
+            callback.apply(context, arguments);
+        };
+
+        var prefix = "?";
+        if (url.indexOf("?") >= 0)
+            prefix = "&";
+
+        url += prefix + "callback=" + encodeURIComponent("gh.__jsonp_callbacks[" + id + "]");
+       
+        script.setAttribute("src", apiRoot2 + url);
+        console.log('jsonp2');
+        console.log(script);
+        document.getElementsByTagName('head')[0].appendChild(script);
+    },
 
     // Send an HTTP POST. Unfortunately, it isn't possible to support a callback
     // with the resulting data. (Please prove me wrong if you can!)
@@ -57,36 +82,70 @@
     // restrictions on ajax calls. Basically, a form is created that will POST
     // to the GitHub API URL, stuck inside an iframe so that it won't redirect
     // this page, and then submitted.
+    // THIS FUNCTION NOW DEPENDS ON jQuery
     post = function (url, vals) {
-        var
-        form = document.createElement("form"),
-        iframe = document.createElement("iframe"),
-        doc = iframe.contentDocument !== undefined ?
-            iframe.contentDocument :
-            iframe.contentWindow.document,
-        key, field;
-        vals = vals || {};
-
-        form.setAttribute("method", "post");
-        form.setAttribute("action", apiRoot + url);
-        for (key in vals) {
-            if (vals.hasOwnProperty(key)) {
-                field = document.createElement("input");
-                field.type = "hidden";
-                field.value = encodeURIComponent(vals[key]);
-                form.appendChild(field);
+        var key, field;
+        
+        var iframe = $('<iframe name="githubpull" style="display: none;">').appendTo($("body"));
+        var doc = null;
+        var values = "";
+        doc = window.frames["githubpull"].document;
+        
+        console.log(vals);
+        if(_.isArray(vals) || !_.isString(vals))
+        {
+            vals = vals || {};
+            for (key in vals) {
+                if (vals.hasOwnProperty(key)) {
+                    values += "&values[" + key + "]=" + vals[key]
+                }
             }
+        }else if(_.isString(vals)){
+            values = "&" + vals
         }
-
-        iframe.setAttribute("style", "display: none;");
+        var form = document.createElement("form");
+        form.method = "post";
+        form.action = apiRoot + url + "?login=" + authUsername + "&token=" + authToken + values;
+        
+        console.log(form);
+        doc.body.appendChild(form);
+        form.submit();
+    },
+    
+    post2 = function (url, vals) {
+        var key, field;
+        
+        var iframe = $('<iframe name="githubpull" style="display: none;">').appendTo($("body"));
+        var doc = null;
+        var values = "";
+        doc = window.frames["githubpull"].document;
+        
+        console.log(vals);
+        if(_.isArray(vals) || !_.isString(vals))
+        {
+            vals = vals || {};
+            for (key in vals) {
+                if (vals.hasOwnProperty(key)) {
+                    values += "&" + key + "=" + vals[key]
+                }
+            }
+        }else if(_.isString(vals)){
+            values = "&" + vals
+        }
+        var form = document.createElement("form");
+        form.method = "post";
+        form.action = apiRoot + url + "?login=" + authUsername + "&token=" + authToken + values;
+        
+        console.log(form);
         doc.body.appendChild(form);
-        document.body.appendChild(iframe);
         form.submit();
     },
 
     // This helper function will throw a TypeError if the library user is not
     // properly authenticated. Otherwise, it silently returns.
     authRequired = function (username) {
+        console.log("authUsername: " + authUsername);
+        console.log("authToken: " + authToken)
         if (!authUsername || !authToken || authUsername !== username) {
             throw new TypeError("gh: Must be authenticated to do that.");
         }
@@ -159,19 +218,61 @@
     //
     //     TODO: example
     gh.user.prototype.update = function (params) {
-        authRequired(this.username);
-        var key, postData = {
-            login: authUsername,
-            token: authToken
-        };
+        //authRequired(this.username);
+        var key, postData = {};
         for (key in params) {
             if (params.hasOwnProperty(key)) {
-                postData["values["+key+"]"] = encodeURIComponent(params[key]);
+                postData[key] = encodeURIComponent(params[key]);
             }
         }
         post("user/show/" + this.username, postData);
         return this;
     };
+    
+    gh.user.prototype.listKeys = function (callback, context) {
+        jsonp("user/keys", callback, context);
+        return this;
+    }
+    
+    // { "title" : "cheese", "key" : "alkj309jv309mz89eur0923m09cvj23=="
+    gh.user.prototype.addKey = function (params){
+        var key, postData = {};
+        for(key in params){
+            if(params.hasOwnProperty(key)){
+                postData[key] = encodeURIComponent(params[key]);
+            }
+        }
+        post("user/key/add" + postData);
+        return this;
+    }
+    
+    // { "id" : "987" }
+    gh.user.prototype.removeKey = function (params){
+        var key, postData = {};
+        for(key in params){
+            if(params.hasOwnProperty(key)){
+                postData[key] = encodeURIComponent(params[key]);
+            }
+        }
+        post("user/key/remove" + postData);
+        return this;
+    }
+    
+    // "email=test@test.com"
+    gh.user.prototype.addEmail = function (params){
+        post("user/email/add" + params);
+        return this;
+    }
+    
+    // "email=test@test.com"
+    gh.user.prototype.removeEmail = function (params){
+        post("user/email/remove" + params);
+        return this;
+    }
+    
+    gh.user.prototype.listEmail = function (callback, context){
+        jsonp("user/emails", callback, context);
+    }
 
     // Get a list of who this user is following.
     //
@@ -192,7 +293,7 @@
     //
     //     TODO: example
     gh.user.prototype.follow = function (user) {
-        authRequired.call(this);
+        //authRequired.call(this.username);
         post("user/follow/" + user);
         return this;
     };
@@ -202,7 +303,7 @@
     //
     //     TODO: example
     gh.user.prototype.unfollow = function (user) {
-        authRequired.call(this);
+        //authRequired.call(this);
         post("user/unfollow/" + user);
         return this;
     };
@@ -240,7 +341,7 @@
     // they are just a collaborator on, and do not own). Must be authenticated
     // as this user.
     gh.user.prototype.pushable = function (callback, context) {
-        authRequired(authUsername);
+        //authRequired(authUsername);
         jsonp("repos/pushable", callback, context);
     };
 
@@ -250,7 +351,7 @@
             jsonp(this.username, callback, context);
             return this;
         }
-    );
+        );
 
     // Search users for `query`.
     gh.user.search = function (query, callback, context) {
@@ -307,32 +408,32 @@
     // Get all tags for this repo.
     gh.repo.prototype.tags = function (callback, context) {
         jsonp("repos/show/" + this.user + "/" + this.repo + "/tags",
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Get all branches in this repo.
     gh.repo.prototype.branches = function (callback, context) {
         jsonp("repos/show/" + this.user + "/" + this.repo + "/branches",
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Gather line count information on the language(s) used in this repo.
     gh.repo.prototype.languages = function (callback, context) {
         jsonp("/repos/show/" + this.user + "/" + this.repo + "/languages",
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Gather data on all the forks of this repo.
     gh.repo.prototype.network = function (callback, context) {
         jsonp("repos/show/" + this.user + "/" + this.repo + "/network",
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
@@ -343,16 +444,16 @@
         if (showAnon)
             url += "/anon";
         jsonp(url,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Get all of the collaborators for this repo.
     gh.repo.prototype.collaborators = function (callback, context) {
         jsonp("repos/show/" + this.user + "/" + this.repo + "/collaborators",
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
@@ -383,17 +484,18 @@
         post("repo/set/public/" + this.repo);
         return this;
     };
-
+    
     // Search for repositories. `opts` may include `start_page` or `language`,
     // which must be capitalized.
     gh.repo.search = function (query, opts, callback, context) {
-        var url = "repos/search/" + query.replace(" ", "+");
+         var url = "repos/search/" + query.replace(" ", "+");
         if (typeof opts === "function") {
             opts = {};
             callback = arguments[1];
             context = arguments[2];
         }
         url += "?" + paramify(opts);
+        jsonp(url, callback, context);
         return this;
     };
 
@@ -430,24 +532,24 @@
 
     gh.commit.prototype.show = function (callback, context) {
         jsonp("commits/show/" + this.user + "/" + this.repo + "/" + this.sha,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Get a list of all commits on a repos branch.
     gh.commit.forBranch = function (user, repo, branch, callback, context) {
         jsonp("commits/list/" + user + "/" + repo + "/" + branch,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Get a list of all commits on this path (file or dir).
     gh.commit.forPath = function (user, repo, branch, path, callback, context) {
         jsonp("commits/list/" + user + "/" + repo + "/" + branch + "/" + path,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
@@ -464,16 +566,16 @@
     // View this issue's info.
     gh.issue.prototype.show = function (callback, context) {
         jsonp("issues/show/" + this.user + "/" + this.repo + "/" + this.number,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Get a list of all comments on this issue.
     gh.issue.prototype.comments = function (callback, context) {
         jsonp("issues/comments/" + this.user + "/" + this.repo + "/" + this.number,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
@@ -517,24 +619,27 @@
     // Comment on this issue as the user that is authenticated.
     gh.issue.prototype.comment = function (comment) {
         authRequired(authUsername);
-        post("/issues/comment/" + user + "/" + repo + "/" + this.number, {
+        post2("issues/comment/" + this.user + "/" + this.repo + "/" + this.number, {
             comment: comment
         });
         return this;
     };
 
     // Get all issues' labels for the repo.
-    gh.issue.labels = function (user, repo) {
+    gh.issue.prototype.labels = function (user, repo, callback, context) {
         jsonp("issues/labels/" + user + "/" + repo,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Open an issue. Must be authenticated.
     gh.issue.open = function (repo, title, body) {
+        console.log(repo);
+        console.log(title);
+        console.log(body);
         authRequired(authUsername);
-        post("issues/open/" + authUsername + "/" + repo, {
+        post2("issues/open/" + authUsername + "/" + repo, {
             title: title,
             body: body
         });
@@ -544,8 +649,8 @@
     // Search a repository's issue tracker. `state` can be "open" or "closed".
     gh.issue.search = function (user, repo, state, query, callback, context) {
         jsonp("/issues/search/" + user + "/" + repo + "/" + state + "/" + query,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
@@ -553,8 +658,8 @@
     // "closed".
     gh.issue.list = function (user, repo, state, callback, context) {
         jsonp("issues/list/" + user + "/" + repo + "/" + state,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
@@ -567,21 +672,15 @@
         this.id = id;
     };
 
-    gh.gist.prototype.show = withTempApiRoot(
-        "http://gist.github.com/api/v1/json/",
-        function (callback, context) {
-            jsonp(this.id, callback, cont);
+    gh.gist.prototype.show = function (callback, context) {
+            jsonp2("api/v1/json/" + this.id, callback, context);
             return this;
-        }
-    );
+        };
 
-    gh.gist.prototype.file = withTempApiRoot(
-        "http://gist.github.com/raw/v1/json/",
-        function (filename, callback, context) {
-            jsonp(this.id + "/" + filename, callback, cont);
+    gh.gist.prototype.file = function (filename, callback, context) {
+            jsonp2("raw/" + this.id + "/" + filename, callback, context);
             return this;
-        }
-    );
+        };
 
     // ### Objects
 
@@ -596,40 +695,40 @@
     // Get the contents of a tree by tree SHA
     gh.object.prototype.tree = function (sha, callback, context) {
         jsonp("tree/show/" + this.user + "/" + this.repo + "/" + sha,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Get the data about a blob by tree SHA and path
     gh.object.prototype.blob = function (path, sha, callback, context) {
         jsonp("blob/show/" + this.user + "/" + this.repo + "/" + sha + "/" + path,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Get only blob meta
     gh.object.prototype.blobMeta = function (path, sha, callback, context) {
         jsonp("blob/show/" + this.user + "/" + this.repo + "/" + sha + "/" + path + "?meta=1",
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Get list of blobs
     gh.object.prototype.blobAll = function (branch, callback, context) {
         jsonp("blob/all/" + this.user + "/" + this.repo + "/" + branch,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
     // Get meta of each blob in tree
     gh.object.prototype.blobFull = function (sha, callback, context) {
         jsonp("blob/full/" + this.user + "/" + this.repo + "/" + sha,
-              callback,
-              context);
+            callback,
+            context);
         return this;
     };
 
@@ -647,21 +746,21 @@
         "http://github.com/",
         function (nethash, start, end, callback, context) {
             jsonp(this.user + "/" + this.repo + "/network_data_chunk?"
-                  + nethash + "&" + start + "&" + end,
-                  callback,
-                  context);
+                + nethash + "&" + start + "&" + end,
+                callback,
+                context);
             return this;
         }
-    );
+        );
 
     gh.network.prototype.meta = withTempApiRoot(
         "http://github.com/",
         function (callback, context) {
             jsonp(this.user + "/" + this.repo + "/network_meta",
-                  callback,
-                  context);
+                callback,
+                context);
             return this;
         }
-    );
+        );
 
 }(window));
\ No newline at end of file

From a03fc9e3650d8f881e262f3becf1086989eab414 Mon Sep 17 00:00:00 2001
From: Levi DeHaan <levi@dehaandev.com>
Date: Wed, 9 Mar 2011 14:01:46 -0800
Subject: [PATCH 02/10] updated examples and some caveats

---
 README.md | 51 +++++++++++++++++++++++++++++++++++++--------------
 1 file changed, 37 insertions(+), 14 deletions(-)

diff --git a/README.md b/README.md
index 9cbb202..fded07e 100644
--- a/README.md
+++ b/README.md
@@ -1,20 +1,43 @@
 Exposes `gh` to the global environment. Tries to follow both the form of Github
-HTTP API and JS style.
+HTTP API and JS style. 
+You cannot load javascript files in using this library yet, as it gets the data from github inside <script> tags.
+so really the gists wont work unless you know for sure you wont get any with JS in them, same with looking at repo code that has JS in it.
 
-    gh.authenticate("fitzgen", "sdfk32we-FAKE-uydfs7f-rhrwe8r7");
-    var huddlej = gh.user("huddlej");
-    huddlej.show(function (data) {
-        console.log(data.user);
+    //authenticating as yourself, i'd recommend using something like Amplify to store the user credentials in localstorage
+    amplify.store("username", "YourUserName");
+    amplify.store("token", "yourincrediblylongandtotallyawesometoken");
+    gh.authenticate(amplify.store("username"), amplify.store("password"));
+    var user = gh.user(amplify.store("username"));
+    
+   //this pulls data showing your user information from github
+   //puts your gravatar image inside a div
+   user.show(function (data) {
+        var data = data.user
+        console.log(data);
+        $('#somediv").html('<img src="http://gravatar.com/avatar/' + data.gravatar+id + '.jpg?s=80">');
     });
-    huddlej.repos(function (data) {
-        console.log("Number of repos: " + data.repositories.length);
-    });
-    var wujs = gh.repo("fitzgen", "wu.js")
-    wujs.show(function (data) {
-        console.log("Number of watchers: " + data.repository.watchers);
-    });
-    wujs.update({ has_wiki: 0 }); // Unfortunately, no callbacks with POSTs :(
 
+    //show a users repositories, iterates through them and then appends them to a div
+    new repo.constructor.forUser(amplify.store("username"), RepoListCallback, data);
+    
+    function RepoListCallback(data){
+        console.log(data);
+        if(data.repositories.length > 0) {
+           for(var r=0;r<data.repositories.length; r++){
+               $('<h1>' + data.repositories[r].name + '</h1>').appendTo('#somediv');
+         }
+     }
+   
+
+    //grabbing data from a form and sending it to update a user
+    var params = {
+                "company" : $("#companyname").val(),
+                "name"      : $("#name").val
+     }
+
+     gh.user(user).update(params); //There is no callback to this post
+
+   
 COMPLETE
 ========
 
@@ -29,5 +52,5 @@ COMPLETE
 
 TODO
 ====
-
+* Make calls to github to pull repository files without using <script>
 * Documentation
\ No newline at end of file

From 789928699d869737af8051999fbc4fb918e66b8f Mon Sep 17 00:00:00 2001
From: th3m4d0n3 <th3m4d0n3@alien.th3m4db0x.com>
Date: Wed, 9 Mar 2011 15:10:20 -0700
Subject: [PATCH 03/10] modified readme a bit

---
 README.md | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index fded07e..29399ce 100644
--- a/README.md
+++ b/README.md
@@ -3,7 +3,7 @@ HTTP API and JS style.
 You cannot load javascript files in using this library yet, as it gets the data from github inside <script> tags.
 so really the gists wont work unless you know for sure you wont get any with JS in them, same with looking at repo code that has JS in it.
 
-    //authenticating as yourself, i'd recommend using something like Amplify to store the user credentials in localstorage
+    //authenticating as yourself, I'd recommend using something like Amplify to store the user credentials in localstorage
     amplify.store("username", "YourUserName");
     amplify.store("token", "yourincrediblylongandtotallyawesometoken");
     gh.authenticate(amplify.store("username"), amplify.store("password"));
@@ -53,4 +53,4 @@ COMPLETE
 TODO
 ====
 * Make calls to github to pull repository files without using <script>
-* Documentation
\ No newline at end of file
+* Documentation

From 94d69b30841a7a532a0609e594c37f5c1a90d4ba Mon Sep 17 00:00:00 2001
From: th3m4d0n3 <th3m4d0n3@alien.th3m4db0x.com>
Date: Wed, 9 Mar 2011 15:22:09 -0700
Subject: [PATCH 04/10] updated readme

---
 README.md | 78 +++++++++++++++++++++++++++++++------------------------
 1 file changed, 44 insertions(+), 34 deletions(-)

diff --git a/README.md b/README.md
index 29399ce..b93f8e9 100644
--- a/README.md
+++ b/README.md
@@ -1,44 +1,54 @@
-Exposes `gh` to the global environment. Tries to follow both the form of Github
-HTTP API and JS style. 
+Github API Js
+===============
+
+Exposes `gh` to the global environment. Tries to follow both the form of GithubHTTP API and JS style. 
+
 You cannot load javascript files in using this library yet, as it gets the data from github inside <script> tags.
+
 so really the gists wont work unless you know for sure you wont get any with JS in them, same with looking at repo code that has JS in it.
 
-    //authenticating as yourself, I'd recommend using something like Amplify to store the user credentials in localstorage
-    amplify.store("username", "YourUserName");
-    amplify.store("token", "yourincrediblylongandtotallyawesometoken");
-    gh.authenticate(amplify.store("username"), amplify.store("password"));
-    var user = gh.user(amplify.store("username"));
-    
-   //this pulls data showing your user information from github
-   //puts your gravatar image inside a div
-   user.show(function (data) {
-        var data = data.user
-        console.log(data);
-        $('#somediv").html('<img src="http://gravatar.com/avatar/' + data.gravatar+id + '.jpg?s=80">');
-    });
-
-    //show a users repositories, iterates through them and then appends them to a div
-    new repo.constructor.forUser(amplify.store("username"), RepoListCallback, data);
+###Example Code
+
+>##Authentication
+>    //authenticating as yourself, I'd recommend using something like Amplify to store the user credentials in localstorage
+>    `amplify.store("username", "YourUserName");
+>    amplify.store("token", "yourincrediblylongandtotallyawesometoken");
+>    gh.authenticate(amplify.store("username"), amplify.store("password"));
+>    var user = gh.user(amplify.store("username"));`
     
-    function RepoListCallback(data){
-        console.log(data);
-        if(data.repositories.length > 0) {
-           for(var r=0;r<data.repositories.length; r++){
-               $('<h1>' + data.repositories[r].name + '</h1>').appendTo('#somediv');
-         }
-     }
-   
+>##User Information
+>  //this pulls data showing your user information from github
+>   //puts your gravatar image inside a div
+>   `user.show(function (data) {
+>        var data = data.user
+>        console.log(data);
+>        $('#somediv").html('<img src="http://gravatar.com/avatar/' + data.gravatar+id + '.jpg?s=80">');
+>    });`
 
-    //grabbing data from a form and sending it to update a user
-    var params = {
-                "company" : $("#companyname").val(),
-                "name"      : $("#name").val
-     }
+>##Repository Information
+>    //show a users repositories, iterates through them and then appends them to a div
+>    `new repo.constructor.forUser(amplify.store("username"), RepoListCallback, data);`
+>    
+>    `function RepoListCallback(data){
+>        console.log(data);
+>        if(data.repositories.length > 0) {
+>           for(var r=0;r<data.repositories.length; r++){
+>               $('<h1>' + data.repositories[r].name + '</h1>').appendTo('#somediv');
+>         }
+>     }`
+   
 
-     gh.user(user).update(params); //There is no callback to this post
+>##Sending data
+>    //grabbing data from a form and sending it to update a user
+>    `var params = {
+>                "company" : $("#companyname").val(),
+>                "name"      : $("#name").val
+>     }`
+>
+>     `gh.user(user).update(params);`   //There is no callback available for sending data
 
    
-COMPLETE
+###COMPLETE
 ========
 
 * Authentication
@@ -50,7 +60,7 @@ COMPLETE
 * Network
 * Objects
 
-TODO
+###TODO
 ====
 * Make calls to github to pull repository files without using <script>
 * Documentation

From 39f3231bf7c51ce73ec923966dc493a9e060fb83 Mon Sep 17 00:00:00 2001
From: th3m4d0n3 <th3m4d0n3@alien.th3m4db0x.com>
Date: Wed, 9 Mar 2011 15:24:05 -0700
Subject: [PATCH 05/10] oops had to fix the readme

---
 README.md | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index b93f8e9..7b84fd3 100644
--- a/README.md
+++ b/README.md
@@ -3,7 +3,7 @@ Github API Js
 
 Exposes `gh` to the global environment. Tries to follow both the form of GithubHTTP API and JS style. 
 
-You cannot load javascript files in using this library yet, as it gets the data from github inside <script> tags.
+You cannot load javascript files in using this library yet, as it gets the data from github inside `<script>` tags.
 
 so really the gists wont work unless you know for sure you wont get any with JS in them, same with looking at repo code that has JS in it.
 
@@ -62,5 +62,5 @@ so really the gists wont work unless you know for sure you wont get any with JS
 
 ###TODO
 ====
-* Make calls to github to pull repository files without using <script>
+* Make calls to github to pull repository files without using `<script>`
 * Documentation

From 005a85825eb8d62fdc6aa0000e7a8d3e1318c2fe Mon Sep 17 00:00:00 2001
From: Levi DeHaan <levi@dehaandev.com>
Date: Wed, 9 Mar 2011 14:27:02 -0800
Subject: [PATCH 06/10] does the readme need to look perfect? yes. it does.

---
 README.md | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/README.md b/README.md
index 7b84fd3..b1138ee 100644
--- a/README.md
+++ b/README.md
@@ -10,15 +10,16 @@ so really the gists wont work unless you know for sure you wont get any with JS
 ###Example Code
 
 >##Authentication
->    //authenticating as yourself, I'd recommend using something like Amplify to store the user credentials in localstorage
+>    authenticating as yourself, I'd recommend using something like Amplify to store the user credentials in localstorage
+>
 >    `amplify.store("username", "YourUserName");
 >    amplify.store("token", "yourincrediblylongandtotallyawesometoken");
 >    gh.authenticate(amplify.store("username"), amplify.store("password"));
 >    var user = gh.user(amplify.store("username"));`
     
 >##User Information
->  //this pulls data showing your user information from github
->   //puts your gravatar image inside a div
+>  this pulls data showing your user information from github puts your gravatar image inside a div
+>
 >   `user.show(function (data) {
 >        var data = data.user
 >        console.log(data);
@@ -26,7 +27,8 @@ so really the gists wont work unless you know for sure you wont get any with JS
 >    });`
 
 >##Repository Information
->    //show a users repositories, iterates through them and then appends them to a div
+>    show a users repositories, iterates through them and then appends them to a div
+>
 >    `new repo.constructor.forUser(amplify.store("username"), RepoListCallback, data);`
 >    
 >    `function RepoListCallback(data){
@@ -39,7 +41,8 @@ so really the gists wont work unless you know for sure you wont get any with JS
    
 
 >##Sending data
->    //grabbing data from a form and sending it to update a user
+>    grabbing data from a form and sending it to update a user
+>
 >    `var params = {
 >                "company" : $("#companyname").val(),
 >                "name"      : $("#name").val

From d26a2398c193227ce2f304735b50e2774fbbab0b Mon Sep 17 00:00:00 2001
From: Levi DeHaan <levi@dehaandev.com>
Date: Wed, 9 Mar 2011 14:29:31 -0800
Subject: [PATCH 07/10] still not there

---
 README.md | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/README.md b/README.md
index b1138ee..4cadbe9 100644
--- a/README.md
+++ b/README.md
@@ -12,10 +12,10 @@ so really the gists wont work unless you know for sure you wont get any with JS
 >##Authentication
 >    authenticating as yourself, I'd recommend using something like Amplify to store the user credentials in localstorage
 >
->    `amplify.store("username", "YourUserName");
->    amplify.store("token", "yourincrediblylongandtotallyawesometoken");
->    gh.authenticate(amplify.store("username"), amplify.store("password"));
->    var user = gh.user(amplify.store("username"));`
+>        amplify.store("username", "YourUserName");
+>        amplify.store("token", "yourincrediblylongandtotallyawesometoken");
+>        gh.authenticate(amplify.store("username"), amplify.store("password"));
+>        var user = gh.user(amplify.store("username"));`
     
 >##User Information
 >  this pulls data showing your user information from github puts your gravatar image inside a div

From cae6c5e79aa0a1b8902c2306d58fdf05538e3b4a Mon Sep 17 00:00:00 2001
From: Levi DeHaan <levi@dehaandev.com>
Date: Wed, 9 Mar 2011 14:31:03 -0800
Subject: [PATCH 08/10] arr

---
 README.md | 34 ++++++++++++++++++----------------
 1 file changed, 18 insertions(+), 16 deletions(-)

diff --git a/README.md b/README.md
index 4cadbe9..f4cf321 100644
--- a/README.md
+++ b/README.md
@@ -20,35 +20,37 @@ so really the gists wont work unless you know for sure you wont get any with JS
 >##User Information
 >  this pulls data showing your user information from github puts your gravatar image inside a div
 >
->   `user.show(function (data) {
->        var data = data.user
->        console.log(data);
->        $('#somediv").html('<img src="http://gravatar.com/avatar/' + data.gravatar+id + '.jpg?s=80">');
->    });`
+>       user.show(function (data) {
+>            var data = data.user
+>            console.log(data);
+>            $('#somediv").html('<img src="http://gravatar.com/avatar/' + data.gravatar+id + '.jpg?s=80">');
+>        });
 
 >##Repository Information
 >    show a users repositories, iterates through them and then appends them to a div
 >
->    `new repo.constructor.forUser(amplify.store("username"), RepoListCallback, data);`
+>        new repo.constructor.forUser(amplify.store("username"), RepoListCallback, data);`
 >    
->    `function RepoListCallback(data){
->        console.log(data);
->        if(data.repositories.length > 0) {
->           for(var r=0;r<data.repositories.length; r++){
->               $('<h1>' + data.repositories[r].name + '</h1>').appendTo('#somediv');
->         }
->     }`
+>        function RepoListCallback(data){
+>            console.log(data);
+>             if(data.repositories.length > 0) {
+>                   for(var r=0;r<data.repositories.length; r++){
+>                       $('<h1>' + data.repositories[r].name + '</h1>').appendTo('#somediv');
+>                   }
+>              }
    
 
 >##Sending data
 >    grabbing data from a form and sending it to update a user
 >
->    `var params = {
+>        var params = {
 >                "company" : $("#companyname").val(),
 >                "name"      : $("#name").val
->     }`
+>         }
+>
+>         gh.user(user).update(params);
 >
->     `gh.user(user).update(params);`   //There is no callback available for sending data
+>   There is no callback available for sending data
 
    
 ###COMPLETE

From 4474d8aa424670fedfbfc7a89cc52d5e22ef1216 Mon Sep 17 00:00:00 2001
From: Levi DeHaan <levi@dehaandev.com>
Date: Mon, 4 Apr 2011 01:20:41 -0700
Subject: [PATCH 09/10] Some more modifications

---
 github.js | 142 ++++++++++++++++--------------------------------------
 1 file changed, 41 insertions(+), 101 deletions(-)

diff --git a/github.js b/github.js
index bda05d3..a80ee5a 100644
--- a/github.js
+++ b/github.js
@@ -2,6 +2,12 @@
 //
 // Tries to map one-to-one with the GitHub API V2, but in a Javascripty manner.
 
+if (typeof window.console === "undefined" ) {
+    window.console = {
+        log: $.noop
+    };
+};
+
 (function (globals) {
 
     // Before we implement the API methods, we will define all of our private
@@ -19,79 +25,38 @@
     // Send a JSONP request to the Github API that calls `callback` with
     // the `context` argument as `this`.
     //
-    // The `url` parameter is concatenated with the apiRoot, for the reasons
-    // mentioned above. The way that we are supporting non-global, anonymous
-    // functions is by sticking them in the globally exposed
-    // `gh.__jsonp_callbacks` object with a "unique" `id` that is the current
-    // time in milliseconds. Once the callback is called, it is deleted from the
-    // object to prevent memory leaks.
     jsonp = function (url, callback, context) {
-        var id = +new Date,
-        script = document.createElement("script");
-
-        while (gh.__jsonp_callbacks[id] !== undefined)
-            id += Math.random(); // Avoid slight possibility of id clashes.
-
-        gh.__jsonp_callbacks[id] = function () {
-            delete gh.__jsonp_callbacks[id];
-            callback.apply(context, arguments);
-        };
-
-        var prefix = "?";
-        if (url.indexOf("?") >= 0)
-            prefix = "&";
-
-        url += prefix + "callback=" + encodeURIComponent("gh.__jsonp_callbacks[" + id + "]");
+        var data = {};
+        
         if (authUsername && authToken) {
-            url += "&login=" + authUsername + "&token=" + authToken;
+            data.login = authUsername;
+            data.token = authToken;
         }
-        script.setAttribute("src", apiRoot + url);
-
-        document.getElementsByTagName('head')[0].appendChild(script);
+        
+        $.getJSON( apiRoot + url, data, function () {
+            callback.apply(context, arguments);
+        });
     },
     
     jsonp2 = function (url, callback, context) {
-        apiRoot2 = 'https://gist.github.com/'
-        var id = +new Date,
-        script = document.createElement("script");
-
-        while (gh.__jsonp_callbacks[id] !== undefined)
-            id += Math.random(); // Avoid slight possibility of id clashes.
-
-        gh.__jsonp_callbacks[id] = function () {
-            delete gh.__jsonp_callbacks[id];
+        var data = {};
+        
+        if (authUsername && authToken) {
+            data.login = authUsername;
+            data.token = authToken;
+        }
+        
+        $.getJSON( apiRoot + url, data, function () {
             callback.apply(context, arguments);
-        };
-
-        var prefix = "?";
-        if (url.indexOf("?") >= 0)
-            prefix = "&";
-
-        url += prefix + "callback=" + encodeURIComponent("gh.__jsonp_callbacks[" + id + "]");
-       
-        script.setAttribute("src", apiRoot2 + url);
-        console.log('jsonp2');
-        console.log(script);
-        document.getElementsByTagName('head')[0].appendChild(script);
+        }, "text");
     },
 
     // Send an HTTP POST. Unfortunately, it isn't possible to support a callback
     // with the resulting data. (Please prove me wrong if you can!)
     //
-    // This is implemented with a hack to get around the cross-domain
-    // restrictions on ajax calls. Basically, a form is created that will POST
-    // to the GitHub API URL, stuck inside an iframe so that it won't redirect
-    // this page, and then submitted.
-    // THIS FUNCTION NOW DEPENDS ON jQuery
     post = function (url, vals) {
-        var key, field;
+        var key, values;
         
-        var iframe = $('<iframe name="githubpull" style="display: none;">').appendTo($("body"));
-        var doc = null;
-        var values = "";
-        doc = window.frames["githubpull"].document;
-        
-        console.log(vals);
         if(_.isArray(vals) || !_.isString(vals))
         {
             vals = vals || {};
@@ -103,49 +68,30 @@
         }else if(_.isString(vals)){
             values = "&" + vals
         }
-        var form = document.createElement("form");
-        form.method = "post";
-        form.action = apiRoot + url + "?login=" + authUsername + "&token=" + authToken + values;
         
-        console.log(form);
-        doc.body.appendChild(form);
-        form.submit();
+        $.post(apiRoot + url + "?login=" + authUsername + "&token=" + authToken + values);
     },
     
     post2 = function (url, vals) {
-        var key, field;
-        
-        var iframe = $('<iframe name="githubpull" style="display: none;">').appendTo($("body"));
-        var doc = null;
-        var values = "";
-        doc = window.frames["githubpull"].document;
+        var key, values;
         
-        console.log(vals);
-        if(_.isArray(vals) || !_.isString(vals))
-        {
+        if (_.isArray(vals) || !_.isString(vals)) {
             vals = vals || {};
             for (key in vals) {
                 if (vals.hasOwnProperty(key)) {
-                    values += "&" + key + "=" + vals[key]
+                    values += "&" + key + "=" + vals[key];
                 }
             }
-        }else if(_.isString(vals)){
-            values = "&" + vals
+        } else if (_.isString(vals)) {
+            values = "&" + vals;
         }
-        var form = document.createElement("form");
-        form.method = "post";
-        form.action = apiRoot + url + "?login=" + authUsername + "&token=" + authToken + values;
         
-        console.log(form);
-        doc.body.appendChild(form);
-        form.submit();
+        $.post(apiRoot + url + "?login=" + authUsername + "&token=" + authToken + values);
     },
 
     // This helper function will throw a TypeError if the library user is not
     // properly authenticated. Otherwise, it silently returns.
     authRequired = function (username) {
-        console.log("authUsername: " + authUsername);
-        console.log("authToken: " + authToken)
         if (!authUsername || !authToken || authUsername !== username) {
             throw new TypeError("gh: Must be authenticated to do that.");
         }
@@ -218,14 +164,7 @@
     //
     //     TODO: example
     gh.user.prototype.update = function (params) {
-        //authRequired(this.username);
-        var key, postData = {};
-        for (key in params) {
-            if (params.hasOwnProperty(key)) {
-                postData[key] = encodeURIComponent(params[key]);
-            }
-        }
-        post("user/show/" + this.username, postData);
+        post("user/show/" + this.username, params);
         return this;
     };
     
@@ -234,6 +173,7 @@
         return this;
     }
     
+    // TODO, this looks broken
     // { "title" : "cheese", "key" : "alkj309jv309mz89eur0923m09cvj23=="
     gh.user.prototype.addKey = function (params){
         var key, postData = {};
@@ -246,6 +186,7 @@
         return this;
     }
     
+    // Todo, also looks broken
     // { "id" : "987" }
     gh.user.prototype.removeKey = function (params){
         var key, postData = {};
@@ -488,7 +429,7 @@
     // Search for repositories. `opts` may include `start_page` or `language`,
     // which must be capitalized.
     gh.repo.search = function (query, opts, callback, context) {
-         var url = "repos/search/" + query.replace(" ", "+");
+        var url = "repos/search/" + query.replace(" ", "+");
         if (typeof opts === "function") {
             opts = {};
             callback = arguments[1];
@@ -618,7 +559,6 @@
 
     // Comment on this issue as the user that is authenticated.
     gh.issue.prototype.comment = function (comment) {
-        authRequired(authUsername);
         post2("issues/comment/" + this.user + "/" + this.repo + "/" + this.number, {
             comment: comment
         });
@@ -673,14 +613,14 @@
     };
 
     gh.gist.prototype.show = function (callback, context) {
-            jsonp2("api/v1/json/" + this.id, callback, context);
-            return this;
-        };
+        jsonp2("api/v1/json/" + this.id, callback, context);
+        return this;
+    };
 
     gh.gist.prototype.file = function (filename, callback, context) {
-            jsonp2("raw/" + this.id + "/" + filename, callback, context);
-            return this;
-        };
+        jsonp2("raw/" + this.id + "/" + filename, callback, context);
+        return this;
+    };
 
     // ### Objects
 

From b19d4f40cbdae52ee52cc3b9817462336f24eab6 Mon Sep 17 00:00:00 2001
From: Levi DeHaan <levi@dehaandev.com>
Date: Mon, 4 Apr 2011 01:21:59 -0700
Subject: [PATCH 10/10] Edited README.md via GitHub

---
 README.md | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/README.md b/README.md
index f4cf321..30cbbbd 100644
--- a/README.md
+++ b/README.md
@@ -1,5 +1,7 @@
 Github API Js
 ===============
+This file has been modified and only works in environments that allow crossdomain scripting such as the BlackBerry Playbook with the WebWorks SDK.
+You must also use jQuery to use this library.
 
 Exposes `gh` to the global environment. Tries to follow both the form of GithubHTTP API and JS style.