{"action":"created","comment":{"url":"https://api.github.com/repos/projecttacoma/bonnie/pulls/comments/93489274","pull_request_review_id":14018440,"id":93489274,"diff_hunk":"@@ -78,10 +78,12 @@ def get_population_set_summaries(current_measure, archived_measure)\n         patient.update_expected_value_structure!(current_measure)\n       end\n \n-      # recalculate the patient information for the newly uploaded measure\n+      # recalculate the patient information for the newly uploaded measure and\n+      # note that the patient has measure history\n       calculator = BonnieBackendCalculator.new\n       current_measure.populations.each_with_index do |population, population_set_index|\n         patients.each do |patient|\n+          patient.has_measure_history = true # update_calc_results does the patient save to persist this change\n           patient.update_calc_results!(current_measure, population_set_index, calculator)","path":"lib/ext/measure_upload_summary.rb","position":11,"original_position":11,"commit_id":"cb3a72d1bde72b6632e2d794b475b652f6fe0e8e","original_commit_id":"cb3a72d1bde72b6632e2d794b475b652f6fe0e8e","user":{"login":"mulcahyk","id":17731116,"avatar_url":"https://avatars.githubusercontent.com/u/17731116?v=3","gravatar_id":"","url":"https://api.github.com/users/mulcahyk","html_url":"https://github.com/mulcahyk","followers_url":"https://api.github.com/users/mulcahyk/followers","following_url":"https://api.github.com/users/mulcahyk/following{/other_user}","gists_url":"https://api.github.com/users/mulcahyk/gists{/gist_id}","starred_url":"https://api.github.com/users/mulcahyk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mulcahyk/subscriptions","organizations_url":"https://api.github.com/users/mulcahyk/orgs","repos_url":"https://api.github.com/users/mulcahyk/repos","events_url":"https://api.github.com/users/mulcahyk/events{/privacy}","received_events_url":"https://api.github.com/users/mulcahyk/received_events","type":"User","site_admin":false},"body":"There is not error handling on this.  There is also no error handling in the method itself.  ","created_at":"2016-12-21T18:08:20Z","updated_at":"2016-12-21T18:08:20Z","html_url":"https://github.com/projecttacoma/bonnie/pull/626#discussion_r93489274","pull_request_url":"https://api.github.com/repos/projecttacoma/bonnie/pulls/626","_links":{"self":{"href":"https://api.github.com/repos/projecttacoma/bonnie/pulls/comments/93489274"},"html":{"href":"https://github.com/projecttacoma/bonnie/pull/626#discussion_r93489274"},"pull_request":{"href":"https://api.github.com/repos/projecttacoma/bonnie/pulls/626"}}},"pull_request":{"url":"https://api.github.com/repos/projecttacoma/bonnie/pulls/626","id":98200047,"html_url":"https://github.com/projecttacoma/bonnie/pull/626","diff_url":"https://github.com/projecttacoma/bonnie/pull/626.diff","patch_url":"https://github.com/projecttacoma/bonnie/pull/626.patch","issue_url":"https://api.github.com/repos/projecttacoma/bonnie/issues/626","number":626,"state":"open","locked":false,"title":"made the measure history rake setup tasks use the methods that have b…","user":{"login":"edeyoung","id":11574199,"avatar_url":"https://avatars.githubusercontent.com/u/11574199?v=3","gravatar_id":"","url":"https://api.github.com/users/edeyoung","html_url":"https://github.com/edeyoung","followers_url":"https://api.github.com/users/edeyoung/followers","following_url":"https://api.github.com/users/edeyoung/following{/other_user}","gists_url":"https://api.github.com/users/edeyoung/gists{/gist_id}","starred_url":"https://api.github.com/users/edeyoung/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/edeyoung/subscriptions","organizations_url":"https://api.github.com/users/edeyoung/orgs","repos_url":"https://api.github.com/users/edeyoung/repos","events_url":"https://api.github.com/users/edeyoung/events{/privacy}","received_events_url":"https://api.github.com/users/edeyoung/received_events","type":"User","site_admin":false},"body":"…een added onto the patient for updating the expected values and adding calculation results.","created_at":"2016-12-15T17:57:31Z","updated_at":"2016-12-21T18:08:20Z","closed_at":null,"merged_at":null,"merge_commit_sha":"58d393761e195eb090bd0f1cb093bd637e4dc24c","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/projecttacoma/bonnie/pulls/626/commits","review_comments_url":"https://api.github.com/repos/projecttacoma/bonnie/pulls/626/comments","review_comment_url":"https://api.github.com/repos/projecttacoma/bonnie/pulls/comments{/number}","comments_url":"https://api.github.com/repos/projecttacoma/bonnie/issues/626/comments","statuses_url":"https://api.github.com/repos/projecttacoma/bonnie/statuses/cb3a72d1bde72b6632e2d794b475b652f6fe0e8e","head":{"label":"projecttacoma:measure_history_r_rake_uses_patient","ref":"measure_history_r_rake_uses_patient","sha":"cb3a72d1bde72b6632e2d794b475b652f6fe0e8e","user":{"login":"projecttacoma","id":5497023,"avatar_url":"https://avatars.githubusercontent.com/u/5497023?v=3","gravatar_id":"","url":"https://api.github.com/users/projecttacoma","html_url":"https://github.com/projecttacoma","followers_url":"https://api.github.com/users/projecttacoma/followers","following_url":"https://api.github.com/users/projecttacoma/following{/other_user}","gists_url":"https://api.github.com/users/projecttacoma/gists{/gist_id}","starred_url":"https://api.github.com/users/projecttacoma/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/projecttacoma/subscriptions","organizations_url":"https://api.github.com/users/projecttacoma/orgs","repos_url":"https://api.github.com/users/projecttacoma/repos","events_url":"https://api.github.com/users/projecttacoma/events{/privacy}","received_events_url":"https://api.github.com/users/projecttacoma/received_events","type":"Organization","site_admin":false},"repo":{"id":12958279,"name":"bonnie","full_name":"projecttacoma/bonnie","owner":{"login":"projecttacoma","id":5497023,"avatar_url":"https://avatars.githubusercontent.com/u/5497023?v=3","gravatar_id":"","url":"https://api.github.com/users/projecttacoma","html_url":"https://github.com/projecttacoma","followers_url":"https://api.github.com/users/projecttacoma/followers","following_url":"https://api.github.com/users/projecttacoma/following{/other_user}","gists_url":"https://api.github.com/users/projecttacoma/gists{/gist_id}","starred_url":"https://api.github.com/users/projecttacoma/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/projecttacoma/subscriptions","organizations_url":"https://api.github.com/users/projecttacoma/orgs","repos_url":"https://api.github.com/users/projecttacoma/repos","events_url":"https://api.github.com/users/projecttacoma/events{/privacy}","received_events_url":"https://api.github.com/users/projecttacoma/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/projecttacoma/bonnie","description":"Clinical Quality Measure Testing Tool","fork":false,"url":"https://api.github.com/repos/projecttacoma/bonnie","forks_url":"https://api.github.com/repos/projecttacoma/bonnie/forks","keys_url":"https://api.github.com/repos/projecttacoma/bonnie/keys{/key_id}","collaborators_url":"https://api.github.com/repos/projecttacoma/bonnie/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/projecttacoma/bonnie/teams","hooks_url":"https://api.github.com/repos/projecttacoma/bonnie/hooks","issue_events_url":"https://api.github.com/repos/projecttacoma/bonnie/issues/events{/number}","events_url":"https://api.github.com/repos/projecttacoma/bonnie/events","assignees_url":"https://api.github.com/repos/projecttacoma/bonnie/assignees{/user}","branches_url":"https://api.github.com/repos/projecttacoma/bonnie/branches{/branch}","tags_url":"https://api.github.com/repos/projecttacoma/bonnie/tags","blobs_url":"https://api.github.com/repos/projecttacoma/bonnie/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/projecttacoma/bonnie/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/projecttacoma/bonnie/git/refs{/sha}","trees_url":"https://api.github.com/repos/projecttacoma/bonnie/git/trees{/sha}","statuses_url":"https://api.github.com/repos/projecttacoma/bonnie/statuses/{sha}","languages_url":"https://api.github.com/repos/projecttacoma/bonnie/languages","stargazers_url":"https://api.github.com/repos/projecttacoma/bonnie/stargazers","contributors_url":"https://api.github.com/repos/projecttacoma/bonnie/contributors","subscribers_url":"https://api.github.com/repos/projecttacoma/bonnie/subscribers","subscription_url":"https://api.github.com/repos/projecttacoma/bonnie/subscription","commits_url":"https://api.github.com/repos/projecttacoma/bonnie/commits{/sha}","git_commits_url":"https://api.github.com/repos/projecttacoma/bonnie/git/commits{/sha}","comments_url":"https://api.github.com/repos/projecttacoma/bonnie/comments{/number}","issue_comment_url":"https://api.github.com/repos/projecttacoma/bonnie/issues/comments{/number}","contents_url":"https://api.github.com/repos/projecttacoma/bonnie/contents/{+path}","compare_url":"https://api.github.com/repos/projecttacoma/bonnie/compare/{base}...{head}","merges_url":"https://api.github.com/repos/projecttacoma/bonnie/merges","archive_url":"https://api.github.com/repos/projecttacoma/bonnie/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/projecttacoma/bonnie/downloads","issues_url":"https://api.github.com/repos/projecttacoma/bonnie/issues{/number}","pulls_url":"https://api.github.com/repos/projecttacoma/bonnie/pulls{/number}","milestones_url":"https://api.github.com/repos/projecttacoma/bonnie/milestones{/number}","notifications_url":"https://api.github.com/repos/projecttacoma/bonnie/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/projecttacoma/bonnie/labels{/name}","releases_url":"https://api.github.com/repos/projecttacoma/bonnie/releases{/id}","deployments_url":"https://api.github.com/repos/projecttacoma/bonnie/deployments","created_at":"2013-09-19T20:11:14Z","updated_at":"2016-12-17T16:22:39Z","pushed_at":"2016-12-21T17:51:40Z","git_url":"git://github.com/projecttacoma/bonnie.git","ssh_url":"git@github.com:projecttacoma/bonnie.git","clone_url":"https://github.com/projecttacoma/bonnie.git","svn_url":"https://github.com/projecttacoma/bonnie","homepage":null,"size":99340,"stargazers_count":17,"watchers_count":17,"language":"CoffeeScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":17,"mirror_url":null,"open_issues_count":14,"forks":17,"open_issues":14,"watchers":17,"default_branch":"master"}},"base":{"label":"projecttacoma:measure_history_rebase","ref":"measure_history_rebase","sha":"853ad8d1a9ad83a35102fc84cc01c28fdb610fee","user":{"login":"projecttacoma","id":5497023,"avatar_url":"https://avatars.githubusercontent.com/u/5497023?v=3","gravatar_id":"","url":"https://api.github.com/users/projecttacoma","html_url":"https://github.com/projecttacoma","followers_url":"https://api.github.com/users/projecttacoma/followers","following_url":"https://api.github.com/users/projecttacoma/following{/other_user}","gists_url":"https://api.github.com/users/projecttacoma/gists{/gist_id}","starred_url":"https://api.github.com/users/projecttacoma/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/projecttacoma/subscriptions","organizations_url":"https://api.github.com/users/projecttacoma/orgs","repos_url":"https://api.github.com/users/projecttacoma/repos","events_url":"https://api.github.com/users/projecttacoma/events{/privacy}","received_events_url":"https://api.github.com/users/projecttacoma/received_events","type":"Organization","site_admin":false},"repo":{"id":12958279,"name":"bonnie","full_name":"projecttacoma/bonnie","owner":{"login":"projecttacoma","id":5497023,"avatar_url":"https://avatars.githubusercontent.com/u/5497023?v=3","gravatar_id":"","url":"https://api.github.com/users/projecttacoma","html_url":"https://github.com/projecttacoma","followers_url":"https://api.github.com/users/projecttacoma/followers","following_url":"https://api.github.com/users/projecttacoma/following{/other_user}","gists_url":"https://api.github.com/users/projecttacoma/gists{/gist_id}","starred_url":"https://api.github.com/users/projecttacoma/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/projecttacoma/subscriptions","organizations_url":"https://api.github.com/users/projecttacoma/orgs","repos_url":"https://api.github.com/users/projecttacoma/repos","events_url":"https://api.github.com/users/projecttacoma/events{/privacy}","received_events_url":"https://api.github.com/users/projecttacoma/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/projecttacoma/bonnie","description":"Clinical Quality Measure Testing Tool","fork":false,"url":"https://api.github.com/repos/projecttacoma/bonnie","forks_url":"https://api.github.com/repos/projecttacoma/bonnie/forks","keys_url":"https://api.github.com/repos/projecttacoma/bonnie/keys{/key_id}","collaborators_url":"https://api.github.com/repos/projecttacoma/bonnie/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/projecttacoma/bonnie/teams","hooks_url":"https://api.github.com/repos/projecttacoma/bonnie/hooks","issue_events_url":"https://api.github.com/repos/projecttacoma/bonnie/issues/events{/number}","events_url":"https://api.github.com/repos/projecttacoma/bonnie/events","assignees_url":"https://api.github.com/repos/projecttacoma/bonnie/assignees{/user}","branches_url":"https://api.github.com/repos/projecttacoma/bonnie/branches{/branch}","tags_url":"https://api.github.com/repos/projecttacoma/bonnie/tags","blobs_url":"https://api.github.com/repos/projecttacoma/bonnie/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/projecttacoma/bonnie/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/projecttacoma/bonnie/git/refs{/sha}","trees_url":"https://api.github.com/repos/projecttacoma/bonnie/git/trees{/sha}","statuses_url":"https://api.github.com/repos/projecttacoma/bonnie/statuses/{sha}","languages_url":"https://api.github.com/repos/projecttacoma/bonnie/languages","stargazers_url":"https://api.github.com/repos/projecttacoma/bonnie/stargazers","contributors_url":"https://api.github.com/repos/projecttacoma/bonnie/contributors","subscribers_url":"https://api.github.com/repos/projecttacoma/bonnie/subscribers","subscription_url":"https://api.github.com/repos/projecttacoma/bonnie/subscription","commits_url":"https://api.github.com/repos/projecttacoma/bonnie/commits{/sha}","git_commits_url":"https://api.github.com/repos/projecttacoma/bonnie/git/commits{/sha}","comments_url":"https://api.github.com/repos/projecttacoma/bonnie/comments{/number}","issue_comment_url":"https://api.github.com/repos/projecttacoma/bonnie/issues/comments{/number}","contents_url":"https://api.github.com/repos/projecttacoma/bonnie/contents/{+path}","compare_url":"https://api.github.com/repos/projecttacoma/bonnie/compare/{base}...{head}","merges_url":"https://api.github.com/repos/projecttacoma/bonnie/merges","archive_url":"https://api.github.com/repos/projecttacoma/bonnie/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/projecttacoma/bonnie/downloads","issues_url":"https://api.github.com/repos/projecttacoma/bonnie/issues{/number}","pulls_url":"https://api.github.com/repos/projecttacoma/bonnie/pulls{/number}","milestones_url":"https://api.github.com/repos/projecttacoma/bonnie/milestones{/number}","notifications_url":"https://api.github.com/repos/projecttacoma/bonnie/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/projecttacoma/bonnie/labels{/name}","releases_url":"https://api.github.com/repos/projecttacoma/bonnie/releases{/id}","deployments_url":"https://api.github.com/repos/projecttacoma/bonnie/deployments","created_at":"2013-09-19T20:11:14Z","updated_at":"2016-12-17T16:22:39Z","pushed_at":"2016-12-21T17:51:40Z","git_url":"git://github.com/projecttacoma/bonnie.git","ssh_url":"git@github.com:projecttacoma/bonnie.git","clone_url":"https://github.com/projecttacoma/bonnie.git","svn_url":"https://github.com/projecttacoma/bonnie","homepage":null,"size":99340,"stargazers_count":17,"watchers_count":17,"language":"CoffeeScript","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":17,"mirror_url":null,"open_issues_count":14,"forks":17,"open_issues":14,"watchers":17,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/projecttacoma/bonnie/pulls/626"},"html":{"href":"https://github.com/projecttacoma/bonnie/pull/626"},"issue":{"href":"https://api.github.com/repos/projecttacoma/bonnie/issues/626"},"comments":{"href":"https://api.github.com/repos/projecttacoma/bonnie/issues/626/comments"},"review_comments":{"href":"https://api.github.com/repos/projecttacoma/bonnie/pulls/626/comments"},"review_comment":{"href":"https://api.github.com/repos/projecttacoma/bonnie/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/projecttacoma/bonnie/pulls/626/commits"},"statuses":{"href":"https://api.github.com/repos/projecttacoma/bonnie/statuses/cb3a72d1bde72b6632e2d794b475b652f6fe0e8e"}}}}
{"actor":{"display_login":"mulcahyk"}}