netmackan / ATimeTracker

Open Source Time Tracker for Android
GNU General Public License v2.0
120 stars 56 forks source link

State loss on double screen rotation #147

Open davirec opened 2 years ago

davirec commented 2 years ago

Hi, this issue has relation with #146

A data loss problem occurs when data is accidentally deleted or state variables are accidentally assigned with default or initial values. This issue is related to the activity's lifecycle (1). When a screen rotation occurs, the activity is destroyed and resumed. Data loss will happen if the developer does not save the variables before the destruction and restore them after creation. To avoid this, the developers have to implement both the logic necessary to save the activity state in the onSaveInstanceState() callback method and the logic to resume its state in the onRestoreInstanceState() callback method. Another way to avoid data loss problems is to use the view model, designed to store and manage UI-related data in a lifecycle-conscious way (2).

We found failures similar to issue #146 in activities:

Before rotation and After rotation
Before rotation and After rotation
Before rotation and After rotation
Before rotation and After rotation
Before rotation and After rotation

Note:

  1. https://developer.android.com/guide/components/activities/activity-lifecycle
  2. https://developer.android.com/topic/libraries/architecture/viewmodel
netmackan commented 2 years ago

Thanks for the report.

Possible duplicate of one or more of #57 #55 #52 #51 #59 #49 #48 #47 #46 #44 #43.

I modified the title as "data loss" would sound like persisted data would be lost (i.e. in this case the saved time ranges that matters for this app) but as I understand it the data that is lost here is the message in the dialog box or currently selected values before applying them, which while irritating and certainly are bugs, are much less severe then loosing any data with business value.

We should look into the links provided with explanation about how to maintain the state variables.