GitHub PullRequestReviewCommentEvent at 2014-02-06 19:52:13
Repository: wdi_4_rails_hw_nested_has_many
Actor: grantovich
Payload: {"comment":{"url":"https://api.github.com/repos/ga-wdi-boston/wdi_4_rails_hw_nested_has_many/pulls/comments/9515323","id":9515323,"diff_hunk":"@@ -0,0 +1,53 @@\n+class ArticlesController < ApplicationController\n+\tbefore_action :find_article, only: [:show, :edit, :update]\n+  before_action :find_user, only: [:new, :create, :edit, :update, :show]\n+\n+\tdef index\n+\t\tif params[:user_id]","path":"app/controllers/articles_controller.rb","position":6,"original_position":6,"commit_id":"4d5486ce01f2b559c089b6ac24d02e50712a28f4","original_commit_id":"4d5486ce01f2b559c089b6ac24d02e50712a28f4","user":{"login":"grantovich","id":394835,"avatar_url":"https://gravatar.com/avatar/b6c92201404bd9b616fbba5e783cb3d8?d=https%3A%2F%2Fidenticons.github.com%2F50bf48b85ed71ba0f25be359e2495785.png&r=x","gravatar_id":"b6c92201404bd9b616fbba5e783cb3d8","url":"https://api.github.com/users/grantovich","html_url":"https://github.com/grantovich","followers_url":"https://api.github.com/users/grantovich/followers","following_url":"https://api.github.com/users/grantovich/following{/other_user}","gists_url":"https://api.github.com/users/grantovich/gists{/gist_id}","starred_url":"https://api.github.com/users/grantovich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/grantovich/subscriptions","organizations_url":"https://api.github.com/users/grantovich/orgs","repos_url":"https://api.github.com/users/grantovich/repos","events_url":"https://api.github.com/users/grantovich/events{/privacy}","received_events_url":"https://api.github.com/users/grantovich/received_events","type":"User","site_admin":false},"body":"The more conventional way to check this would be `if params[:user_id].present?` (this catches things like the value being a blank string instead of nil)","created_at":"2014-02-06T19:52:13Z","updated_at":"2014-02-06T19:52:13Z","html_url":"https://github.com/ga-wdi-boston/wdi_4_rails_hw_nested_has_many/pull/10#discussion_r9515323","pull_request_url":"https://api.github.com/repos/ga-wdi-boston/wdi_4_rails_hw_nested_has_many/pulls/10","_links":{"self":{"href":"https://api.github.com/repos/ga-wdi-boston/wdi_4_rails_hw_nested_has_many/pulls/comments/9515323"},"html":{"href":"https://github.com/ga-wdi-boston/wdi_4_rails_hw_nested_has_many/pull/10#discussion_r9515323"},"pull_request":{"href":"https://api.github.com/repos/ga-wdi-boston/wdi_4_rails_hw_nested_has_many/pulls/10"}}}}