{"comment":{"url":"https://api.github.com/repos/gkrawiec/Wall-B/comments/9724868","html_url":"https://github.com/gkrawiec/Wall-B/commit/bc53251bc020901a6c56645a91d550b29f2a7fd6#commitcomment-9724868","id":9724868,"user":{"login":"jfarmer","id":37304,"avatar_url":"https://avatars.githubusercontent.com/u/37304?v=3","gravatar_id":"","url":"https://api.github.com/users/jfarmer","html_url":"https://github.com/jfarmer","followers_url":"https://api.github.com/users/jfarmer/followers","following_url":"https://api.github.com/users/jfarmer/following{/other_user}","gists_url":"https://api.github.com/users/jfarmer/gists{/gist_id}","starred_url":"https://api.github.com/users/jfarmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jfarmer/subscriptions","organizations_url":"https://api.github.com/users/jfarmer/orgs","repos_url":"https://api.github.com/users/jfarmer/repos","events_url":"https://api.github.com/users/jfarmer/events{/privacy}","received_events_url":"https://api.github.com/users/jfarmer/received_events","type":"User","site_admin":false},"position":5,"line":4,"path":"views/delete_wall.erb","commit_id":"bc53251bc020901a6c56645a91d550b29f2a7fd6","created_at":"2015-02-12T20:56:56Z","updated_at":"2015-02-12T20:56:56Z","body":"Use `<%= wall.id %>` instead of `params[:id]` here.  Using `params[:id]` creates a hidden dependency between your view and the code rendering the view, viz., the `:id` param must correspond to a `Wall` id and must always be present.  The only thing your view should need as data is the `Wall` you're looking to display and your view shouldn't care about what information was needed to get that `Wall`."}}