feat: allow control over patches update (#1063)

Co-authored-by: oSumAtrIX <johan.melkonyan1@web.de>

lib/services/github_api.dart

@@ -72,6 +72,39 @@ class GithubAPI {
     }
   }
 
+  Future<Map<String, dynamic>?> getPatchesRelease(
+    String repoName,
+    String version,
+  ) async {
+    try {
+      final response = await _dio.get(
+        '/repos/$repoName/releases/tags/$version',
+      );
+      return response.data;
+    } on Exception catch (e) {
+      if (kDebugMode) {
+        print(e);
+      }
+      return null;
+    }
+  }
+
+  Future<Map<String, dynamic>?> getLatestPatchesRelease(
+    String repoName,
+  ) async {
+    try {
+      final response = await _dio.get(
+        '/repos/$repoName/releases/latest',
+      );
+      return response.data;
+    } on Exception catch (e) {
+      if (kDebugMode) {
+        print(e);
+      }
+      return null;
+    }
+  }
+
   Future<Map<String, dynamic>?> getLatestManagerRelease(
     String repoName,
   ) async {
@@ -164,10 +197,37 @@ class GithubAPI {
     return null;
   }
 
-  Future<List<Patch>> getPatches(String repoName) async {
+  Future<File?> getPatchesReleaseFile(
+    String extension,
+    String repoName,
+    String version,
+  ) async {
+    try {
+      final Map<String, dynamic>? release =
+          await getPatchesRelease(repoName, version);
+      if (release != null) {
+        final Map<String, dynamic>? asset =
+            (release['assets'] as List<dynamic>).firstWhereOrNull(
+          (asset) => (asset['name'] as String).endsWith(extension),
+        );
+        if (asset != null) {
+          return await DefaultCacheManager().getSingleFile(
+            asset['browser_download_url'],
+          );
+        }
+      }
+    } on Exception catch (e) {
+      if (kDebugMode) {
+        print(e);
+      }
+    }
+    return null;
+  }
+
+  Future<List<Patch>> getPatches(String repoName, String version) async {
     List<Patch> patches = [];
     try {
-      final File? f = await getLatestReleaseFile('.json', repoName);
+      final File? f = await getPatchesReleaseFile('.json', repoName, version);
       if (f != null) {
         final List<dynamic> list = jsonDecode(f.readAsStringSync());
         patches = list.map((patch) => Patch.fromJson(patch)).toList();
@@ -180,21 +240,4 @@ class GithubAPI {
 
     return patches;
   }
-
-  Future<String> getLastestReleaseVersion(String repoName) async {
-    try {
-      final Map<String, dynamic>? release = await getLatestRelease(repoName);
-      if (release != null) {
-        return release['tag_name'];
-      } else {
-        return 'Unknown';
-      }
-    } on Exception catch (e) {
-      if (kDebugMode) {
-        print(e);
-      }
-
-      return 'Unknown';
-    }
-  }
 }

lib/services/manager_api.dart

@@ -13,6 +13,7 @@ import 'package:revanced_manager/services/revanced_api.dart';
 import 'package:revanced_manager/services/root_api.dart';
 import 'package:revanced_manager/utils/check_for_supported_patch.dart';
 import 'package:shared_preferences/shared_preferences.dart';
+import 'package:timeago/timeago.dart';
 
 @lazySingleton
 class ManagerAPI {
@@ -81,6 +82,22 @@ class ManagerAPI {
     await _prefs.setString('patchesRepo', value);
   }
 
+  bool getPatchesConsent() {
+    return _prefs.getBool('patchesConsent') ?? false;
+  }
+
+  Future<void> setPatchesConsent(bool consent) async {
+    await _prefs.setBool('patchesConsent', consent);
+  }
+
+  bool isPatchesAutoUpdate() {
+    return _prefs.getBool('patchesAutoUpdate') ?? false;
+  }
+
+  Future<void> setPatchesAutoUpdate(bool value) async {
+    await _prefs.setBool('patchesAutoUpdate', value);
+  }
+
   String getIntegrationsRepo() {
     return _prefs.getString('integrationsRepo') ?? defaultIntegrationsRepo;
   }
@@ -205,11 +222,8 @@ class ManagerAPI {
   Future<List<Patch>> getPatches() async {
     try {
       final String repoName = getPatchesRepo();
-      if (repoName == defaultPatchesRepo) {
-        return await _revancedAPI.getPatches();
-      } else {
-        return await _githubAPI.getPatches(repoName);
-      }
+      final String currentVersion = await getCurrentPatchesVersion();
+      return await _githubAPI.getPatches(repoName, currentVersion);
     } on Exception catch (e) {
       if (kDebugMode) {
         print(e);
@@ -221,14 +235,12 @@ class ManagerAPI {
   Future<File?> downloadPatches() async {
     try {
       final String repoName = getPatchesRepo();
-      if (repoName == defaultPatchesRepo) {
-        return await _revancedAPI.getLatestReleaseFile(
-          '.jar',
-          defaultPatchesRepo,
-        );
-      } else {
-        return await _githubAPI.getLatestReleaseFile('.jar', repoName);
-      }
+      final String currentVersion = await getCurrentPatchesVersion();
+      return await _githubAPI.getPatchesReleaseFile(
+        '.jar',
+        repoName,
+        currentVersion,
+      );
     } on Exception catch (e) {
       if (kDebugMode) {
         print(e);
@@ -263,11 +275,23 @@ class ManagerAPI {
     );
   }
 
-  Future<String?> getLatestPatcherReleaseTime() async {
-    return await _revancedAPI.getLatestReleaseTime(
-      '.gz',
-      defaultPatcherRepo,
-    );
+  Future<String?> getLatestPatchesReleaseTime() async {
+    if (isDefaultPatchesRepo()) {
+      return await _revancedAPI.getLatestReleaseTime(
+        '.json',
+        defaultPatchesRepo,
+      );
+    } else {
+      final release =
+          await _githubAPI.getLatestPatchesRelease(getPatchesRepo());
+      if (release != null) {
+        final DateTime timestamp =
+            DateTime.parse(release['created_at'] as String);
+        return format(timestamp, locale: 'en_short');
+      } else {
+        return null;
+      }
+    }
   }
 
   Future<String?> getLatestManagerReleaseTime() async {
@@ -285,10 +309,19 @@ class ManagerAPI {
   }
 
   Future<String?> getLatestPatchesVersion() async {
-    return await _revancedAPI.getLatestReleaseVersion(
-      '.json',
-      defaultPatchesRepo,
-    );
+    if (isDefaultPatchesRepo()) {
+      return await _revancedAPI.getLatestReleaseVersion(
+        '.json',
+        defaultPatchesRepo,
+      );
+    } else {
+      final release = await _githubAPI.getLatestPatchesRelease(getPatchesRepo());
+      if (release != null) {
+        return release['tag_name'];
+      } else {
+        return null;
+      }
+    }
   }
 
   Future<String> getCurrentManagerVersion() async {
@@ -296,16 +329,17 @@ class ManagerAPI {
     return packageInfo.version;
   }
 
-  Future<String?> getCurrentPatchesVersion() async {
-    if (isDefaultPatchesRepo()) {
-      patchesVersion = await getLatestPatchesVersion();
-      // print('Patches version: $patchesVersion');
-    } else {
-      // fetch from github
-      patchesVersion =
-          await _githubAPI.getLastestReleaseVersion(getPatchesRepo());
+  Future<String> getCurrentPatchesVersion() async {
+    patchesVersion = _prefs.getString('patchesVersion') ?? '0.0.0';
+    if (patchesVersion == '0.0.0' || isPatchesAutoUpdate()) {
+      patchesVersion = await getLatestPatchesVersion() ?? '0.0.0';
+      await setCurrentPatchesVersion(patchesVersion!);
     }
-    return patchesVersion ?? '0.0.0';
+    return patchesVersion!;
+  }
+
+  Future<void> setCurrentPatchesVersion(String version) async {
+    await _prefs.setString('patchesVersion', version);
   }
 
   Future<List<PatchedApplication>> getAppsToRemove(

lib/services/revanced_api.dart

@@ -7,7 +7,6 @@ import 'package:dio_cache_interceptor/dio_cache_interceptor.dart';
 import 'package:flutter/foundation.dart';
 import 'package:flutter_cache_manager/flutter_cache_manager.dart';
 import 'package:injectable/injectable.dart';
-import 'package:revanced_manager/models/patch.dart';
 import 'package:timeago/timeago.dart';
 
 @lazySingleton
@@ -64,19 +63,6 @@ class RevancedAPI {
     return contributors;
   }
 
-  Future<List<Patch>> getPatches() async {
-    try {
-      final response = await _dio.get('/patches');
-      final List<dynamic> patches = response.data;
-      return patches.map((patch) => Patch.fromJson(patch)).toList();
-    } on Exception catch (e) {
-      if (kDebugMode) {
-        print(e);
-      }
-      return List.empty();
-    }
-  }
-
   Future<Map<String, dynamic>?> _getLatestRelease(
     String extension,
     String repoName,