{"action":"created","comment":{"url":"https://api.github.com/repos/mobcore/MafiaGame/pulls/comments/32308528","id":32308528,"diff_hunk":"@@ -1,43 +1,26 @@\n package Character;\n \n-import Enumerators.Roles;\n-import GameEngine.CannotGetPlayerException;\n-import GameEngine.Character;\n-import GameEngine.GameEngine;\n-import GameEngine.GameMessage;\n-import GameEngine.Player;\n+import java.util.List;\n \n-//****************************************************************\n-//DONE DONE DONE DONE DONE DONE DONE DONE DONE DONE DONE DONE DONE\n-//this is a tag for all the characters/classes that are done\n-//*****************************************************************\n+import Character.CharacterFactory.Roles;\n+import GameEngine.GameRegistration.Player;\n+import Resources.GameLog;\n \n public class Doctor extends Character {\n \n-\tpublic Doctor(Player player) {\n+\tprotected Doctor(Player player) {\n \t\tsuper(Roles.Doctor, player);\n \t}\n \n \t@Override\n-\tpublic String doAction() throws CannotGetPlayerException {\n-\t\t\n-\t\tif (getTarget().size() != 1) {\n-\t\t\treturn GameMessage.NO_ACTION;\n+\tpublic boolean performAction(List<Player> alive_player, Character target) {\n+\t\tif(this.character_status.isBlocked()) {\n+\t\t\tresult = GameLog.Character.BLOCKED(player);\n+\t\t\treturn true;\n \t\t}\n-\t\t\n-\t\tPlayer target = getTarget().get(0);\n-\t\t\n-\t\tif(!GameEngine.getCharacter(target).isAlive()) {\n-\t\t\treturn GameMessage.TARGET_DEAD;\n-\t\t}\n-\t\tif (this.isRoleBlocked()) {\n-\t\t\treturn GameMessage.DOCTOR_FEEDBACK();\n-\t\t}\n-\t\t\n-\t\tGameEngine.getCharacter(target).addVisitor(getPlayer());\n-\n-\t\tGameEngine.getCharacter(target).healPlayer();\n-\t\t\n-\t\treturn GameMessage.DOCTOR_FEEDBACK();\n+\t\ttarget.character_status.getVisitors().add(player);\n+\t\ttarget.character_status.heal();\n+\t\tresult = GameLog.Character.DOCTOR_SUCCESS(player, target.player);","path":"src/Character/Doctor.java","position":53,"original_position":53,"commit_id":"a4901ed5e1941268e66ab8674c3cc7ab58909e44","original_commit_id":"a4901ed5e1941268e66ab8674c3cc7ab58909e44","user":{"login":"andyytung","id":9043974,"avatar_url":"https://avatars.githubusercontent.com/u/9043974?v=3","gravatar_id":"","url":"https://api.github.com/users/andyytung","html_url":"https://github.com/andyytung","followers_url":"https://api.github.com/users/andyytung/followers","following_url":"https://api.github.com/users/andyytung/following{/other_user}","gists_url":"https://api.github.com/users/andyytung/gists{/gist_id}","starred_url":"https://api.github.com/users/andyytung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andyytung/subscriptions","organizations_url":"https://api.github.com/users/andyytung/orgs","repos_url":"https://api.github.com/users/andyytung/repos","events_url":"https://api.github.com/users/andyytung/events{/privacy}","received_events_url":"https://api.github.com/users/andyytung/received_events","type":"User","site_admin":false},"body":"it's probably better to just return result in this case","created_at":"2015-06-12T11:35:20Z","updated_at":"2015-06-12T11:35:20Z","html_url":"https://github.com/mobcore/MafiaGame/pull/14#discussion_r32308528","pull_request_url":"https://api.github.com/repos/mobcore/MafiaGame/pulls/14","_links":{"self":{"href":"https://api.github.com/repos/mobcore/MafiaGame/pulls/comments/32308528"},"html":{"href":"https://github.com/mobcore/MafiaGame/pull/14#discussion_r32308528"},"pull_request":{"href":"https://api.github.com/repos/mobcore/MafiaGame/pulls/14"}}},"pull_request":{"url":"https://api.github.com/repos/mobcore/MafiaGame/pulls/14","id":37315561,"html_url":"https://github.com/mobcore/MafiaGame/pull/14","diff_url":"https://github.com/mobcore/MafiaGame/pull/14.diff","patch_url":"https://github.com/mobcore/MafiaGame/pull/14.patch","issue_url":"https://api.github.com/repos/mobcore/MafiaGame/issues/14","number":14,"state":"closed","locked":false,"title":"Refractors","user":{"login":"andyytung","id":9043974,"avatar_url":"https://avatars.githubusercontent.com/u/9043974?v=3","gravatar_id":"","url":"https://api.github.com/users/andyytung","html_url":"https://github.com/andyytung","followers_url":"https://api.github.com/users/andyytung/followers","following_url":"https://api.github.com/users/andyytung/following{/other_user}","gists_url":"https://api.github.com/users/andyytung/gists{/gist_id}","starred_url":"https://api.github.com/users/andyytung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andyytung/subscriptions","organizations_url":"https://api.github.com/users/andyytung/orgs","repos_url":"https://api.github.com/users/andyytung/repos","events_url":"https://api.github.com/users/andyytung/events{/privacy}","received_events_url":"https://api.github.com/users/andyytung/received_events","type":"User","site_admin":false},"body":"Vote System works.\r\nMafia Main renamed to Shell Commands because Connie says the name makes more sense.","created_at":"2015-06-10T00:53:15Z","updated_at":"2015-06-12T11:35:20Z","closed_at":"2015-06-12T06:00:32Z","merged_at":"2015-06-12T06:00:32Z","merge_commit_sha":"f1cc8112bca55ed2cbcaf349a269156f858a1280","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/mobcore/MafiaGame/pulls/14/commits","review_comments_url":"https://api.github.com/repos/mobcore/MafiaGame/pulls/14/comments","review_comment_url":"https://api.github.com/repos/mobcore/MafiaGame/pulls/comments{/number}","comments_url":"https://api.github.com/repos/mobcore/MafiaGame/issues/14/comments","statuses_url":"https://api.github.com/repos/mobcore/MafiaGame/statuses/a4901ed5e1941268e66ab8674c3cc7ab58909e44","head":{"label":"mobcore:refractors","ref":"refractors","sha":"a4901ed5e1941268e66ab8674c3cc7ab58909e44","user":{"login":"mobcore","id":12805872,"avatar_url":"https://avatars.githubusercontent.com/u/12805872?v=3","gravatar_id":"","url":"https://api.github.com/users/mobcore","html_url":"https://github.com/mobcore","followers_url":"https://api.github.com/users/mobcore/followers","following_url":"https://api.github.com/users/mobcore/following{/other_user}","gists_url":"https://api.github.com/users/mobcore/gists{/gist_id}","starred_url":"https://api.github.com/users/mobcore/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mobcore/subscriptions","organizations_url":"https://api.github.com/users/mobcore/orgs","repos_url":"https://api.github.com/users/mobcore/repos","events_url":"https://api.github.com/users/mobcore/events{/privacy}","received_events_url":"https://api.github.com/users/mobcore/received_events","type":"Organization","site_admin":false},"repo":{"id":29073188,"name":"MafiaGame","full_name":"mobcore/MafiaGame","owner":{"login":"mobcore","id":12805872,"avatar_url":"https://avatars.githubusercontent.com/u/12805872?v=3","gravatar_id":"","url":"https://api.github.com/users/mobcore","html_url":"https://github.com/mobcore","followers_url":"https://api.github.com/users/mobcore/followers","following_url":"https://api.github.com/users/mobcore/following{/other_user}","gists_url":"https://api.github.com/users/mobcore/gists{/gist_id}","starred_url":"https://api.github.com/users/mobcore/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mobcore/subscriptions","organizations_url":"https://api.github.com/users/mobcore/orgs","repos_url":"https://api.github.com/users/mobcore/repos","events_url":"https://api.github.com/users/mobcore/events{/privacy}","received_events_url":"https://api.github.com/users/mobcore/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/mobcore/MafiaGame","description":"Our mafia game repository","fork":false,"url":"https://api.github.com/repos/mobcore/MafiaGame","forks_url":"https://api.github.com/repos/mobcore/MafiaGame/forks","keys_url":"https://api.github.com/repos/mobcore/MafiaGame/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mobcore/MafiaGame/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mobcore/MafiaGame/teams","hooks_url":"https://api.github.com/repos/mobcore/MafiaGame/hooks","issue_events_url":"https://api.github.com/repos/mobcore/MafiaGame/issues/events{/number}","events_url":"https://api.github.com/repos/mobcore/MafiaGame/events","assignees_url":"https://api.github.com/repos/mobcore/MafiaGame/assignees{/user}","branches_url":"https://api.github.com/repos/mobcore/MafiaGame/branches{/branch}","tags_url":"https://api.github.com/repos/mobcore/MafiaGame/tags","blobs_url":"https://api.github.com/repos/mobcore/MafiaGame/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mobcore/MafiaGame/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mobcore/MafiaGame/git/refs{/sha}","trees_url":"https://api.github.com/repos/mobcore/MafiaGame/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mobcore/MafiaGame/statuses/{sha}","languages_url":"https://api.github.com/repos/mobcore/MafiaGame/languages","stargazers_url":"https://api.github.com/repos/mobcore/MafiaGame/stargazers","contributors_url":"https://api.github.com/repos/mobcore/MafiaGame/contributors","subscribers_url":"https://api.github.com/repos/mobcore/MafiaGame/subscribers","subscription_url":"https://api.github.com/repos/mobcore/MafiaGame/subscription","commits_url":"https://api.github.com/repos/mobcore/MafiaGame/commits{/sha}","git_commits_url":"https://api.github.com/repos/mobcore/MafiaGame/git/commits{/sha}","comments_url":"https://api.github.com/repos/mobcore/MafiaGame/comments{/number}","issue_comment_url":"https://api.github.com/repos/mobcore/MafiaGame/issues/comments{/number}","contents_url":"https://api.github.com/repos/mobcore/MafiaGame/contents/{+path}","compare_url":"https://api.github.com/repos/mobcore/MafiaGame/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mobcore/MafiaGame/merges","archive_url":"https://api.github.com/repos/mobcore/MafiaGame/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mobcore/MafiaGame/downloads","issues_url":"https://api.github.com/repos/mobcore/MafiaGame/issues{/number}","pulls_url":"https://api.github.com/repos/mobcore/MafiaGame/pulls{/number}","milestones_url":"https://api.github.com/repos/mobcore/MafiaGame/milestones{/number}","notifications_url":"https://api.github.com/repos/mobcore/MafiaGame/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mobcore/MafiaGame/labels{/name}","releases_url":"https://api.github.com/repos/mobcore/MafiaGame/releases{/id}","created_at":"2015-01-10T22:23:20Z","updated_at":"2015-06-10T13:59:08Z","pushed_at":"2015-06-12T07:12:23Z","git_url":"git://github.com/mobcore/MafiaGame.git","ssh_url":"git@github.com:mobcore/MafiaGame.git","clone_url":"https://github.com/mobcore/MafiaGame.git","svn_url":"https://github.com/mobcore/MafiaGame","homepage":null,"size":3976,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":1,"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"base":{"label":"mobcore:master","ref":"master","sha":"303c979220fbe45943d73fe84a03bf6ee86c8b5b","user":{"login":"mobcore","id":12805872,"avatar_url":"https://avatars.githubusercontent.com/u/12805872?v=3","gravatar_id":"","url":"https://api.github.com/users/mobcore","html_url":"https://github.com/mobcore","followers_url":"https://api.github.com/users/mobcore/followers","following_url":"https://api.github.com/users/mobcore/following{/other_user}","gists_url":"https://api.github.com/users/mobcore/gists{/gist_id}","starred_url":"https://api.github.com/users/mobcore/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mobcore/subscriptions","organizations_url":"https://api.github.com/users/mobcore/orgs","repos_url":"https://api.github.com/users/mobcore/repos","events_url":"https://api.github.com/users/mobcore/events{/privacy}","received_events_url":"https://api.github.com/users/mobcore/received_events","type":"Organization","site_admin":false},"repo":{"id":29073188,"name":"MafiaGame","full_name":"mobcore/MafiaGame","owner":{"login":"mobcore","id":12805872,"avatar_url":"https://avatars.githubusercontent.com/u/12805872?v=3","gravatar_id":"","url":"https://api.github.com/users/mobcore","html_url":"https://github.com/mobcore","followers_url":"https://api.github.com/users/mobcore/followers","following_url":"https://api.github.com/users/mobcore/following{/other_user}","gists_url":"https://api.github.com/users/mobcore/gists{/gist_id}","starred_url":"https://api.github.com/users/mobcore/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mobcore/subscriptions","organizations_url":"https://api.github.com/users/mobcore/orgs","repos_url":"https://api.github.com/users/mobcore/repos","events_url":"https://api.github.com/users/mobcore/events{/privacy}","received_events_url":"https://api.github.com/users/mobcore/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/mobcore/MafiaGame","description":"Our mafia game repository","fork":false,"url":"https://api.github.com/repos/mobcore/MafiaGame","forks_url":"https://api.github.com/repos/mobcore/MafiaGame/forks","keys_url":"https://api.github.com/repos/mobcore/MafiaGame/keys{/key_id}","collaborators_url":"https://api.github.com/repos/mobcore/MafiaGame/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/mobcore/MafiaGame/teams","hooks_url":"https://api.github.com/repos/mobcore/MafiaGame/hooks","issue_events_url":"https://api.github.com/repos/mobcore/MafiaGame/issues/events{/number}","events_url":"https://api.github.com/repos/mobcore/MafiaGame/events","assignees_url":"https://api.github.com/repos/mobcore/MafiaGame/assignees{/user}","branches_url":"https://api.github.com/repos/mobcore/MafiaGame/branches{/branch}","tags_url":"https://api.github.com/repos/mobcore/MafiaGame/tags","blobs_url":"https://api.github.com/repos/mobcore/MafiaGame/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/mobcore/MafiaGame/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/mobcore/MafiaGame/git/refs{/sha}","trees_url":"https://api.github.com/repos/mobcore/MafiaGame/git/trees{/sha}","statuses_url":"https://api.github.com/repos/mobcore/MafiaGame/statuses/{sha}","languages_url":"https://api.github.com/repos/mobcore/MafiaGame/languages","stargazers_url":"https://api.github.com/repos/mobcore/MafiaGame/stargazers","contributors_url":"https://api.github.com/repos/mobcore/MafiaGame/contributors","subscribers_url":"https://api.github.com/repos/mobcore/MafiaGame/subscribers","subscription_url":"https://api.github.com/repos/mobcore/MafiaGame/subscription","commits_url":"https://api.github.com/repos/mobcore/MafiaGame/commits{/sha}","git_commits_url":"https://api.github.com/repos/mobcore/MafiaGame/git/commits{/sha}","comments_url":"https://api.github.com/repos/mobcore/MafiaGame/comments{/number}","issue_comment_url":"https://api.github.com/repos/mobcore/MafiaGame/issues/comments{/number}","contents_url":"https://api.github.com/repos/mobcore/MafiaGame/contents/{+path}","compare_url":"https://api.github.com/repos/mobcore/MafiaGame/compare/{base}...{head}","merges_url":"https://api.github.com/repos/mobcore/MafiaGame/merges","archive_url":"https://api.github.com/repos/mobcore/MafiaGame/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/mobcore/MafiaGame/downloads","issues_url":"https://api.github.com/repos/mobcore/MafiaGame/issues{/number}","pulls_url":"https://api.github.com/repos/mobcore/MafiaGame/pulls{/number}","milestones_url":"https://api.github.com/repos/mobcore/MafiaGame/milestones{/number}","notifications_url":"https://api.github.com/repos/mobcore/MafiaGame/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/mobcore/MafiaGame/labels{/name}","releases_url":"https://api.github.com/repos/mobcore/MafiaGame/releases{/id}","created_at":"2015-01-10T22:23:20Z","updated_at":"2015-06-10T13:59:08Z","pushed_at":"2015-06-12T07:12:23Z","git_url":"git://github.com/mobcore/MafiaGame.git","ssh_url":"git@github.com:mobcore/MafiaGame.git","clone_url":"https://github.com/mobcore/MafiaGame.git","svn_url":"https://github.com/mobcore/MafiaGame","homepage":null,"size":3976,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":1,"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/mobcore/MafiaGame/pulls/14"},"html":{"href":"https://github.com/mobcore/MafiaGame/pull/14"},"issue":{"href":"https://api.github.com/repos/mobcore/MafiaGame/issues/14"},"comments":{"href":"https://api.github.com/repos/mobcore/MafiaGame/issues/14/comments"},"review_comments":{"href":"https://api.github.com/repos/mobcore/MafiaGame/pulls/14/comments"},"review_comment":{"href":"https://api.github.com/repos/mobcore/MafiaGame/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/mobcore/MafiaGame/pulls/14/commits"},"statuses":{"href":"https://api.github.com/repos/mobcore/MafiaGame/statuses/a4901ed5e1941268e66ab8674c3cc7ab58909e44"}}}}