{"action":"created","issue":{"url":"https://api.github.com/repos/apache/cordova-plugin-media/issues/109","repository_url":"https://api.github.com/repos/apache/cordova-plugin-media","labels_url":"https://api.github.com/repos/apache/cordova-plugin-media/issues/109/labels{/name}","comments_url":"https://api.github.com/repos/apache/cordova-plugin-media/issues/109/comments","events_url":"https://api.github.com/repos/apache/cordova-plugin-media/issues/109/events","html_url":"https://github.com/apache/cordova-plugin-media/pull/109","id":172715632,"number":109,"title":"CB-11086 (Android) Fixed a crash when setVolume() is called","user":{"login":"alsorokin","id":10210197,"avatar_url":"https://avatars.githubusercontent.com/u/10210197?v=3","gravatar_id":"","url":"https://api.github.com/users/alsorokin","html_url":"https://github.com/alsorokin","followers_url":"https://api.github.com/users/alsorokin/followers","following_url":"https://api.github.com/users/alsorokin/following{/other_user}","gists_url":"https://api.github.com/users/alsorokin/gists{/gist_id}","starred_url":"https://api.github.com/users/alsorokin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/alsorokin/subscriptions","organizations_url":"https://api.github.com/users/alsorokin/orgs","repos_url":"https://api.github.com/users/alsorokin/repos","events_url":"https://api.github.com/users/alsorokin/events{/privacy}","received_events_url":"https://api.github.com/users/alsorokin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-08-23T14:33:49Z","updated_at":"2016-08-24T09:44:22Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/cordova-plugin-media/pulls/109","html_url":"https://github.com/apache/cordova-plugin-media/pull/109","diff_url":"https://github.com/apache/cordova-plugin-media/pull/109.diff","patch_url":"https://github.com/apache/cordova-plugin-media/pull/109.patch"},"body":"### Platforms affected\r\nAndroid\r\n\r\n### What does this PR do?\r\nThis is a fix for https://issues.apache.org/jira/browse/CB-11086\r\nIt fixes the crash by checking if player is null when trying to set player's volume.\r\n\r\n### What testing has been done on this change?\r\nTested on Android emulator 4.4, also checked that added test spec is green on iOS and Windows.\r\n\r\n### Checklist\r\n- [x] [ICLA](http://www.apache.org/licenses/icla.txt) has been signed and submitted to secretary@apache.org.\r\n- [x] [Reported an issue](http://cordova.apache.org/contribute/issues.html) in the JIRA database\r\n- [x] Commit message follows the format: \"CB-3232: (android) Fix bug with resolving file paths\", where CB-xxxx is the JIRA ID & \"android\" is the platform affected.\r\n- [x] Added automated test coverage as appropriate for this change.\r\n"},"comment":{"url":"https://api.github.com/repos/apache/cordova-plugin-media/issues/comments/242010686","html_url":"https://github.com/apache/cordova-plugin-media/pull/109#issuecomment-242010686","issue_url":"https://api.github.com/repos/apache/cordova-plugin-media/issues/109","id":242010686,"user":{"login":"cordova-qa","id":19274979,"avatar_url":"https://avatars.githubusercontent.com/u/19274979?v=3","gravatar_id":"","url":"https://api.github.com/users/cordova-qa","html_url":"https://github.com/cordova-qa","followers_url":"https://api.github.com/users/cordova-qa/followers","following_url":"https://api.github.com/users/cordova-qa/following{/other_user}","gists_url":"https://api.github.com/users/cordova-qa/gists{/gist_id}","starred_url":"https://api.github.com/users/cordova-qa/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cordova-qa/subscriptions","organizations_url":"https://api.github.com/users/cordova-qa/orgs","repos_url":"https://api.github.com/users/cordova-qa/repos","events_url":"https://api.github.com/users/cordova-qa/events{/privacy}","received_events_url":"https://api.github.com/users/cordova-qa/received_events","type":"User","site_admin":false},"created_at":"2016-08-24T09:44:22Z","updated_at":"2016-08-24T09:44:22Z","body":"Cordova CI Build has completed successfully.\n\n**Commit**     - [Link](https://github.com/apache/cordova-plugin-media/pull/109/commits/67166333ebbbc1b8ac27e6825a7d73be30cea255)\n**Dashboard** - [Link](http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65/)\n\n| Builder Name  | Console Output | Test Report | Device Logs  |\n|     :---:     |     :---:      |   :---:     |     :---:    |\n| [Windows 8.1 Store]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-8.1-store/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-8.1-store/console) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-8.1-store/testReport/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-8.1-store/artifact/) |\n| [Windows 10  Store]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-10-store/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-10-store/console) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-10-store/testReport/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-10-store/artifact/) |\n| [Windows 8.1 Phone]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-8.1-phone/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-8.1-phone/console) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-8.1-phone/testReport/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=windows-8.1-phone/artifact/) |\n| [iOS]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=ios/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=ios/console) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=ios/testReport/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=ios/artifact/) |\n| [Android]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=android/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=android/console) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=android/testReport/) | [Link]( http://cordova-ci.cloudapp.net:8080/job/cordova-plugin-media-pr/65//PLATFORM=android/artifact/) |\n \n"}}
{"actor":{"display_login":"cordova-qa"}}