{"action":"created","issue":{"url":"https://api.github.com/repos/mozilla/pdf.js/issues/7237","repository_url":"https://api.github.com/repos/mozilla/pdf.js","labels_url":"https://api.github.com/repos/mozilla/pdf.js/issues/7237/labels{/name}","comments_url":"https://api.github.com/repos/mozilla/pdf.js/issues/7237/comments","events_url":"https://api.github.com/repos/mozilla/pdf.js/issues/7237/events","html_url":"https://github.com/mozilla/pdf.js/pull/7237","id":150301458,"number":7237,"title":"Prevent accidentally overriding the `error` function in the `commonobj` messageHandler in api.js (issue 7232)","user":{"login":"Snuffleupagus","id":2692120,"avatar_url":"https://avatars.githubusercontent.com/u/2692120?v=3","gravatar_id":"","url":"https://api.github.com/users/Snuffleupagus","html_url":"https://github.com/Snuffleupagus","followers_url":"https://api.github.com/users/Snuffleupagus/followers","following_url":"https://api.github.com/users/Snuffleupagus/following{/other_user}","gists_url":"https://api.github.com/users/Snuffleupagus/gists{/gist_id}","starred_url":"https://api.github.com/users/Snuffleupagus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Snuffleupagus/subscriptions","organizations_url":"https://api.github.com/users/Snuffleupagus/orgs","repos_url":"https://api.github.com/users/Snuffleupagus/repos","events_url":"https://api.github.com/users/Snuffleupagus/events{/privacy}","received_events_url":"https://api.github.com/users/Snuffleupagus/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/mozilla/pdf.js/labels/1-core","name":"1-core","color":"DDDDDD"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-04-22T08:13:17Z","updated_at":"2016-04-22T08:38:03Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mozilla/pdf.js/pulls/7237","html_url":"https://github.com/mozilla/pdf.js/pull/7237","diff_url":"https://github.com/mozilla/pdf.js/pull/7237.diff","patch_url":"https://github.com/mozilla/pdf.js/pull/7237.patch"},"body":"This naming issue has been present since PR #3529, but at least I cannot find any issues/bugs that seem to have been caused by it, which is good.\r\n\r\nThe patch also removes an unnecessary `else` branch, since an already existing `break` means that it's redundant.\r\n\r\nFixes #7232.\r\n\r\n*Perhaps somewhat easier reviewing with https://github.com/mozilla/pdf.js/pull/7237/files?w=1.*"},"comment":{"url":"https://api.github.com/repos/mozilla/pdf.js/issues/comments/213326459","html_url":"https://github.com/mozilla/pdf.js/pull/7237#issuecomment-213326459","issue_url":"https://api.github.com/repos/mozilla/pdf.js/issues/7237","id":213326459,"user":{"login":"pdfjsbot","id":1036361,"avatar_url":"https://avatars.githubusercontent.com/u/1036361?v=3","gravatar_id":"","url":"https://api.github.com/users/pdfjsbot","html_url":"https://github.com/pdfjsbot","followers_url":"https://api.github.com/users/pdfjsbot/followers","following_url":"https://api.github.com/users/pdfjsbot/following{/other_user}","gists_url":"https://api.github.com/users/pdfjsbot/gists{/gist_id}","starred_url":"https://api.github.com/users/pdfjsbot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pdfjsbot/subscriptions","organizations_url":"https://api.github.com/users/pdfjsbot/orgs","repos_url":"https://api.github.com/users/pdfjsbot/repos","events_url":"https://api.github.com/users/pdfjsbot/events{/privacy}","received_events_url":"https://api.github.com/users/pdfjsbot/received_events","type":"User","site_admin":false},"created_at":"2016-04-22T08:38:02Z","updated_at":"2016-04-22T08:38:02Z","body":"#### From: Bot.io (Windows)\n\n----\n\n# Success\n\nFull output at http://107.22.172.223:8877/ef0cb5cf72bd776/output.txt\n\nTotal script time: 20.27 mins\n\n+ **Font tests:** Passed\n+ **Unit tests:** Passed\n+ **Regression tests:** Passed"}}