Remove cache version handling from provider

This now happens as a database migration

Author: rolandgeider

lib/database/exercises/exercise_database.dart

@@ -73,6 +73,7 @@ class ExerciseDatabase extends _$ExerciseDatabase {
   // Named constructor for creating in-memory database
   ExerciseDatabase.inMemory(super.e);
 
+  /// Note that this needs to be bumped if the JSON response from the server changes
   @override
   int get schemaVersion => 1;
 

lib/database/ingredients/ingredients_database.dart

@@ -26,6 +26,7 @@ class IngredientDatabase extends _$IngredientDatabase {
   // Named constructor for creating in-memory database
   IngredientDatabase.inMemory(super.e);
 
+  /// Note that this needs to be bumped if the JSON response from the server changes
   @override
   int get schemaVersion => 2;
 

lib/providers/exercises.dart

@@ -362,41 +362,9 @@ class ExercisesProvider with ChangeNotifier {
     return exercise;
   }
 
-  /// Checks the required cache version
-  ///
-  /// This is needed since the content of the exercise cache (the API response)
-  /// can change and we need to invalidate it as a result
-  Future<void> checkExerciseCacheVersion() async {
-    final prefs = await SharedPreferences.getInstance();
-    if (prefs.containsKey(PREFS_EXERCISE_CACHE_VERSION)) {
-      final cacheVersion = prefs.getInt(PREFS_EXERCISE_CACHE_VERSION)!;
-
-      // Cache has has a different version, reset
-      if (cacheVersion != CACHE_VERSION) {
-        database.delete(database.exercises).go();
-      }
-      await prefs.setInt(PREFS_EXERCISE_CACHE_VERSION, CACHE_VERSION);
-
-      // Cache has no version key, reset
-      // Note: this is only needed for very old apps that update and could probably
-      // be just removed in the future
-    } else {
-      database.delete(database.exercises).go();
-      await prefs.setInt(PREFS_EXERCISE_CACHE_VERSION, CACHE_VERSION);
-    }
-  }
-
   Future<void> initCacheTimesLocalPrefs({forceInit = false}) async {
     final prefs = await SharedPreferences.getInstance();
 
-    // TODO: The exercise data was previously saved in PREFS_EXERCISES. This
-    //       can now be deleted. After some time when we can be sure all users
-    //       have updated their app, we can also remove this line and the
-    //       PREFS_EXERCISES constant
-    if (prefs.containsKey(PREFS_EXERCISES)) {
-      prefs.remove(PREFS_EXERCISES);
-    }
-
     final initDate = DateTime(2023, 1, 1).toIso8601String();
 
     if (forceInit || !prefs.containsKey(PREFS_LAST_UPDATED_MUSCLES)) {
@@ -429,7 +397,6 @@ class ExercisesProvider with ChangeNotifier {
     clear();
 
     await initCacheTimesLocalPrefs();
-    await checkExerciseCacheVersion();
 
     // Load categories, muscles, equipment and languages
     await Future.wait([