GitHub PullRequestReviewCommentEvent at 2014-04-24 17:54:46
Repository: serrano
Actor: naegelyd
Payload: {"comment":{"url":"https://api.github.com/repos/cbmi/serrano/pulls/comments/11961681","id":11961681,"diff_hunk":"@@ -83,7 +83,13 @@ def _get_request_object(request, attrs=None, klass=None, key=None):\n         kwargs['session'] = True\n \n     try:\n-        return klass.objects.get(**kwargs)","path":"serrano/resources/base.py","position":4,"original_position":4,"commit_id":"07bd7390400513b05925953e6cda4c0c259b7aac","original_commit_id":"07bd7390400513b05925953e6cda4c0c259b7aac","user":{"login":"naegelyd","id":3911912,"avatar_url":"https://avatars.githubusercontent.com/u/3911912?","gravatar_id":"f2b6f2cec914521893ab3f5de23d1a4c","url":"https://api.github.com/users/naegelyd","html_url":"https://github.com/naegelyd","followers_url":"https://api.github.com/users/naegelyd/followers","following_url":"https://api.github.com/users/naegelyd/following{/other_user}","gists_url":"https://api.github.com/users/naegelyd/gists{/gist_id}","starred_url":"https://api.github.com/users/naegelyd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/naegelyd/subscriptions","organizations_url":"https://api.github.com/users/naegelyd/orgs","repos_url":"https://api.github.com/users/naegelyd/repos","events_url":"https://api.github.com/users/naegelyd/events{/privacy}","received_events_url":"https://api.github.com/users/naegelyd/received_events","type":"User","site_admin":false},"body":"I do not think that filter will every throw `klass.DoesNotExist` so I believe this will need to be updated to something like:\r\n\r\n```python\r\nobjects = klass.objects.filter(**kwargs)\r\n\r\nif objects.exists():\r\n    return objects.latest('modified')\r\n```","created_at":"2014-04-24T17:54:46Z","updated_at":"2014-04-24T17:54:46Z","html_url":"https://github.com/cbmi/serrano/pull/179#discussion_r11961681","pull_request_url":"https://api.github.com/repos/cbmi/serrano/pulls/179","_links":{"self":{"href":"https://api.github.com/repos/cbmi/serrano/pulls/comments/11961681"},"html":{"href":"https://github.com/cbmi/serrano/pull/179#discussion_r11961681"},"pull_request":{"href":"https://api.github.com/repos/cbmi/serrano/pulls/179"}}}}