GitHub PullRequestReviewCommentEvent at 2014-02-22 15:02:54
Repository: Elgg
Actor: git001
Payload: {"comment":{"url":"https://api.github.com/repos/Elgg/Elgg/pulls/comments/9971466","id":9971466,"diff_hunk":"@@ -948,7 +948,7 @@ function elgg_view_annotation(ElggAnnotation $annotation, array $vars = array(),\n function elgg_view_entity_list($entities, $vars = array(), $offset = 0, $limit = 10, $full_view = true,\n $list_type_toggle = true, $pagination = true) {\n \n-\tif (!$vars[\"limit\"] && !$vars[\"offset\"]) {\n+\tif (!$vars[\"limit\"] && (array_key_exists('offset',$vars) && !is_null($vars[\"offset\"]))) {","path":"engine/lib/views.php","position":5,"original_position":5,"commit_id":"0b673a237a436ec04c3df45678b4ae702f7e6798","original_commit_id":"0b673a237a436ec04c3df45678b4ae702f7e6798","user":{"login":"git001","id":1579468,"avatar_url":"https://gravatar.com/avatar/e6967bf97986d357b31681aaf4ef11ae?d=https%3A%2F%2Fidenticons.github.com%2F6992177a6f67f1cf121421f4f40946aa.png&r=x","gravatar_id":"e6967bf97986d357b31681aaf4ef11ae","url":"https://api.github.com/users/git001","html_url":"https://github.com/git001","followers_url":"https://api.github.com/users/git001/followers","following_url":"https://api.github.com/users/git001/following{/other_user}","gists_url":"https://api.github.com/users/git001/gists{/gist_id}","starred_url":"https://api.github.com/users/git001/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/git001/subscriptions","organizations_url":"https://api.github.com/users/git001/orgs","repos_url":"https://api.github.com/users/git001/repos","events_url":"https://api.github.com/users/git001/events{/privacy}","received_events_url":"https://api.github.com/users/git001/received_events","type":"User","site_admin":false},"body":"Up to you. I have no preference about the place of the fix.","created_at":"2014-02-22T15:02:54Z","updated_at":"2014-02-22T15:02:54Z","html_url":"https://github.com/Elgg/Elgg/pull/6552#discussion_r9971466","pull_request_url":"https://api.github.com/repos/Elgg/Elgg/pulls/6552","_links":{"self":{"href":"https://api.github.com/repos/Elgg/Elgg/pulls/comments/9971466"},"html":{"href":"https://github.com/Elgg/Elgg/pull/6552#discussion_r9971466"},"pull_request":{"href":"https://api.github.com/repos/Elgg/Elgg/pulls/6552"}}}}