GitHub PullRequestReviewCommentEvent at 2014-12-22 12:26:38
Repository: zf2
Actor: Ocramius
Payload: {"action":"created","comment":{"url":"https://api.github.com/repos/zendframework/zf2/pulls/comments/22164020","id":22164020,"diff_hunk":"@@ -40,13 +40,45 @@ public function __construct($options = null)\n \n     /**\n      * Returns the name of the set adapter\n-     * @todo inconsitent: get adapter should return the adapter and not the name\n      *\n-     * @return string\n+     * @throws Exception\\RuntimeException\n+     * @throws Exception\\InvalidArgumentException\n+     * @return Encrypt\\EncryptionAlgorithmInterface","path":"library/Zend/Filter/Encrypt.php","position":9,"original_position":9,"commit_id":"1f6733d4dfbc5aba1e405528039ad5d63416a981","original_commit_id":"1f6733d4dfbc5aba1e405528039ad5d63416a981","user":{"login":"Ocramius","id":154256,"avatar_url":"https://avatars.githubusercontent.com/u/154256?v=3","gravatar_id":"","url":"https://api.github.com/users/Ocramius","html_url":"https://github.com/Ocramius","followers_url":"https://api.github.com/users/Ocramius/followers","following_url":"https://api.github.com/users/Ocramius/following{/other_user}","gists_url":"https://api.github.com/users/Ocramius/gists{/gist_id}","starred_url":"https://api.github.com/users/Ocramius/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Ocramius/subscriptions","organizations_url":"https://api.github.com/users/Ocramius/orgs","repos_url":"https://api.github.com/users/Ocramius/repos","events_url":"https://api.github.com/users/Ocramius/events{/privacy}","received_events_url":"https://api.github.com/users/Ocramius/received_events","type":"User","site_admin":false},"body":"Yes, but breaks have to be avoided where possible","created_at":"2014-12-22T12:26:38Z","updated_at":"2014-12-22T12:26:38Z","html_url":"https://github.com/zendframework/zf2/pull/6758#discussion_r22164020","pull_request_url":"https://api.github.com/repos/zendframework/zf2/pulls/6758","_links":{"self":{"href":"https://api.github.com/repos/zendframework/zf2/pulls/comments/22164020"},"html":{"href":"https://github.com/zendframework/zf2/pull/6758#discussion_r22164020"},"pull_request":{"href":"https://api.github.com/repos/zendframework/zf2/pulls/6758"}}},"pull_request":{"url":"https://api.github.com/repos/zendframework/zf2/pulls/6758","id":22642845,"html_url":"https://github.com/zendframework/zf2/pull/6758","diff_url":"https://github.com/zendframework/zf2/pull/6758.diff","patch_url":"https://github.com/zendframework/zf2/pull/6758.patch","issue_url":"https://api.github.com/repos/zendframework/zf2/issues/6758","number":6758,"state":"open","locked":false,"title":"implements @todo for getAdapter() return adapter instance at Zend\\Filter\\Encrypt","user":{"login":"samsonasik","id":459648,"avatar_url":"https://avatars.githubusercontent.com/u/459648?v=3","gravatar_id":"","url":"https://api.github.com/users/samsonasik","html_url":"https://github.com/samsonasik","followers_url":"https://api.github.com/users/samsonasik/followers","following_url":"https://api.github.com/users/samsonasik/following{/other_user}","gists_url":"https://api.github.com/users/samsonasik/gists{/gist_id}","starred_url":"https://api.github.com/users/samsonasik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/samsonasik/subscriptions","organizations_url":"https://api.github.com/users/samsonasik/orgs","repos_url":"https://api.github.com/users/samsonasik/repos","events_url":"https://api.github.com/users/samsonasik/events{/privacy}","received_events_url":"https://api.github.com/users/samsonasik/received_events","type":"User","site_admin":false},"body":"and add <code>getAdapterName()</code> to get name of adapter same as <code>Zend\\Filter\\Compress</code> class to make it consistent.","created_at":"2014-10-13T12:54:42Z","updated_at":"2014-12-22T12:26:38Z","closed_at":null,"merged_at":null,"merge_commit_sha":"e9c996d21df5085d9190b2810430b4ec6d544b1d","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/zendframework/zf2/pulls/6758/commits","review_comments_url":"https://api.github.com/repos/zendframework/zf2/pulls/6758/comments","review_comment_url":"https://api.github.com/repos/zendframework/zf2/pulls/comments/{number}","comments_url":"https://api.github.com/repos/zendframework/zf2/issues/6758/comments","statuses_url":"https://api.github.com/repos/zendframework/zf2/statuses/1f6733d4dfbc5aba1e405528039ad5d63416a981","head":{"label":"samsonasik:imp.todo.adapter","ref":"imp.todo.adapter","sha":"1f6733d4dfbc5aba1e405528039ad5d63416a981","user":{"login":"samsonasik","id":459648,"avatar_url":"https://avatars.githubusercontent.com/u/459648?v=3","gravatar_id":"","url":"https://api.github.com/users/samsonasik","html_url":"https://github.com/samsonasik","followers_url":"https://api.github.com/users/samsonasik/followers","following_url":"https://api.github.com/users/samsonasik/following{/other_user}","gists_url":"https://api.github.com/users/samsonasik/gists{/gist_id}","starred_url":"https://api.github.com/users/samsonasik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/samsonasik/subscriptions","organizations_url":"https://api.github.com/users/samsonasik/orgs","repos_url":"https://api.github.com/users/samsonasik/repos","events_url":"https://api.github.com/users/samsonasik/events{/privacy}","received_events_url":"https://api.github.com/users/samsonasik/received_events","type":"User","site_admin":false},"repo":{"id":5946915,"name":"zf2","full_name":"samsonasik/zf2","owner":{"login":"samsonasik","id":459648,"avatar_url":"https://avatars.githubusercontent.com/u/459648?v=3","gravatar_id":"","url":"https://api.github.com/users/samsonasik","html_url":"https://github.com/samsonasik","followers_url":"https://api.github.com/users/samsonasik/followers","following_url":"https://api.github.com/users/samsonasik/following{/other_user}","gists_url":"https://api.github.com/users/samsonasik/gists{/gist_id}","starred_url":"https://api.github.com/users/samsonasik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/samsonasik/subscriptions","organizations_url":"https://api.github.com/users/samsonasik/orgs","repos_url":"https://api.github.com/users/samsonasik/repos","events_url":"https://api.github.com/users/samsonasik/events{/privacy}","received_events_url":"https://api.github.com/users/samsonasik/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/samsonasik/zf2","description":"Official Zend Framework 2.0 git repository","fork":true,"url":"https://api.github.com/repos/samsonasik/zf2","forks_url":"https://api.github.com/repos/samsonasik/zf2/forks","keys_url":"https://api.github.com/repos/samsonasik/zf2/keys{/key_id}","collaborators_url":"https://api.github.com/repos/samsonasik/zf2/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/samsonasik/zf2/teams","hooks_url":"https://api.github.com/repos/samsonasik/zf2/hooks","issue_events_url":"https://api.github.com/repos/samsonasik/zf2/issues/events{/number}","events_url":"https://api.github.com/repos/samsonasik/zf2/events","assignees_url":"https://api.github.com/repos/samsonasik/zf2/assignees{/user}","branches_url":"https://api.github.com/repos/samsonasik/zf2/branches{/branch}","tags_url":"https://api.github.com/repos/samsonasik/zf2/tags","blobs_url":"https://api.github.com/repos/samsonasik/zf2/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/samsonasik/zf2/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/samsonasik/zf2/git/refs{/sha}","trees_url":"https://api.github.com/repos/samsonasik/zf2/git/trees{/sha}","statuses_url":"https://api.github.com/repos/samsonasik/zf2/statuses/{sha}","languages_url":"https://api.github.com/repos/samsonasik/zf2/languages","stargazers_url":"https://api.github.com/repos/samsonasik/zf2/stargazers","contributors_url":"https://api.github.com/repos/samsonasik/zf2/contributors","subscribers_url":"https://api.github.com/repos/samsonasik/zf2/subscribers","subscription_url":"https://api.github.com/repos/samsonasik/zf2/subscription","commits_url":"https://api.github.com/repos/samsonasik/zf2/commits{/sha}","git_commits_url":"https://api.github.com/repos/samsonasik/zf2/git/commits{/sha}","comments_url":"https://api.github.com/repos/samsonasik/zf2/comments{/number}","issue_comment_url":"https://api.github.com/repos/samsonasik/zf2/issues/comments/{number}","contents_url":"https://api.github.com/repos/samsonasik/zf2/contents/{+path}","compare_url":"https://api.github.com/repos/samsonasik/zf2/compare/{base}...{head}","merges_url":"https://api.github.com/repos/samsonasik/zf2/merges","archive_url":"https://api.github.com/repos/samsonasik/zf2/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/samsonasik/zf2/downloads","issues_url":"https://api.github.com/repos/samsonasik/zf2/issues{/number}","pulls_url":"https://api.github.com/repos/samsonasik/zf2/pulls{/number}","milestones_url":"https://api.github.com/repos/samsonasik/zf2/milestones{/number}","notifications_url":"https://api.github.com/repos/samsonasik/zf2/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/samsonasik/zf2/labels{/name}","releases_url":"https://api.github.com/repos/samsonasik/zf2/releases{/id}","created_at":"2012-09-25T08:19:09Z","updated_at":"2014-12-13T21:54:40Z","pushed_at":"2014-12-22T10:49:25Z","git_url":"git://github.com/samsonasik/zf2.git","ssh_url":"git@github.com:samsonasik/zf2.git","clone_url":"https://github.com/samsonasik/zf2.git","svn_url":"https://github.com/samsonasik/zf2","homepage":"http://framework.zend.com/zf2","size":96087,"stargazers_count":0,"watchers_count":0,"language":"PHP","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":1,"mirror_url":null,"open_issues_count":0,"forks":1,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"zendframework:develop","ref":"develop","sha":"e6f6eb70d28aa1b69b145b6f3519234958122434","user":{"login":"zendframework","id":296074,"avatar_url":"https://avatars.githubusercontent.com/u/296074?v=3","gravatar_id":"","url":"https://api.github.com/users/zendframework","html_url":"https://github.com/zendframework","followers_url":"https://api.github.com/users/zendframework/followers","following_url":"https://api.github.com/users/zendframework/following{/other_user}","gists_url":"https://api.github.com/users/zendframework/gists{/gist_id}","starred_url":"https://api.github.com/users/zendframework/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zendframework/subscriptions","organizations_url":"https://api.github.com/users/zendframework/orgs","repos_url":"https://api.github.com/users/zendframework/repos","events_url":"https://api.github.com/users/zendframework/events{/privacy}","received_events_url":"https://api.github.com/users/zendframework/received_events","type":"Organization","site_admin":false},"repo":{"id":702550,"name":"zf2","full_name":"zendframework/zf2","owner":{"login":"zendframework","id":296074,"avatar_url":"https://avatars.githubusercontent.com/u/296074?v=3","gravatar_id":"","url":"https://api.github.com/users/zendframework","html_url":"https://github.com/zendframework","followers_url":"https://api.github.com/users/zendframework/followers","following_url":"https://api.github.com/users/zendframework/following{/other_user}","gists_url":"https://api.github.com/users/zendframework/gists{/gist_id}","starred_url":"https://api.github.com/users/zendframework/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zendframework/subscriptions","organizations_url":"https://api.github.com/users/zendframework/orgs","repos_url":"https://api.github.com/users/zendframework/repos","events_url":"https://api.github.com/users/zendframework/events{/privacy}","received_events_url":"https://api.github.com/users/zendframework/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/zendframework/zf2","description":"Official Zend Framework 2 git repository","fork":false,"url":"https://api.github.com/repos/zendframework/zf2","forks_url":"https://api.github.com/repos/zendframework/zf2/forks","keys_url":"https://api.github.com/repos/zendframework/zf2/keys{/key_id}","collaborators_url":"https://api.github.com/repos/zendframework/zf2/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/zendframework/zf2/teams","hooks_url":"https://api.github.com/repos/zendframework/zf2/hooks","issue_events_url":"https://api.github.com/repos/zendframework/zf2/issues/events{/number}","events_url":"https://api.github.com/repos/zendframework/zf2/events","assignees_url":"https://api.github.com/repos/zendframework/zf2/assignees{/user}","branches_url":"https://api.github.com/repos/zendframework/zf2/branches{/branch}","tags_url":"https://api.github.com/repos/zendframework/zf2/tags","blobs_url":"https://api.github.com/repos/zendframework/zf2/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/zendframework/zf2/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/zendframework/zf2/git/refs{/sha}","trees_url":"https://api.github.com/repos/zendframework/zf2/git/trees{/sha}","statuses_url":"https://api.github.com/repos/zendframework/zf2/statuses/{sha}","languages_url":"https://api.github.com/repos/zendframework/zf2/languages","stargazers_url":"https://api.github.com/repos/zendframework/zf2/stargazers","contributors_url":"https://api.github.com/repos/zendframework/zf2/contributors","subscribers_url":"https://api.github.com/repos/zendframework/zf2/subscribers","subscription_url":"https://api.github.com/repos/zendframework/zf2/subscription","commits_url":"https://api.github.com/repos/zendframework/zf2/commits{/sha}","git_commits_url":"https://api.github.com/repos/zendframework/zf2/git/commits{/sha}","comments_url":"https://api.github.com/repos/zendframework/zf2/comments{/number}","issue_comment_url":"https://api.github.com/repos/zendframework/zf2/issues/comments/{number}","contents_url":"https://api.github.com/repos/zendframework/zf2/contents/{+path}","compare_url":"https://api.github.com/repos/zendframework/zf2/compare/{base}...{head}","merges_url":"https://api.github.com/repos/zendframework/zf2/merges","archive_url":"https://api.github.com/repos/zendframework/zf2/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/zendframework/zf2/downloads","issues_url":"https://api.github.com/repos/zendframework/zf2/issues{/number}","pulls_url":"https://api.github.com/repos/zendframework/zf2/pulls{/number}","milestones_url":"https://api.github.com/repos/zendframework/zf2/milestones{/number}","notifications_url":"https://api.github.com/repos/zendframework/zf2/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/zendframework/zf2/labels{/name}","releases_url":"https://api.github.com/repos/zendframework/zf2/releases{/id}","created_at":"2010-06-04T02:42:05Z","updated_at":"2014-12-21T14:39:57Z","pushed_at":"2014-12-17T12:49:55Z","git_url":"git://github.com/zendframework/zf2.git","ssh_url":"git@github.com:zendframework/zf2.git","clone_url":"https://github.com/zendframework/zf2.git","svn_url":"https://github.com/zendframework/zf2","homepage":"http://framework.zend.com/","size":212876,"stargazers_count":4959,"watchers_count":4959,"language":"PHP","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3011,"mirror_url":null,"open_issues_count":623,"forks":3011,"open_issues":623,"watchers":4959,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/zendframework/zf2/pulls/6758"},"html":{"href":"https://github.com/zendframework/zf2/pull/6758"},"issue":{"href":"https://api.github.com/repos/zendframework/zf2/issues/6758"},"comments":{"href":"https://api.github.com/repos/zendframework/zf2/issues/6758/comments"},"review_comments":{"href":"https://api.github.com/repos/zendframework/zf2/pulls/6758/comments"},"review_comment":{"href":"https://api.github.com/repos/zendframework/zf2/pulls/comments/{number}"},"commits":{"href":"https://api.github.com/repos/zendframework/zf2/pulls/6758/commits"},"statuses":{"href":"https://api.github.com/repos/zendframework/zf2/statuses/1f6733d4dfbc5aba1e405528039ad5d63416a981"}}}}