{"action":"created","issue":{"url":"https://api.github.com/repos/osmdroid/osmdroid/issues/1040","repository_url":"https://api.github.com/repos/osmdroid/osmdroid","labels_url":"https://api.github.com/repos/osmdroid/osmdroid/issues/1040/labels{/name}","comments_url":"https://api.github.com/repos/osmdroid/osmdroid/issues/1040/comments","events_url":"https://api.github.com/repos/osmdroid/osmdroid/issues/1040/events","html_url":"https://github.com/osmdroid/osmdroid/issues/1040","id":325879583,"node_id":"MDU6SXNzdWUzMjU4Nzk1ODM=","number":1040,"title":"6.0.2 issue in controller.animateTo()","user":{"login":"InI4","id":4115837,"node_id":"MDQ6VXNlcjQxMTU4Mzc=","avatar_url":"https://avatars1.githubusercontent.com/u/4115837?v=4","gravatar_id":"","url":"https://api.github.com/users/InI4","html_url":"https://github.com/InI4","followers_url":"https://api.github.com/users/InI4/followers","following_url":"https://api.github.com/users/InI4/following{/other_user}","gists_url":"https://api.github.com/users/InI4/gists{/gist_id}","starred_url":"https://api.github.com/users/InI4/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/InI4/subscriptions","organizations_url":"https://api.github.com/users/InI4/orgs","repos_url":"https://api.github.com/users/InI4/repos","events_url":"https://api.github.com/users/InI4/events{/privacy}","received_events_url":"https://api.github.com/users/InI4/received_events","type":"User","site_admin":false},"labels":[{"id":84507107,"node_id":"MDU6TGFiZWw4NDUwNzEwNw==","url":"https://api.github.com/repos/osmdroid/osmdroid/labels/bug","name":"bug","color":"fc2929","default":true}],"state":"open","locked":false,"assignee":{"login":"monsieurtanuki","id":11576431,"node_id":"MDQ6VXNlcjExNTc2NDMx","avatar_url":"https://avatars0.githubusercontent.com/u/11576431?v=4","gravatar_id":"","url":"https://api.github.com/users/monsieurtanuki","html_url":"https://github.com/monsieurtanuki","followers_url":"https://api.github.com/users/monsieurtanuki/followers","following_url":"https://api.github.com/users/monsieurtanuki/following{/other_user}","gists_url":"https://api.github.com/users/monsieurtanuki/gists{/gist_id}","starred_url":"https://api.github.com/users/monsieurtanuki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/monsieurtanuki/subscriptions","organizations_url":"https://api.github.com/users/monsieurtanuki/orgs","repos_url":"https://api.github.com/users/monsieurtanuki/repos","events_url":"https://api.github.com/users/monsieurtanuki/events{/privacy}","received_events_url":"https://api.github.com/users/monsieurtanuki/received_events","type":"User","site_admin":false},"assignees":[{"login":"monsieurtanuki","id":11576431,"node_id":"MDQ6VXNlcjExNTc2NDMx","avatar_url":"https://avatars0.githubusercontent.com/u/11576431?v=4","gravatar_id":"","url":"https://api.github.com/users/monsieurtanuki","html_url":"https://github.com/monsieurtanuki","followers_url":"https://api.github.com/users/monsieurtanuki/followers","following_url":"https://api.github.com/users/monsieurtanuki/following{/other_user}","gists_url":"https://api.github.com/users/monsieurtanuki/gists{/gist_id}","starred_url":"https://api.github.com/users/monsieurtanuki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/monsieurtanuki/subscriptions","organizations_url":"https://api.github.com/users/monsieurtanuki/orgs","repos_url":"https://api.github.com/users/monsieurtanuki/repos","events_url":"https://api.github.com/users/monsieurtanuki/events{/privacy}","received_events_url":"https://api.github.com/users/monsieurtanuki/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/osmdroid/osmdroid/milestones/13","html_url":"https://github.com/osmdroid/osmdroid/milestone/13","labels_url":"https://api.github.com/repos/osmdroid/osmdroid/milestones/13/labels","id":3134927,"node_id":"MDk6TWlsZXN0b25lMzEzNDkyNw==","number":13,"title":"v6.0.2","description":null,"creator":{"login":"spyhunter99","id":1975321,"node_id":"MDQ6VXNlcjE5NzUzMjE=","avatar_url":"https://avatars2.githubusercontent.com/u/1975321?v=4","gravatar_id":"","url":"https://api.github.com/users/spyhunter99","html_url":"https://github.com/spyhunter99","followers_url":"https://api.github.com/users/spyhunter99/followers","following_url":"https://api.github.com/users/spyhunter99/following{/other_user}","gists_url":"https://api.github.com/users/spyhunter99/gists{/gist_id}","starred_url":"https://api.github.com/users/spyhunter99/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/spyhunter99/subscriptions","organizations_url":"https://api.github.com/users/spyhunter99/orgs","repos_url":"https://api.github.com/users/spyhunter99/repos","events_url":"https://api.github.com/users/spyhunter99/events{/privacy}","received_events_url":"https://api.github.com/users/spyhunter99/received_events","type":"User","site_admin":false},"open_issues":4,"closed_issues":20,"state":"open","created_at":"2018-02-22T00:52:07Z","updated_at":"2018-07-08T20:07:34Z","due_on":null,"closed_at":null},"comments":27,"created_at":"2018-05-23T21:26:36Z","updated_at":"2018-07-09T05:03:47Z","closed_at":null,"author_association":"NONE","body":"## Issue Type\r\n\r\n[X ] Question\r\n[X ] Bug\r\n\r\n## Description and/or steps/code to reproduce the problem\r\n\r\nI have a self written overlay whichs `draw()` method line 1 is an `if ( shadow || map.isAnimating() ) return;`. \r\n\r\nNow when I send a `controller.animateTo()`, my overlay painted with v6.0.1 after animation target was reached. However with v.6.0.2-SNAPSHOT from right now the `draw()` method does not seem to get triggered after animation has finished. Consequently the overlay is not being drawn at all. Minimally touching and moving the map though triggers a correct `overlay.draw()`.\r\n\r\nI see no way to achieve a overlay draw after animation now.\r\n\r\nUsing `controller.setCenter()` instead of `animateTo()` works fine, so problem seems to come from the animation.\r\n\r\n## Environment\r\n\r\nFound this behavior change on all 3 tested emulators.\r\n\r\n### If it's a bug, version(s) of android this affects:\r\n\r\nfully reproducible on 7.1.1, 8.1\r\nvery often reproducible on 6.0\r\n\r\n### Version of osmdroid the issue relates to:\r\n\r\n6.0.2-SNAPSHOT\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/osmdroid/osmdroid/issues/comments/403360628","html_url":"https://github.com/osmdroid/osmdroid/issues/1040#issuecomment-403360628","issue_url":"https://api.github.com/repos/osmdroid/osmdroid/issues/1040","id":403360628,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMzM2MDYyOA==","user":{"login":"InI4","id":4115837,"node_id":"MDQ6VXNlcjQxMTU4Mzc=","avatar_url":"https://avatars1.githubusercontent.com/u/4115837?v=4","gravatar_id":"","url":"https://api.github.com/users/InI4","html_url":"https://github.com/InI4","followers_url":"https://api.github.com/users/InI4/followers","following_url":"https://api.github.com/users/InI4/following{/other_user}","gists_url":"https://api.github.com/users/InI4/gists{/gist_id}","starred_url":"https://api.github.com/users/InI4/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/InI4/subscriptions","organizations_url":"https://api.github.com/users/InI4/orgs","repos_url":"https://api.github.com/users/InI4/repos","events_url":"https://api.github.com/users/InI4/events{/privacy}","received_events_url":"https://api.github.com/users/InI4/received_events","type":"User","site_admin":false},"created_at":"2018-07-09T05:03:47Z","updated_at":"2018-07-09T05:03:47Z","author_association":"NONE","body":"MapListener events are somewhat painful anyway and there is a lot of code floating around to get rid of most of those events (`DelayedMapListener`s) anyway. If it works out, wouldn't doing events in `onAnimationUpdate()`, probably zoom and scroll close to each other, flood clients even more?\r\n\r\nJust to reiterate another possible minor variant. Reactivate the old implementations of `animateTo` and `zoom` (instead of calling the new combined one). Deprecate `IMapController`, `MapListener`, `MapEvent`, `ScrollEvent` and `ZoomEvent`. Recommend `Overlay` to follow map changes and document the new combined `animateTo()` to fire no MapListener events.\r\n\r\n**Pro:** Breaks no legacy clients. **Contra:** Reactivates old code which was nicely cleaned away.\r\n\r\n"}}
{"actor":{"display_login":"InI4"}}