{"comment":{"url":"https://api.github.com/repos/kaltura/mwEmbed/comments/9241960","html_url":"https://github.com/kaltura/mwEmbed/commit/fec6364a564d2ab46ad2009544a7a3dc2e15e396#commitcomment-9241960","id":9241960,"user":{"login":"mdale","id":96058,"avatar_url":"https://avatars.githubusercontent.com/u/96058?v=3","gravatar_id":"","url":"https://api.github.com/users/mdale","html_url":"https://github.com/mdale","followers_url":"https://api.github.com/users/mdale/followers","following_url":"https://api.github.com/users/mdale/following{/other_user}","gists_url":"https://api.github.com/users/mdale/gists{/gist_id}","starred_url":"https://api.github.com/users/mdale/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mdale/subscriptions","organizations_url":"https://api.github.com/users/mdale/orgs","repos_url":"https://api.github.com/users/mdale/repos","events_url":"https://api.github.com/users/mdale/events{/privacy}","received_events_url":"https://api.github.com/users/mdale/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"fec6364a564d2ab46ad2009544a7a3dc2e15e396","created_at":"2015-01-12T14:27:13Z","updated_at":"2015-01-12T14:27:13Z","body":"why do we want to allow run-time override parameter for handle player errors display? ... Would be cleaner to break this functionality out into a \"displayPlayerError\"; this makes things easy to follow, instad of having to look up what ,true means.\r\n\r\nAs a general rule avoid appending parameters to change actions for functions."}}