GitHub PullRequestReviewCommentEvent at 2014-05-12 18:37:58
Repository: bb_extra
Actor: chinhodado
Payload: {"comment":{"url":"https://api.github.com/repos/chinhodado/bb_extra/pulls/comments/12546127","id":12546127,"diff_hunk":"@@ -243,16 +243,22 @@ function getTierInfo () {\n \n     function getTier(category) {\n         // parse the response text into DOM\n+        var tierResult = \"N/A\";\n         var doc = document.implementation.createHTMLDocument(\"Tier\");\n         doc.documentElement.innerHTML = sessionStorage[category];\n \n         var tables = doc.getElementsByClassName(\"wikitable\");\n+        \n+        if (!tables) {\n+            return tierResult;\n+        }\n \n-        var tierResult = \"N/A\";\n         var tiers = ['X', 'S+', 'S', 'A+', 'A', 'B', 'C', 'D', 'E'];\n         var famName = (document.getElementById(\"WikiaPageHeader\").getElementsByTagName(\"h1\"))[0].innerHTML;\n \n         for (var i = 0; i < 9; i++){ // 9 tables\n+            if (!tables[i])","path":"script.js","position":19,"original_position":19,"commit_id":"170bb001864772b9a8bd488bd41851a9b186c443","original_commit_id":"170bb001864772b9a8bd488bd41851a9b186c443","user":{"login":"chinhodado","id":1388219,"avatar_url":"https://avatars.githubusercontent.com/u/1388219?","gravatar_id":"3decb53828ce2c77a7fb28f7fd987234","url":"https://api.github.com/users/chinhodado","html_url":"https://github.com/chinhodado","followers_url":"https://api.github.com/users/chinhodado/followers","following_url":"https://api.github.com/users/chinhodado/following{/other_user}","gists_url":"https://api.github.com/users/chinhodado/gists{/gist_id}","starred_url":"https://api.github.com/users/chinhodado/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chinhodado/subscriptions","organizations_url":"https://api.github.com/users/chinhodado/orgs","repos_url":"https://api.github.com/users/chinhodado/repos","events_url":"https://api.github.com/users/chinhodado/events{/privacy}","received_events_url":"https://api.github.com/users/chinhodado/received_events","type":"User","site_admin":false},"body":"right, it's because of the C-E tiers merge","created_at":"2014-05-12T18:37:58Z","updated_at":"2014-05-12T18:37:58Z","html_url":"https://github.com/chinhodado/bb_extra/pull/4#discussion_r12546127","pull_request_url":"https://api.github.com/repos/chinhodado/bb_extra/pulls/4","_links":{"self":{"href":"https://api.github.com/repos/chinhodado/bb_extra/pulls/comments/12546127"},"html":{"href":"https://github.com/chinhodado/bb_extra/pull/4#discussion_r12546127"},"pull_request":{"href":"https://api.github.com/repos/chinhodado/bb_extra/pulls/4"}}}}