{"action":"created","issue":{"url":"https://api.github.com/repos/mapbox/mapbox-navigation-android/issues/569","repository_url":"https://api.github.com/repos/mapbox/mapbox-navigation-android","labels_url":"https://api.github.com/repos/mapbox/mapbox-navigation-android/issues/569/labels{/name}","comments_url":"https://api.github.com/repos/mapbox/mapbox-navigation-android/issues/569/comments","events_url":"https://api.github.com/repos/mapbox/mapbox-navigation-android/issues/569/events","html_url":"https://github.com/mapbox/mapbox-navigation-android/pull/569","id":279702413,"number":569,"title":"Fixing NullPointerException when navigationPresenter is not yet initialized","user":{"login":"kitttn","id":29702926,"avatar_url":"https://avatars2.githubusercontent.com/u/29702926?v=4","gravatar_id":"","url":"https://api.github.com/users/kitttn","html_url":"https://github.com/kitttn","followers_url":"https://api.github.com/users/kitttn/followers","following_url":"https://api.github.com/users/kitttn/following{/other_user}","gists_url":"https://api.github.com/users/kitttn/gists{/gist_id}","starred_url":"https://api.github.com/users/kitttn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kitttn/subscriptions","organizations_url":"https://api.github.com/users/kitttn/orgs","repos_url":"https://api.github.com/users/kitttn/repos","events_url":"https://api.github.com/users/kitttn/events{/privacy}","received_events_url":"https://api.github.com/users/kitttn/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-12-06T10:33:19Z","updated_at":"2017-12-06T14:54:08Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/mapbox/mapbox-navigation-android/pulls/569","html_url":"https://github.com/mapbox/mapbox-navigation-android/pull/569","diff_url":"https://github.com/mapbox/mapbox-navigation-android/pull/569.diff","patch_url":"https://github.com/mapbox/mapbox-navigation-android/pull/569.patch"},"body":"Issue type: Bug\r\nAffected devices: All tested\r\n\r\nWhen trying to scroll map which is not being loaded immediately, application crashes, because navigationPresenter is not initialized at that moment. "},"comment":{"url":"https://api.github.com/repos/mapbox/mapbox-navigation-android/issues/comments/349663084","html_url":"https://github.com/mapbox/mapbox-navigation-android/pull/569#issuecomment-349663084","issue_url":"https://api.github.com/repos/mapbox/mapbox-navigation-android/issues/569","id":349663084,"user":{"login":"danesfeder","id":8434572,"avatar_url":"https://avatars0.githubusercontent.com/u/8434572?v=4","gravatar_id":"","url":"https://api.github.com/users/danesfeder","html_url":"https://github.com/danesfeder","followers_url":"https://api.github.com/users/danesfeder/followers","following_url":"https://api.github.com/users/danesfeder/following{/other_user}","gists_url":"https://api.github.com/users/danesfeder/gists{/gist_id}","starred_url":"https://api.github.com/users/danesfeder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danesfeder/subscriptions","organizations_url":"https://api.github.com/users/danesfeder/orgs","repos_url":"https://api.github.com/users/danesfeder/repos","events_url":"https://api.github.com/users/danesfeder/events{/privacy}","received_events_url":"https://api.github.com/users/danesfeder/received_events","type":"User","site_admin":false},"created_at":"2017-12-06T14:54:07Z","updated_at":"2017-12-06T14:54:07Z","author_association":"CONTRIBUTOR","body":"Hey @kitttn thanks for the contribution!  I ran into this same issue previously and thought I resolved it by not adding the adding the map scroll listener until after the presenter is initialized:\r\n\r\nhttps://github.com/kitttn/mapbox-navigation-android/blob/40760ac0c31879e9450b8aa067b7655c96da1fd9/libandroid-navigation-ui/src/main/java/com/mapbox/services/android/navigation/ui/v5/NavigationView.java#L182-L185\r\n\r\n^ All of this is being done after the style has loaded \r\n\r\nI will circle back though to re-test and if needed let's add this null check - thanks again! "}}
{"actor":{"display_login":"danesfeder"}}