GitHub PullRequestReviewCommentEvent at 2014-01-13 15:12:15
Repository: vraptor4
Actor: garcia-jj
Payload: {"comment":{"url":"https://api.github.com/repos/caelum/vraptor4/pulls/comments/8827460","id":8827460,"diff_hunk":"@@ -47,10 +47,10 @@ public SessionFlashScope(HttpSession session) {\n \n \t@Override\n \tpublic Object[] consumeParameters(ControllerMethod method) {\n-\t\tObject[] args = (Object[]) session.getAttribute(nameFor(method));\n-\t\tif (args != null) {\n-\t\t\tsession.removeAttribute(nameFor(method));\n-\t\t}\n+\t\tString param = nameFor(method);\n+\t\tObject[] args = (Object[]) session.getAttribute(param);\n+\t\tsession.removeAttribute(param);","path":"vraptor-core/src/main/java/br/com/caelum/vraptor/view/SessionFlashScope.java","position":10,"original_position":10,"commit_id":"62fe19ddb7501c4371a8646e2f4316cb70ee0eaf","original_commit_id":"62fe19ddb7501c4371a8646e2f4316cb70ee0eaf","user":{"login":"garcia-jj","id":140387,"avatar_url":"https://gravatar.com/avatar/471af576b6918dbd3401168f7a42f098?d=https%3A%2F%2Fidenticons.github.com%2F15d891c8f5c95aa42fc72145a8678760.png&r=x","gravatar_id":"471af576b6918dbd3401168f7a42f098","url":"https://api.github.com/users/garcia-jj","html_url":"https://github.com/garcia-jj","followers_url":"https://api.github.com/users/garcia-jj/followers","following_url":"https://api.github.com/users/garcia-jj/following{/other_user}","gists_url":"https://api.github.com/users/garcia-jj/gists{/gist_id}","starred_url":"https://api.github.com/users/garcia-jj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/garcia-jj/subscriptions","organizations_url":"https://api.github.com/users/garcia-jj/orgs","repos_url":"https://api.github.com/users/garcia-jj/repos","events_url":"https://api.github.com/users/garcia-jj/events{/privacy}","received_events_url":"https://api.github.com/users/garcia-jj/received_events","type":"User","site_admin":false},"body":"By Javadocs: Removes the object bound with the specified name from this session. If the session does **not have** an object bound with the specified name, this method **does nothing**.\r\n","created_at":"2014-01-13T15:12:15Z","updated_at":"2014-01-13T15:12:15Z","html_url":"https://github.com/caelum/vraptor4/pull/309#discussion_r8827460","pull_request_url":"https://api.github.com/repos/caelum/vraptor4/pulls/309","_links":{"self":{"href":"https://api.github.com/repos/caelum/vraptor4/pulls/comments/8827460"},"html":{"href":"https://github.com/caelum/vraptor4/pull/309#discussion_r8827460"},"pull_request":{"href":"https://api.github.com/repos/caelum/vraptor4/pulls/309"}}}}