{"action":"created","comment":{"url":"https://api.github.com/repos/rg3/youtube-dl/pulls/comments/82571265","pull_request_review_id":3468935,"id":82571265,"diff_hunk":"@@ -50,7 +50,11 @@ def _parse_jwplayer_data(self, jwplayer_data, video_id=None, require_title=True,\n             # JWPlayer backward compatibility: flattened sources\n             # https://github.com/jwplayer/jwplayer/blob/v7.4.3/src/js/playlist/item.js#L29-L35\n             if 'sources' not in video_data:\n-                video_data['sources'] = [video_data]\n+                if isinstance(video_data, list):","path":"youtube_dl/extractor/jwplatform.py","position":5,"original_position":5,"commit_id":"cb2e9ec69e135aefb4bb1c8f3e40bcddac3bfd7a","original_commit_id":"cb2e9ec69e135aefb4bb1c8f3e40bcddac3bfd7a","user":{"login":"johnhawkinson","id":1270718,"avatar_url":"https://avatars.githubusercontent.com/u/1270718?v=3","gravatar_id":"","url":"https://api.github.com/users/johnhawkinson","html_url":"https://github.com/johnhawkinson","followers_url":"https://api.github.com/users/johnhawkinson/followers","following_url":"https://api.github.com/users/johnhawkinson/following{/other_user}","gists_url":"https://api.github.com/users/johnhawkinson/gists{/gist_id}","starred_url":"https://api.github.com/users/johnhawkinson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/johnhawkinson/subscriptions","organizations_url":"https://api.github.com/users/johnhawkinson/orgs","repos_url":"https://api.github.com/users/johnhawkinson/repos","events_url":"https://api.github.com/users/johnhawkinson/events{/privacy}","received_events_url":"https://api.github.com/users/johnhawkinson/received_events","type":"User","site_admin":false},"body":"As it stands, its not going to break other websites. Without this patch, when an array is passed in, a type error exception gets thrown, I believe from L57 where it makes an array of an array.\r\n\r\nThat said, how would you suggest fixing it in `iqm2.py` without substantial code duplication?","created_at":"2016-10-10T09:36:46Z","updated_at":"2016-10-10T09:36:47Z","html_url":"https://github.com/rg3/youtube-dl/pull/10854#discussion_r82571265","pull_request_url":"https://api.github.com/repos/rg3/youtube-dl/pulls/10854","_links":{"self":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/comments/82571265"},"html":{"href":"https://github.com/rg3/youtube-dl/pull/10854#discussion_r82571265"},"pull_request":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/10854"}}},"pull_request":{"url":"https://api.github.com/repos/rg3/youtube-dl/pulls/10854","id":88202694,"html_url":"https://github.com/rg3/youtube-dl/pull/10854","diff_url":"https://github.com/rg3/youtube-dl/pull/10854.diff","patch_url":"https://github.com/rg3/youtube-dl/pull/10854.patch","issue_url":"https://api.github.com/repos/rg3/youtube-dl/issues/10854","number":10854,"state":"open","locked":false,"title":"[Iqm2] new extractor for Accela/IQM2","user":{"login":"johnhawkinson","id":1270718,"avatar_url":"https://avatars.githubusercontent.com/u/1270718?v=3","gravatar_id":"","url":"https://api.github.com/users/johnhawkinson","html_url":"https://github.com/johnhawkinson","followers_url":"https://api.github.com/users/johnhawkinson/followers","following_url":"https://api.github.com/users/johnhawkinson/following{/other_user}","gists_url":"https://api.github.com/users/johnhawkinson/gists{/gist_id}","starred_url":"https://api.github.com/users/johnhawkinson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/johnhawkinson/subscriptions","organizations_url":"https://api.github.com/users/johnhawkinson/orgs","repos_url":"https://api.github.com/users/johnhawkinson/repos","events_url":"https://api.github.com/users/johnhawkinson/events{/privacy}","received_events_url":"https://api.github.com/users/johnhawkinson/received_events","type":"User","site_admin":false},"body":"---\r\n\r\n### Before submitting a *pull request* make sure you have:\r\n- [x] At least skimmed through [adding new extractor tutorial](https://github.com/rg3/youtube-dl#adding-support-for-a-new-site) and [youtube-dl coding conventions](https://github.com/rg3/youtube-dl#youtube-dl-coding-conventions) sections\r\n- [x] [Searched](https://github.com/rg3/youtube-dl/search?q=is%3Apr&type=Issues) the bugtracker for similar pull requests\r\n\r\n### In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under [Unlicense](http://unlicense.org/). Check one of the following options:\r\n- [x] I am the original author of this code and I am willing to release it under [Unlicense](http://unlicense.org/)\r\n- [ ] I am not the original author of this code but it is in public domain or released under [Unlicense](http://unlicense.org/) (provide reliable evidence)\r\n\r\n### What is the purpose of your *pull request*?\r\n- [ ] Bug fix\r\n- [ ] Improvement\r\n- [x] New extractor\r\n- [ ] New feature\r\n\r\n---\r\n\r\n### Description of your *pull request* and other information\r\n\r\n\r\nNew extractor for .iqm2.com videos.\r\n\r\nIQM2 aka Accela is a municipal meeting management platform that\r\n(among other things) stores livestreamed video from municipal\r\nmeetings.  After a hefty (several-hour) processing time, that video\r\nis available in easily downloadable form from their web portal, but\r\nprior to that, the video can only be watched in realtime through\r\nJWPlayer. This extractor is designed to download the realtime video\r\nprior to download links being available. ","created_at":"2016-10-06T05:55:51Z","updated_at":"2016-10-10T09:36:47Z","closed_at":null,"merged_at":null,"merge_commit_sha":"a13721ce22e51c3bc819c5815dd87487e6944c8f","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/rg3/youtube-dl/pulls/10854/commits","review_comments_url":"https://api.github.com/repos/rg3/youtube-dl/pulls/10854/comments","review_comment_url":"https://api.github.com/repos/rg3/youtube-dl/pulls/comments{/number}","comments_url":"https://api.github.com/repos/rg3/youtube-dl/issues/10854/comments","statuses_url":"https://api.github.com/repos/rg3/youtube-dl/statuses/cb2e9ec69e135aefb4bb1c8f3e40bcddac3bfd7a","head":{"label":"johnhawkinson:iqm2","ref":"iqm2","sha":"cb2e9ec69e135aefb4bb1c8f3e40bcddac3bfd7a","user":{"login":"johnhawkinson","id":1270718,"avatar_url":"https://avatars.githubusercontent.com/u/1270718?v=3","gravatar_id":"","url":"https://api.github.com/users/johnhawkinson","html_url":"https://github.com/johnhawkinson","followers_url":"https://api.github.com/users/johnhawkinson/followers","following_url":"https://api.github.com/users/johnhawkinson/following{/other_user}","gists_url":"https://api.github.com/users/johnhawkinson/gists{/gist_id}","starred_url":"https://api.github.com/users/johnhawkinson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/johnhawkinson/subscriptions","organizations_url":"https://api.github.com/users/johnhawkinson/orgs","repos_url":"https://api.github.com/users/johnhawkinson/repos","events_url":"https://api.github.com/users/johnhawkinson/events{/privacy}","received_events_url":"https://api.github.com/users/johnhawkinson/received_events","type":"User","site_admin":false},"repo":{"id":70123841,"name":"youtube-dl","full_name":"johnhawkinson/youtube-dl","owner":{"login":"johnhawkinson","id":1270718,"avatar_url":"https://avatars.githubusercontent.com/u/1270718?v=3","gravatar_id":"","url":"https://api.github.com/users/johnhawkinson","html_url":"https://github.com/johnhawkinson","followers_url":"https://api.github.com/users/johnhawkinson/followers","following_url":"https://api.github.com/users/johnhawkinson/following{/other_user}","gists_url":"https://api.github.com/users/johnhawkinson/gists{/gist_id}","starred_url":"https://api.github.com/users/johnhawkinson/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/johnhawkinson/subscriptions","organizations_url":"https://api.github.com/users/johnhawkinson/orgs","repos_url":"https://api.github.com/users/johnhawkinson/repos","events_url":"https://api.github.com/users/johnhawkinson/events{/privacy}","received_events_url":"https://api.github.com/users/johnhawkinson/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/johnhawkinson/youtube-dl","description":"Command-line program to download videos from YouTube.com and other video sites","fork":true,"url":"https://api.github.com/repos/johnhawkinson/youtube-dl","forks_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/forks","keys_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/keys{/key_id}","collaborators_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/teams","hooks_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/hooks","issue_events_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/issues/events{/number}","events_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/events","assignees_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/assignees{/user}","branches_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/branches{/branch}","tags_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/tags","blobs_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/git/refs{/sha}","trees_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/git/trees{/sha}","statuses_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/statuses/{sha}","languages_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/languages","stargazers_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/stargazers","contributors_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/contributors","subscribers_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/subscribers","subscription_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/subscription","commits_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/commits{/sha}","git_commits_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/git/commits{/sha}","comments_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/comments{/number}","issue_comment_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/issues/comments{/number}","contents_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/contents/{+path}","compare_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/compare/{base}...{head}","merges_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/merges","archive_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/downloads","issues_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/issues{/number}","pulls_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/pulls{/number}","milestones_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/milestones{/number}","notifications_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/labels{/name}","releases_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/releases{/id}","deployments_url":"https://api.github.com/repos/johnhawkinson/youtube-dl/deployments","created_at":"2016-10-06T04:52:46Z","updated_at":"2016-10-06T04:53:13Z","pushed_at":"2016-10-10T06:41:15Z","git_url":"git://github.com/johnhawkinson/youtube-dl.git","ssh_url":"git@github.com:johnhawkinson/youtube-dl.git","clone_url":"https://github.com/johnhawkinson/youtube-dl.git","svn_url":"https://github.com/johnhawkinson/youtube-dl","homepage":"http://rg3.github.io/youtube-dl/","size":41966,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"rg3:master","ref":"master","sha":"017eb829343dfff9b70ab7f2278053f35cee953c","user":{"login":"rg3","id":53487,"avatar_url":"https://avatars.githubusercontent.com/u/53487?v=3","gravatar_id":"","url":"https://api.github.com/users/rg3","html_url":"https://github.com/rg3","followers_url":"https://api.github.com/users/rg3/followers","following_url":"https://api.github.com/users/rg3/following{/other_user}","gists_url":"https://api.github.com/users/rg3/gists{/gist_id}","starred_url":"https://api.github.com/users/rg3/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rg3/subscriptions","organizations_url":"https://api.github.com/users/rg3/orgs","repos_url":"https://api.github.com/users/rg3/repos","events_url":"https://api.github.com/users/rg3/events{/privacy}","received_events_url":"https://api.github.com/users/rg3/received_events","type":"User","site_admin":false},"repo":{"id":1039520,"name":"youtube-dl","full_name":"rg3/youtube-dl","owner":{"login":"rg3","id":53487,"avatar_url":"https://avatars.githubusercontent.com/u/53487?v=3","gravatar_id":"","url":"https://api.github.com/users/rg3","html_url":"https://github.com/rg3","followers_url":"https://api.github.com/users/rg3/followers","following_url":"https://api.github.com/users/rg3/following{/other_user}","gists_url":"https://api.github.com/users/rg3/gists{/gist_id}","starred_url":"https://api.github.com/users/rg3/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rg3/subscriptions","organizations_url":"https://api.github.com/users/rg3/orgs","repos_url":"https://api.github.com/users/rg3/repos","events_url":"https://api.github.com/users/rg3/events{/privacy}","received_events_url":"https://api.github.com/users/rg3/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/rg3/youtube-dl","description":"Command-line program to download videos from YouTube.com and other video sites","fork":false,"url":"https://api.github.com/repos/rg3/youtube-dl","forks_url":"https://api.github.com/repos/rg3/youtube-dl/forks","keys_url":"https://api.github.com/repos/rg3/youtube-dl/keys{/key_id}","collaborators_url":"https://api.github.com/repos/rg3/youtube-dl/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/rg3/youtube-dl/teams","hooks_url":"https://api.github.com/repos/rg3/youtube-dl/hooks","issue_events_url":"https://api.github.com/repos/rg3/youtube-dl/issues/events{/number}","events_url":"https://api.github.com/repos/rg3/youtube-dl/events","assignees_url":"https://api.github.com/repos/rg3/youtube-dl/assignees{/user}","branches_url":"https://api.github.com/repos/rg3/youtube-dl/branches{/branch}","tags_url":"https://api.github.com/repos/rg3/youtube-dl/tags","blobs_url":"https://api.github.com/repos/rg3/youtube-dl/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/rg3/youtube-dl/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/rg3/youtube-dl/git/refs{/sha}","trees_url":"https://api.github.com/repos/rg3/youtube-dl/git/trees{/sha}","statuses_url":"https://api.github.com/repos/rg3/youtube-dl/statuses/{sha}","languages_url":"https://api.github.com/repos/rg3/youtube-dl/languages","stargazers_url":"https://api.github.com/repos/rg3/youtube-dl/stargazers","contributors_url":"https://api.github.com/repos/rg3/youtube-dl/contributors","subscribers_url":"https://api.github.com/repos/rg3/youtube-dl/subscribers","subscription_url":"https://api.github.com/repos/rg3/youtube-dl/subscription","commits_url":"https://api.github.com/repos/rg3/youtube-dl/commits{/sha}","git_commits_url":"https://api.github.com/repos/rg3/youtube-dl/git/commits{/sha}","comments_url":"https://api.github.com/repos/rg3/youtube-dl/comments{/number}","issue_comment_url":"https://api.github.com/repos/rg3/youtube-dl/issues/comments{/number}","contents_url":"https://api.github.com/repos/rg3/youtube-dl/contents/{+path}","compare_url":"https://api.github.com/repos/rg3/youtube-dl/compare/{base}...{head}","merges_url":"https://api.github.com/repos/rg3/youtube-dl/merges","archive_url":"https://api.github.com/repos/rg3/youtube-dl/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/rg3/youtube-dl/downloads","issues_url":"https://api.github.com/repos/rg3/youtube-dl/issues{/number}","pulls_url":"https://api.github.com/repos/rg3/youtube-dl/pulls{/number}","milestones_url":"https://api.github.com/repos/rg3/youtube-dl/milestones{/number}","notifications_url":"https://api.github.com/repos/rg3/youtube-dl/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/rg3/youtube-dl/labels{/name}","releases_url":"https://api.github.com/repos/rg3/youtube-dl/releases{/id}","deployments_url":"https://api.github.com/repos/rg3/youtube-dl/deployments","created_at":"2010-10-31T14:35:07Z","updated_at":"2016-10-10T09:25:33Z","pushed_at":"2016-10-10T06:41:20Z","git_url":"git://github.com/rg3/youtube-dl.git","ssh_url":"git@github.com:rg3/youtube-dl.git","clone_url":"https://github.com/rg3/youtube-dl.git","svn_url":"https://github.com/rg3/youtube-dl","homepage":"http://rg3.github.io/youtube-dl/","size":127257,"stargazers_count":20683,"watchers_count":20683,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":3962,"mirror_url":null,"open_issues_count":1451,"forks":3962,"open_issues":1451,"watchers":20683,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/10854"},"html":{"href":"https://github.com/rg3/youtube-dl/pull/10854"},"issue":{"href":"https://api.github.com/repos/rg3/youtube-dl/issues/10854"},"comments":{"href":"https://api.github.com/repos/rg3/youtube-dl/issues/10854/comments"},"review_comments":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/10854/comments"},"review_comment":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/rg3/youtube-dl/pulls/10854/commits"},"statuses":{"href":"https://api.github.com/repos/rg3/youtube-dl/statuses/cb2e9ec69e135aefb4bb1c8f3e40bcddac3bfd7a"}}}}
{"actor":{"display_login":"johnhawkinson"}}