Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/#646 bis #735

Merged
merged 2 commits into from
Oct 14, 2017
Merged

Conversation

monsieurtanuki
Copy link
Collaborator

No description provided.

Impacted classes:
* `MapTileApproximater`: cleaner method `detach`
* `MapTileProviderBasic`: method `isDowngradedMode` now takes into account `useDataConnection`
Impacted classes:
* `MapTileApproximater`: cleaner method `detach`
* `MapTileProviderBasic`: method `isDowngradedMode` now takes into account `useDataConnection`
@monsieurtanuki
Copy link
Collaborator Author

@secaw Sorry for the very late #646 answer: I got drowned into #329 and #725.
I added your main suggestions - the rest seemed a bit too specific.

@secaw
Copy link
Contributor

secaw commented Oct 13, 2017

Well, glad, I could contribute just a tiny bit :-). Great more work by the way, thanks!

@@ -127,6 +127,7 @@ public void detach(){
*/
@Override
protected boolean isDowngradedMode() {
return mNetworkAvailabilityCheck != null && !mNetworkAvailabilityCheck.getNetworkAvailable();
return (mNetworkAvailabilityCheck != null && !mNetworkAvailabilityCheck.getNetworkAvailable())
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this change somehow related to #646?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed: we discussed that @secaw and I, and you can find more details at the end of #646.

@spyhunter99 spyhunter99 merged commit 9e143e8 into osmdroid:master Oct 14, 2017
@monsieurtanuki monsieurtanuki deleted the feature/#646-bis branch January 4, 2018 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants