Add release asset download button (#1493)

lib/app_sources/github.dart

@@ -271,17 +271,14 @@ class GitHub extends AppSource {
         }
       }
 
-      List<MapEntry<String, String>> getReleaseAPKUrls(dynamic release) =>
-          (release['assets'] as List<dynamic>?)
-              ?.map((e) {
-                return (e['name'] != null) &&
-                        ((e['url'] ?? e['browser_download_url']) != null)
-                    ? MapEntry(e['name'] as String,
-                        (e['url'] ?? e['browser_download_url']) as String)
-                    : const MapEntry('', '');
-              })
-              .where((element) => element.key.toLowerCase().endsWith('.apk'))
-              .toList() ??
+      List<MapEntry<String, String>> getReleaseAssetUrls(dynamic release) =>
+          (release['assets'] as List<dynamic>?)?.map((e) {
+            return (e['name'] != null) &&
+                    ((e['url'] ?? e['browser_download_url']) != null)
+                ? MapEntry(e['name'] as String,
+                    (e['url'] ?? e['browser_download_url']) as String)
+                : const MapEntry('', '');
+          }).toList() ??
           [];
 
       DateTime? getPublishDateFromRelease(dynamic rel) =>
@@ -383,7 +380,11 @@ class GitHub extends AppSource {
                 .hasMatch(((releases[i]['body'] as String?) ?? '').trim())) {
           continue;
         }
-        var apkUrls = getReleaseAPKUrls(releases[i]);
+        var allAssetUrls = getReleaseAssetUrls(releases[i]);
+        List<MapEntry<String, String>> apkUrls = allAssetUrls
+            .where((element) => element.key.toLowerCase().endsWith('.apk'))
+            .toList();
+
         apkUrls = filterApks(apkUrls, additionalSettings['apkFilterRegEx'],
             additionalSettings['invertAPKFilter']);
         if (apkUrls.isEmpty && additionalSettings['trackOnly'] != true) {
@@ -391,12 +392,25 @@ class GitHub extends AppSource {
         }
         targetRelease = releases[i];
         targetRelease['apkUrls'] = apkUrls;
+        targetRelease['version'] =
+            targetRelease['tag_name'] ?? targetRelease['name'];
+        if (targetRelease['tarball_url'] != null) {
+          allAssetUrls.add(MapEntry(
+              (targetRelease['version'] ?? 'source') + '.tar.gz',
+              targetRelease['tarball_url']));
+        }
+        if (targetRelease['zipball_url'] != null) {
+          allAssetUrls.add(MapEntry(
+              (targetRelease['version'] ?? 'source') + '.zip',
+              targetRelease['zipball_url']));
+        }
+        targetRelease['allAssetUrls'] = allAssetUrls;
         break;
       }
       if (targetRelease == null) {
         throw NoReleasesError();
       }
-      String? version = targetRelease['tag_name'] ?? targetRelease['name'];
+      String? version = targetRelease['version'];
       DateTime? releaseDate = getReleaseDateFromRelease(
           targetRelease, useLatestAssetDateAsReleaseDate);
       if (version == null) {
@@ -408,7 +422,9 @@ class GitHub extends AppSource {
           targetRelease['apkUrls'] as List<MapEntry<String, String>>,
           getAppNames(standardUrl),
           releaseDate: releaseDate,
-          changeLog: changeLog.isEmpty ? null : changeLog);
+          changeLog: changeLog.isEmpty ? null : changeLog,
+          allAssetUrls:
+              targetRelease['allAssetUrls'] as List<MapEntry<String, String>>);
     } else {
       if (onHttpErrorCode != null) {
         onHttpErrorCode(res);

lib/pages/app.dart

@@ -1,12 +1,15 @@
+import 'package:animations/animations.dart';
 import 'package:easy_localization/easy_localization.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
+import 'package:obtainium/components/generated_form.dart';
 import 'package:obtainium/components/generated_form_modal.dart';
 import 'package:obtainium/custom_errors.dart';
 import 'package:obtainium/main.dart';
 import 'package:obtainium/pages/apps.dart';
 import 'package:obtainium/pages/settings.dart';
 import 'package:obtainium/providers/apps_provider.dart';
+import 'package:obtainium/providers/notifications_provider.dart';
 import 'package:obtainium/providers/settings_provider.dart';
 import 'package:obtainium/providers/source_provider.dart';
 import 'package:url_launcher/url_launcher_string.dart';
@@ -158,6 +161,87 @@ class _AppPageState extends State<AppPage> {
             textAlign: TextAlign.center,
             style: const TextStyle(fontStyle: FontStyle.italic, fontSize: 12),
           ),
+          if (app?.app.apkUrls.isNotEmpty == true ||
+              app?.app.otherAssetUrls.isNotEmpty == true)
+            GestureDetector(
+              onTap: app?.app == null || updating
+                  ? null
+                  : () async {
+                      var allAssetUrls = [
+                        ...app!.app.apkUrls,
+                        ...app.app.otherAssetUrls
+                      ].map((e) => MapEntry(e.value, e.key)).toList();
+                      var values = await showModal(
+                        context: globalNavigatorKey.currentContext ?? context,
+                        builder: (BuildContext ctx) {
+                          return GeneratedFormModal(
+                              title:
+                                  tr('downloadX', args: [tr('releaseAsset')]),
+                              initValid: true,
+                              items: [
+                                [
+                                  GeneratedFormDropdown(
+                                      'assetToDownload', allAssetUrls,
+                                      defaultValue: allAssetUrls[0].key,
+                                      label: tr('selectX', args: [
+                                        tr('releaseAsset').toLowerCase()
+                                      ]))
+                                ]
+                              ]);
+                        },
+                      );
+
+                      if (values != null) {
+                        var downloadUrl = values['assetToDownload'] as String;
+                        var fileName = allAssetUrls
+                            .where((e) => e.key == downloadUrl)
+                            .first
+                            .value;
+                        NotificationsProvider notificationsProvider =
+                            (globalNavigatorKey.currentContext ?? context)
+                                .read<NotificationsProvider>();
+                        try {
+                          showMessage(
+                              '${tr('downloadingX', args: [fileName])}...',
+                              globalNavigatorKey.currentContext ?? context);
+                          await downloadFile(
+                              downloadUrl,
+                              fileName
+                                  .split('.')
+                                  .reversed
+                                  .toList()
+                                  .sublist(1)
+                                  .reversed
+                                  .join('.'), (double? progress) {
+                            notificationsProvider.notify(DownloadNotification(
+                                fileName, progress?.ceil() ?? 0));
+                          }, '/storage/emulated/0/Download',
+                              headers: await source?.getRequestHeaders(
+                                  app.app.additionalSettings,
+                                  forAPKDownload: fileName.endsWith('.apk')
+                                      ? true
+                                      : false));
+                          notificationsProvider.notify(
+                              DownloadedNotification(fileName, downloadUrl));
+                        } catch (e) {
+                          showError(
+                              e, globalNavigatorKey.currentContext ?? context);
+                        } finally {
+                          notificationsProvider
+                              .cancel(DownloadNotification(fileName, 0).id);
+                        }
+                      }
+                    },
+              child: Text(
+                tr('downloadX', args: [tr('releaseAsset').toLowerCase()]),
+                textAlign: TextAlign.center,
+                style: Theme.of(context).textTheme.labelSmall!.copyWith(
+                      decoration:
+                          changeLogFn != null ? TextDecoration.underline : null,
+                      fontStyle: changeLogFn != null ? FontStyle.italic : null,
+                    ),
+              ),
+            ),
           const SizedBox(
             height: 48,
           ),

lib/providers/notifications_provider.dart

@@ -120,6 +120,18 @@ class DownloadNotification extends ObtainiumNotification {
             progPercent: progPercent);
 }
 
+class DownloadedNotification extends ObtainiumNotification {
+  DownloadedNotification(String fileName, String downloadUrl)
+      : super(
+            downloadUrl.hashCode,
+            tr('downloadedX', args: [fileName]),
+            '',
+            'FILE_DOWNLOADED',
+            tr('downloadedXNotifChannel', args: [tr('app')]),
+            tr('downloadedX', args: [tr('app')]),
+            Importance.defaultImportance);
+}
+
 final completeInstallationNotification = ObtainiumNotification(
     1,
     tr('completeAppInstallation'),

lib/providers/source_provider.dart

@@ -47,9 +47,10 @@ class APKDetails {
   late AppNames names;
   late DateTime? releaseDate;
   late String? changeLog;
+  late List<MapEntry<String, String>> allAssetUrls;
 
   APKDetails(this.version, this.apkUrls, this.names,
-      {this.releaseDate, this.changeLog});
+      {this.releaseDate, this.changeLog, this.allAssetUrls = const []});
 }
 
 stringMapListTo2DList(List<MapEntry<String, String>> mapList) =>
@@ -223,6 +224,7 @@ class App {
   String? installedVersion;
   late String latestVersion;
   List<MapEntry<String, String>> apkUrls = [];
+  List<MapEntry<String, String>> otherAssetUrls = [];
   late int preferredApkIndex;
   late Map<String, dynamic> additionalSettings;
   late DateTime? lastUpdateCheck;
@@ -248,7 +250,8 @@ class App {
       this.releaseDate,
       this.changeLog,
       this.overrideSource,
-      this.allowIdChange = false});
+      this.allowIdChange = false,
+      this.otherAssetUrls = const []});
 
   @override
   String toString() {
@@ -280,41 +283,44 @@ class App {
       changeLog: changeLog,
       releaseDate: releaseDate,
       overrideSource: overrideSource,
-      allowIdChange: allowIdChange);
+      allowIdChange: allowIdChange,
+      otherAssetUrls: otherAssetUrls);
 
   factory App.fromJson(Map<String, dynamic> json) {
     json = appJSONCompatibilityModifiers(json);
     return App(
-        json['id'] as String,
-        json['url'] as String,
-        json['author'] as String,
-        json['name'] as String,
-        json['installedVersion'] == null
-            ? null
-            : json['installedVersion'] as String,
-        (json['latestVersion'] ?? tr('unknown')) as String,
-        assumed2DlistToStringMapList(jsonDecode(
-            (json['apkUrls'] ?? '[["placeholder", "placeholder"]]'))),
-        (json['preferredApkIndex'] ?? -1) as int,
-        jsonDecode(json['additionalSettings']) as Map<String, dynamic>,
-        json['lastUpdateCheck'] == null
-            ? null
-            : DateTime.fromMicrosecondsSinceEpoch(json['lastUpdateCheck']),
-        json['pinned'] ?? false,
-        categories: json['categories'] != null
-            ? (json['categories'] as List<dynamic>)
-                .map((e) => e.toString())
-                .toList()
-            : json['category'] != null
-                ? [json['category'] as String]
-                : [],
-        releaseDate: json['releaseDate'] == null
-            ? null
-            : DateTime.fromMicrosecondsSinceEpoch(json['releaseDate']),
-        changeLog:
-            json['changeLog'] == null ? null : json['changeLog'] as String,
-        overrideSource: json['overrideSource'],
-        allowIdChange: json['allowIdChange'] ?? false);
+      json['id'] as String,
+      json['url'] as String,
+      json['author'] as String,
+      json['name'] as String,
+      json['installedVersion'] == null
+          ? null
+          : json['installedVersion'] as String,
+      (json['latestVersion'] ?? tr('unknown')) as String,
+      assumed2DlistToStringMapList(
+          jsonDecode((json['apkUrls'] ?? '[["placeholder", "placeholder"]]'))),
+      (json['preferredApkIndex'] ?? -1) as int,
+      jsonDecode(json['additionalSettings']) as Map<String, dynamic>,
+      json['lastUpdateCheck'] == null
+          ? null
+          : DateTime.fromMicrosecondsSinceEpoch(json['lastUpdateCheck']),
+      json['pinned'] ?? false,
+      categories: json['categories'] != null
+          ? (json['categories'] as List<dynamic>)
+              .map((e) => e.toString())
+              .toList()
+          : json['category'] != null
+              ? [json['category'] as String]
+              : [],
+      releaseDate: json['releaseDate'] == null
+          ? null
+          : DateTime.fromMicrosecondsSinceEpoch(json['releaseDate']),
+      changeLog: json['changeLog'] == null ? null : json['changeLog'] as String,
+      overrideSource: json['overrideSource'],
+      allowIdChange: json['allowIdChange'] ?? false,
+      otherAssetUrls: assumed2DlistToStringMapList(
+          jsonDecode((json['otherAssetUrls'] ?? '[]'))),
+    );
   }
 
   Map<String, dynamic> toJson() => {
@@ -325,6 +331,7 @@ class App {
         'installedVersion': installedVersion,
         'latestVersion': latestVersion,
         'apkUrls': jsonEncode(stringMapListTo2DList(apkUrls)),
+        'otherAssetUrls': jsonEncode(stringMapListTo2DList(otherAssetUrls)),
         'preferredApkIndex': preferredApkIndex,
         'additionalSettings': jsonEncode(additionalSettings),
         'lastUpdateCheck': lastUpdateCheck?.microsecondsSinceEpoch,
@@ -892,8 +899,10 @@ class SourceProvider {
         allowIdChange: currentApp?.allowIdChange ??
             trackOnly ||
                 (source.appIdInferIsOptional &&
-                    inferAppIdIfOptional) // Optional ID inferring may be incorrect - allow correction on first install
-        );
+                    inferAppIdIfOptional), // Optional ID inferring may be incorrect - allow correction on first install
+        otherAssetUrls: apk.allAssetUrls
+            .where((a) => apk.apkUrls.indexWhere((p) => a.key == p.key) < 0)
+            .toList());
     return source.endOfGetAppChanges(finalApp);
   }