fix: Fix external links (openfoodfacts#5754)

* Fix: Donate link * Better handle external links * Typo
monsieurtanuki · Oct 27, 2024 · 3475631 · 3475631
1 parent b07b942
commit 3475631
Show file tree

Hide file tree

Showing 3 changed files with 38 additions and 13 deletions.
diff --git a/packages/smooth_app/lib/helpers/launch_url_helper.dart b/packages/smooth_app/lib/helpers/launch_url_helper.dart
@@ -18,22 +18,26 @@ class LaunchUrlHelper {
       'http(s)?://[a-z]*.open(food|beauty|products|petfood)facts.(net|org)',
     ))) {
       AnalyticsHelper.trackOutlink(url: url);
-      GoRouter.of(context).go(url);
+      GoRouter.of(context).push(url);
     } else {
       return launchURL(url);
     }
   }
 
   /// Launches the url in an external browser.
-  static Future<void> launchURL(String url) async {
+  static Future<void> launchURL(
+    String url, {
+    LaunchMode? mode,
+  }) async {
     AnalyticsHelper.trackOutlink(url: url);
 
     try {
       await launchUrl(
         Uri.parse(url),
-        mode: Platform.isAndroid
-            ? LaunchMode.externalApplication
-            : LaunchMode.platformDefault,
+        mode: mode ??
+            (Platform.isAndroid
+                ? LaunchMode.externalApplication
+                : LaunchMode.platformDefault),
       );
     } catch (e) {
       throw 'Could not launch $url,Error: $e';

diff --git a/packages/smooth_app/lib/pages/navigator/app_navigator.dart b/packages/smooth_app/lib/pages/navigator/app_navigator.dart
@@ -1,5 +1,5 @@
 import 'package:collection/collection.dart';
-import 'package:flutter/widgets.dart';
+import 'package:flutter/material.dart';
 import 'package:go_router/go_router.dart';
 import 'package:openfoodfacts/openfoodfacts.dart';
 import 'package:provider/provider.dart';
@@ -79,8 +79,15 @@ class AppNavigator extends InheritedWidget {
     }
   }
 
-  void pop([dynamic result]) {
-    _router.router.pop(result);
+  /// Returns [true] if the pop was successful
+  /// Returns [false] if there is nothing to pop (= no history)
+  bool pop([dynamic result]) {
+    try {
+      _router.router.pop(result);
+      return true;
+    } on GoError catch (_) {
+      return false;
+    }
   }
 }
 
@@ -242,9 +249,11 @@ class _SmoothGoRouter {
           ],
         ),
         GoRoute(
-          path: '/${_InternalAppRoutes.EXTERNAL_PAGE}/:page',
+          path: '/${_InternalAppRoutes.EXTERNAL_PAGE}',
           builder: (BuildContext context, GoRouterState state) {
-            return ExternalPage(path: state.pathParameters['page']!);
+            return ExternalPage(
+              path: Uri.decodeFull(state.uri.queryParameters['path']!),
+            );
           },
         ),
       ],
@@ -468,5 +477,5 @@ class AppRoutes {
 
   // Open an external link (where path is relative to the OFF website)
   static String EXTERNAL(String path) =>
-      '/${_InternalAppRoutes.EXTERNAL_PAGE}/$path';
+      '/${_InternalAppRoutes.EXTERNAL_PAGE}?path=${Uri.encodeFull(path)}';
 }
diff --git a/packages/smooth_app/lib/pages/navigator/external_page.dart b/packages/smooth_app/lib/pages/navigator/external_page.dart
@@ -2,13 +2,15 @@ import 'dart:io';
 
 import 'package:flutter/material.dart';
 import 'package:flutter_custom_tabs/flutter_custom_tabs.dart' as tabs;
+import 'package:go_router/go_router.dart';
 import 'package:http/http.dart' as http;
 import 'package:openfoodfacts/openfoodfacts.dart';
 import 'package:path/path.dart' as path;
 import 'package:smooth_app/helpers/launch_url_helper.dart';
 import 'package:smooth_app/pages/navigator/app_navigator.dart';
 import 'package:smooth_app/query/product_query.dart';
 import 'package:smooth_app/services/smooth_services.dart';
+import 'package:url_launcher/url_launcher.dart';
 
 /// This screen is only used for deep links!
 ///
@@ -62,6 +64,7 @@ class _ExternalPageState extends State<ExternalPage> {
 
       try {
         if (Platform.isAndroid) {
+          /// Custom tabs
           WidgetsFlutterBinding.ensureInitialized();
           await tabs.launchUrl(
             Uri.parse(url),
@@ -70,13 +73,22 @@ class _ExternalPageState extends State<ExternalPage> {
             ),
           );
         } else {
-          await LaunchUrlHelper.launchURL(url);
+          /// The default browser
+          await LaunchUrlHelper.launchURL(
+            url,
+            mode: LaunchMode.externalApplication,
+          );
         }
       } catch (e) {
         Logs.e('Unable to open an external link', ex: e);
       } finally {
         if (mounted) {
-          AppNavigator.of(context).pop();
+          final bool success = AppNavigator.of(context).pop();
+          if (!success) {
+            /// This page was called with the go() without an history
+            /// (mainly for an external deep link)
+            GoRouter.of(context).go('/');
+          }
         }
       }
     });