{"action":"created","comment":{"url":"https://api.github.com/repos/Automattic/jetpack/pulls/comments/37183395","id":37183395,"diff_hunk":"@@ -1084,14 +1101,16 @@ function get_media_item_v1_1( $media_id ) {\n \n \t\tif ( in_array( $ext, array( 'mp3', 'm4a', 'wav', 'ogg' ) ) ) {\n \t\t\t$metadata = wp_get_attachment_metadata( $media_item->ID );\n+\t\t\t$response['length'] = $metadata['length'];\n \t\t\t$response['exif']   = $metadata;\n \t\t}\n \n \t\tif ( in_array( $ext, array( 'ogv', 'mp4', 'mov', 'wmv', 'avi', 'mpg', '3gp', '3g2', 'm4v' ) ) ) {\n \t\t\t$metadata = wp_get_attachment_metadata( $media_item->ID );\n-\t\t\tif ( isset( $metadata['height'], $metadata['width'] ) ) {\n+\t\t\tif ( isset( $metadata['height'], $metadata['width'], $metadata['length'] ) ) {","path":"class.json-api-endpoints.php","position":87,"original_position":87,"commit_id":"ff6c3652ed86f6c8f12afa96983f65325d40bfb4","original_commit_id":"ff6c3652ed86f6c8f12afa96983f65325d40bfb4","user":{"login":"aduth","id":1779930,"avatar_url":"https://avatars.githubusercontent.com/u/1779930?v=3","gravatar_id":"","url":"https://api.github.com/users/aduth","html_url":"https://github.com/aduth","followers_url":"https://api.github.com/users/aduth/followers","following_url":"https://api.github.com/users/aduth/following{/other_user}","gists_url":"https://api.github.com/users/aduth/gists{/gist_id}","starred_url":"https://api.github.com/users/aduth/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aduth/subscriptions","organizations_url":"https://api.github.com/users/aduth/orgs","repos_url":"https://api.github.com/users/aduth/repos","events_url":"https://api.github.com/users/aduth/events{/privacy}","received_events_url":"https://api.github.com/users/aduth/received_events","type":"User","site_admin":false},"body":"Should this `isset` check be back-ported to WPCOM? Also, I'd expect to see it guarding each property separately, or at least separating dimensions (height/width) from length.","created_at":"2015-08-17T12:43:16Z","updated_at":"2015-08-17T12:43:16Z","html_url":"https://github.com/Automattic/jetpack/pull/2553#discussion_r37183395","pull_request_url":"https://api.github.com/repos/Automattic/jetpack/pulls/2553","_links":{"self":{"href":"https://api.github.com/repos/Automattic/jetpack/pulls/comments/37183395"},"html":{"href":"https://github.com/Automattic/jetpack/pull/2553#discussion_r37183395"},"pull_request":{"href":"https://api.github.com/repos/Automattic/jetpack/pulls/2553"}}},"pull_request":{"url":"https://api.github.com/repos/Automattic/jetpack/pulls/2553","id":42501957,"html_url":"https://github.com/Automattic/jetpack/pull/2553","diff_url":"https://github.com/Automattic/jetpack/pull/2553.diff","patch_url":"https://github.com/Automattic/jetpack/pull/2553.patch","issue_url":"https://api.github.com/repos/Automattic/jetpack/issues/2553","number":2553,"state":"open","locked":false,"title":"Merge wpcom API changes","user":{"login":"lezama","id":104869,"avatar_url":"https://avatars.githubusercontent.com/u/104869?v=3","gravatar_id":"","url":"https://api.github.com/users/lezama","html_url":"https://github.com/lezama","followers_url":"https://api.github.com/users/lezama/followers","following_url":"https://api.github.com/users/lezama/following{/other_user}","gists_url":"https://api.github.com/users/lezama/gists{/gist_id}","starred_url":"https://api.github.com/users/lezama/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lezama/subscriptions","organizations_url":"https://api.github.com/users/lezama/orgs","repos_url":"https://api.github.com/users/lezama/repos","events_url":"https://api.github.com/users/lezama/events{/privacy}","received_events_url":"https://api.github.com/users/lezama/received_events","type":"User","site_admin":false},"body":"Could you review your changes?\r\nPlease check your name if they are good to go on 3.7 :)\r\n\r\n- [ ] @aduth\r\n- [ ] @jkudish\r\n- [ ] @allendav\r\n- [ ] @codebykat\r\n- [ ] @ebinnion\r\n- [ ] @gcorne\r\n- [ ] @rralian\r\n- [ ] @ockham\r\n- [ ] @mtias \r\n- [ ] @enejb \r\n- [x] @mdawaffe \r\n- [ ] @blowery \r\n","created_at":"2015-08-14T21:58:27Z","updated_at":"2015-08-17T12:43:16Z","closed_at":null,"merged_at":null,"merge_commit_sha":"8b88fc6e9b4ca1316940044b4e60f304d3c16b2e","assignee":{"login":"zinigor","id":374293,"avatar_url":"https://avatars.githubusercontent.com/u/374293?v=3","gravatar_id":"","url":"https://api.github.com/users/zinigor","html_url":"https://github.com/zinigor","followers_url":"https://api.github.com/users/zinigor/followers","following_url":"https://api.github.com/users/zinigor/following{/other_user}","gists_url":"https://api.github.com/users/zinigor/gists{/gist_id}","starred_url":"https://api.github.com/users/zinigor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zinigor/subscriptions","organizations_url":"https://api.github.com/users/zinigor/orgs","repos_url":"https://api.github.com/users/zinigor/repos","events_url":"https://api.github.com/users/zinigor/events{/privacy}","received_events_url":"https://api.github.com/users/zinigor/received_events","type":"User","site_admin":false},"milestone":{"url":"https://api.github.com/repos/Automattic/jetpack/milestones/21","html_url":"https://github.com/Automattic/jetpack/milestones/3.7","labels_url":"https://api.github.com/repos/Automattic/jetpack/milestones/21/labels","id":1138279,"number":21,"title":"3.7","description":"","creator":{"login":"kraftbj","id":88897,"avatar_url":"https://avatars.githubusercontent.com/u/88897?v=3","gravatar_id":"","url":"https://api.github.com/users/kraftbj","html_url":"https://github.com/kraftbj","followers_url":"https://api.github.com/users/kraftbj/followers","following_url":"https://api.github.com/users/kraftbj/following{/other_user}","gists_url":"https://api.github.com/users/kraftbj/gists{/gist_id}","starred_url":"https://api.github.com/users/kraftbj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kraftbj/subscriptions","organizations_url":"https://api.github.com/users/kraftbj/orgs","repos_url":"https://api.github.com/users/kraftbj/repos","events_url":"https://api.github.com/users/kraftbj/events{/privacy}","received_events_url":"https://api.github.com/users/kraftbj/received_events","type":"User","site_admin":false},"open_issues":209,"closed_issues":107,"state":"open","created_at":"2015-05-28T15:18:58Z","updated_at":"2015-08-14T22:01:12Z","due_on":null,"closed_at":null},"commits_url":"https://api.github.com/repos/Automattic/jetpack/pulls/2553/commits","review_comments_url":"https://api.github.com/repos/Automattic/jetpack/pulls/2553/comments","review_comment_url":"https://api.github.com/repos/Automattic/jetpack/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Automattic/jetpack/issues/2553/comments","statuses_url":"https://api.github.com/repos/Automattic/jetpack/statuses/ff6c3652ed86f6c8f12afa96983f65325d40bfb4","head":{"label":"Automattic:merge-wpcom-api-changes-3.7","ref":"merge-wpcom-api-changes-3.7","sha":"ff6c3652ed86f6c8f12afa96983f65325d40bfb4","user":{"login":"Automattic","id":887802,"avatar_url":"https://avatars.githubusercontent.com/u/887802?v=3","gravatar_id":"","url":"https://api.github.com/users/Automattic","html_url":"https://github.com/Automattic","followers_url":"https://api.github.com/users/Automattic/followers","following_url":"https://api.github.com/users/Automattic/following{/other_user}","gists_url":"https://api.github.com/users/Automattic/gists{/gist_id}","starred_url":"https://api.github.com/users/Automattic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Automattic/subscriptions","organizations_url":"https://api.github.com/users/Automattic/orgs","repos_url":"https://api.github.com/users/Automattic/repos","events_url":"https://api.github.com/users/Automattic/events{/privacy}","received_events_url":"https://api.github.com/users/Automattic/received_events","type":"Organization","site_admin":false},"repo":{"id":15231212,"name":"jetpack","full_name":"Automattic/jetpack","owner":{"login":"Automattic","id":887802,"avatar_url":"https://avatars.githubusercontent.com/u/887802?v=3","gravatar_id":"","url":"https://api.github.com/users/Automattic","html_url":"https://github.com/Automattic","followers_url":"https://api.github.com/users/Automattic/followers","following_url":"https://api.github.com/users/Automattic/following{/other_user}","gists_url":"https://api.github.com/users/Automattic/gists{/gist_id}","starred_url":"https://api.github.com/users/Automattic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Automattic/subscriptions","organizations_url":"https://api.github.com/users/Automattic/orgs","repos_url":"https://api.github.com/users/Automattic/repos","events_url":"https://api.github.com/users/Automattic/events{/privacy}","received_events_url":"https://api.github.com/users/Automattic/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Automattic/jetpack","description":"Your WordPress, Streamlined.","fork":false,"url":"https://api.github.com/repos/Automattic/jetpack","forks_url":"https://api.github.com/repos/Automattic/jetpack/forks","keys_url":"https://api.github.com/repos/Automattic/jetpack/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Automattic/jetpack/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Automattic/jetpack/teams","hooks_url":"https://api.github.com/repos/Automattic/jetpack/hooks","issue_events_url":"https://api.github.com/repos/Automattic/jetpack/issues/events{/number}","events_url":"https://api.github.com/repos/Automattic/jetpack/events","assignees_url":"https://api.github.com/repos/Automattic/jetpack/assignees{/user}","branches_url":"https://api.github.com/repos/Automattic/jetpack/branches{/branch}","tags_url":"https://api.github.com/repos/Automattic/jetpack/tags","blobs_url":"https://api.github.com/repos/Automattic/jetpack/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Automattic/jetpack/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Automattic/jetpack/git/refs{/sha}","trees_url":"https://api.github.com/repos/Automattic/jetpack/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Automattic/jetpack/statuses/{sha}","languages_url":"https://api.github.com/repos/Automattic/jetpack/languages","stargazers_url":"https://api.github.com/repos/Automattic/jetpack/stargazers","contributors_url":"https://api.github.com/repos/Automattic/jetpack/contributors","subscribers_url":"https://api.github.com/repos/Automattic/jetpack/subscribers","subscription_url":"https://api.github.com/repos/Automattic/jetpack/subscription","commits_url":"https://api.github.com/repos/Automattic/jetpack/commits{/sha}","git_commits_url":"https://api.github.com/repos/Automattic/jetpack/git/commits{/sha}","comments_url":"https://api.github.com/repos/Automattic/jetpack/comments{/number}","issue_comment_url":"https://api.github.com/repos/Automattic/jetpack/issues/comments{/number}","contents_url":"https://api.github.com/repos/Automattic/jetpack/contents/{+path}","compare_url":"https://api.github.com/repos/Automattic/jetpack/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Automattic/jetpack/merges","archive_url":"https://api.github.com/repos/Automattic/jetpack/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Automattic/jetpack/downloads","issues_url":"https://api.github.com/repos/Automattic/jetpack/issues{/number}","pulls_url":"https://api.github.com/repos/Automattic/jetpack/pulls{/number}","milestones_url":"https://api.github.com/repos/Automattic/jetpack/milestones{/number}","notifications_url":"https://api.github.com/repos/Automattic/jetpack/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Automattic/jetpack/labels{/name}","releases_url":"https://api.github.com/repos/Automattic/jetpack/releases{/id}","created_at":"2013-12-16T16:43:01Z","updated_at":"2015-08-17T04:51:32Z","pushed_at":"2015-08-16T21:37:28Z","git_url":"git://github.com/Automattic/jetpack.git","ssh_url":"git@github.com:Automattic/jetpack.git","clone_url":"https://github.com/Automattic/jetpack.git","svn_url":"https://github.com/Automattic/jetpack","homepage":"http://jetpack.me","size":91407,"stargazers_count":386,"watchers_count":386,"language":"PHP","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":261,"mirror_url":null,"open_issues_count":621,"forks":261,"open_issues":621,"watchers":386,"default_branch":"master"}},"base":{"label":"Automattic:master","ref":"master","sha":"5ac4a3b768b4050c4d8253375ea77564c9b278e4","user":{"login":"Automattic","id":887802,"avatar_url":"https://avatars.githubusercontent.com/u/887802?v=3","gravatar_id":"","url":"https://api.github.com/users/Automattic","html_url":"https://github.com/Automattic","followers_url":"https://api.github.com/users/Automattic/followers","following_url":"https://api.github.com/users/Automattic/following{/other_user}","gists_url":"https://api.github.com/users/Automattic/gists{/gist_id}","starred_url":"https://api.github.com/users/Automattic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Automattic/subscriptions","organizations_url":"https://api.github.com/users/Automattic/orgs","repos_url":"https://api.github.com/users/Automattic/repos","events_url":"https://api.github.com/users/Automattic/events{/privacy}","received_events_url":"https://api.github.com/users/Automattic/received_events","type":"Organization","site_admin":false},"repo":{"id":15231212,"name":"jetpack","full_name":"Automattic/jetpack","owner":{"login":"Automattic","id":887802,"avatar_url":"https://avatars.githubusercontent.com/u/887802?v=3","gravatar_id":"","url":"https://api.github.com/users/Automattic","html_url":"https://github.com/Automattic","followers_url":"https://api.github.com/users/Automattic/followers","following_url":"https://api.github.com/users/Automattic/following{/other_user}","gists_url":"https://api.github.com/users/Automattic/gists{/gist_id}","starred_url":"https://api.github.com/users/Automattic/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Automattic/subscriptions","organizations_url":"https://api.github.com/users/Automattic/orgs","repos_url":"https://api.github.com/users/Automattic/repos","events_url":"https://api.github.com/users/Automattic/events{/privacy}","received_events_url":"https://api.github.com/users/Automattic/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Automattic/jetpack","description":"Your WordPress, Streamlined.","fork":false,"url":"https://api.github.com/repos/Automattic/jetpack","forks_url":"https://api.github.com/repos/Automattic/jetpack/forks","keys_url":"https://api.github.com/repos/Automattic/jetpack/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Automattic/jetpack/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Automattic/jetpack/teams","hooks_url":"https://api.github.com/repos/Automattic/jetpack/hooks","issue_events_url":"https://api.github.com/repos/Automattic/jetpack/issues/events{/number}","events_url":"https://api.github.com/repos/Automattic/jetpack/events","assignees_url":"https://api.github.com/repos/Automattic/jetpack/assignees{/user}","branches_url":"https://api.github.com/repos/Automattic/jetpack/branches{/branch}","tags_url":"https://api.github.com/repos/Automattic/jetpack/tags","blobs_url":"https://api.github.com/repos/Automattic/jetpack/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Automattic/jetpack/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Automattic/jetpack/git/refs{/sha}","trees_url":"https://api.github.com/repos/Automattic/jetpack/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Automattic/jetpack/statuses/{sha}","languages_url":"https://api.github.com/repos/Automattic/jetpack/languages","stargazers_url":"https://api.github.com/repos/Automattic/jetpack/stargazers","contributors_url":"https://api.github.com/repos/Automattic/jetpack/contributors","subscribers_url":"https://api.github.com/repos/Automattic/jetpack/subscribers","subscription_url":"https://api.github.com/repos/Automattic/jetpack/subscription","commits_url":"https://api.github.com/repos/Automattic/jetpack/commits{/sha}","git_commits_url":"https://api.github.com/repos/Automattic/jetpack/git/commits{/sha}","comments_url":"https://api.github.com/repos/Automattic/jetpack/comments{/number}","issue_comment_url":"https://api.github.com/repos/Automattic/jetpack/issues/comments{/number}","contents_url":"https://api.github.com/repos/Automattic/jetpack/contents/{+path}","compare_url":"https://api.github.com/repos/Automattic/jetpack/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Automattic/jetpack/merges","archive_url":"https://api.github.com/repos/Automattic/jetpack/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Automattic/jetpack/downloads","issues_url":"https://api.github.com/repos/Automattic/jetpack/issues{/number}","pulls_url":"https://api.github.com/repos/Automattic/jetpack/pulls{/number}","milestones_url":"https://api.github.com/repos/Automattic/jetpack/milestones{/number}","notifications_url":"https://api.github.com/repos/Automattic/jetpack/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Automattic/jetpack/labels{/name}","releases_url":"https://api.github.com/repos/Automattic/jetpack/releases{/id}","created_at":"2013-12-16T16:43:01Z","updated_at":"2015-08-17T04:51:32Z","pushed_at":"2015-08-16T21:37:28Z","git_url":"git://github.com/Automattic/jetpack.git","ssh_url":"git@github.com:Automattic/jetpack.git","clone_url":"https://github.com/Automattic/jetpack.git","svn_url":"https://github.com/Automattic/jetpack","homepage":"http://jetpack.me","size":91407,"stargazers_count":386,"watchers_count":386,"language":"PHP","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":261,"mirror_url":null,"open_issues_count":621,"forks":261,"open_issues":621,"watchers":386,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Automattic/jetpack/pulls/2553"},"html":{"href":"https://github.com/Automattic/jetpack/pull/2553"},"issue":{"href":"https://api.github.com/repos/Automattic/jetpack/issues/2553"},"comments":{"href":"https://api.github.com/repos/Automattic/jetpack/issues/2553/comments"},"review_comments":{"href":"https://api.github.com/repos/Automattic/jetpack/pulls/2553/comments"},"review_comment":{"href":"https://api.github.com/repos/Automattic/jetpack/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Automattic/jetpack/pulls/2553/commits"},"statuses":{"href":"https://api.github.com/repos/Automattic/jetpack/statuses/ff6c3652ed86f6c8f12afa96983f65325d40bfb4"}}}}