mapbox-maps-android icon indicating copy to clipboard operation
mapbox-maps-android copied to clipboard

Custom MapView does not support onRestoreInstanceState

Open Copatych opened this issue 1 year ago • 1 comments

Environment

  • Android OS version: 11
  • Devices affected: Samsung Galaxy A50
  • Maps SDK Version: 11.1.0

Also tested on emulator with api 33

Observed behavior and steps to reproduce

  • Create custom MapView.
  • Override onSaveInstanceState and onRestoreInstanceState.
  • Create navigation from fragment with custom MapView to another fragment.
  • Navigate to another fragment and return to fragment with MapView.

We get this stacktrace:

java.lang.ClassCastException: android.view.AbsSavedState$1 cannot be cast to com.example.mapboxbugs.CustomMapView$SavedState
at com.example.mapboxbugs.CustomMapView.onRestoreInstanceState(CustomMapView.kt:29)
at android.view.View.dispatchRestoreInstanceState(View.java:21559)
at android.view.ViewGroup.dispatchRestoreInstanceState(ViewGroup.java:4001)
at android.view.ViewGroup.dispatchRestoreInstanceState(ViewGroup.java:4007)
at android.view.View.restoreHierarchyState(View.java:21537)
at androidx.fragment.app.Fragment.restoreViewState(Fragment.java:698)
at androidx.fragment.app.Fragment.restoreViewState(Fragment.java:3176)
at androidx.fragment.app.Fragment.performActivityCreated(Fragment.java:3161)
at androidx.fragment.app.FragmentStateManager.activityCreated(FragmentStateManager.java:619)
at androidx.fragment.app.FragmentStateManager.moveToExpectedState(FragmentStateManager.java:275)
at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:1934)
at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(FragmentManager.java:1845)
at androidx.fragment.app.FragmentManager.execPendingActions(FragmentManager.java:1782)
at androidx.fragment.app.FragmentManager$5.run(FragmentManager.java:565)
at android.os.Handler.handleCallback(Handler.java:942)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:201)
at android.os.Looper.loop(Looper.java:288)
at android.app.ActivityThread.main(ActivityThread.java:7872)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:936)

example code https://github.com/Copatych/MapboxBugs/tree/customMapView_onRestoreInstanceState

Expected behavior

Custom MapView supports restoring InstanceState.

Notes / preliminary analysis

Additional links and references

example code https://github.com/Copatych/MapboxBugs/tree/customMapView_onRestoreInstanceState

Copatych avatar Feb 16 '24 09:02 Copatych

This code worked with mapbox version 9

Copatych avatar Feb 16 '24 09:02 Copatych