From 5ffd6b6074170db558daddcc95e126dc6278e1a4 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20Rebelo?= <joserebelo@outlook.com>
Date: Fri, 15 May 2026 23:09:10 +0100
Subject: [PATCH] Set mimeType when sharing files

---
 .../activities/files/FileManagerAdapter.java       | 13 +++++++++++--
 .../activities/workouts/WorkoutDetailsFragment.kt  | 14 ++++++++++----
 .../gadgetbridge/util/AndroidUtils.java            | 13 ++++++-------
 3 files changed, 27 insertions(+), 13 deletions(-)

diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/files/FileManagerAdapter.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/files/FileManagerAdapter.java
index ed02d930f2..ed956f3bbf 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/files/FileManagerAdapter.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/files/FileManagerAdapter.java
@@ -21,6 +21,7 @@ import android.content.Intent;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
+import android.webkit.MimeTypeMap;
 import android.widget.Filter;
 import android.widget.ImageView;
 import android.widget.PopupMenu;
@@ -111,8 +112,16 @@ public class FileManagerAdapter extends RecyclerView.Adapter<FileManagerAdapter.
                     final int itemId = item.getItemId();
                     if (itemId == R.id.file_manager_file_menu_share) {
                         try {
-                            AndroidUtils.shareFile(mContext, file, "*/*");
-                        } catch (final IOException e) {
+                            String mimeType = null;
+                            final String extension = MimeTypeMap.getFileExtensionFromUrl(file.getPath());
+                            if (extension != null) {
+                                mimeType = MimeTypeMap.getSingleton().getMimeTypeFromExtension(extension);
+                            }
+                            if (mimeType == null) {
+                                mimeType = "*/*";
+                            }
+                            AndroidUtils.shareFile(mContext, file, mimeType);
+                        } catch (final Exception e) {
                             GB.toast("Failed to share file", Toast.LENGTH_LONG, GB.ERROR, e);
                         }
                         return true;
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/workouts/WorkoutDetailsFragment.kt b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/workouts/WorkoutDetailsFragment.kt
index 65f8913181..74ff4ff5cf 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/workouts/WorkoutDetailsFragment.kt
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/activities/workouts/WorkoutDetailsFragment.kt
@@ -720,7 +720,7 @@ class WorkoutDetailsFragment : Fragment(), MenuProvider {
         }
 
         try {
-            AndroidUtils.shareFile(requireContext(), gpxFile)
+            AndroidUtils.shareFile(requireContext(), gpxFile, "application/gpx+xml")
         } catch (e: Exception) {
             GB.toast(
                 requireContext(),
@@ -742,7 +742,12 @@ class WorkoutDetailsFragment : Fragment(), MenuProvider {
             FileUtils.makeValidFileName("${DateTimeUtils.formatIso8601(workout.summary.startTime)}_summary.bin")
 
         try {
-            AndroidUtils.shareBytesAsFile(requireContext(), filename, workout.summary.rawSummaryData)
+            AndroidUtils.shareBytesAsFile(
+                requireContext(),
+                filename,
+                workout.summary.rawSummaryData,
+                "application/octet-stream"
+            )
         } catch (e: Exception) {
             GB.toast(
                 requireContext(),
@@ -766,7 +771,7 @@ class WorkoutDetailsFragment : Fragment(), MenuProvider {
         }
 
         try {
-            AndroidUtils.shareFile(requireContext(), file)
+            AndroidUtils.shareFile(requireContext(), file, "application/octet-stream")
         } catch (e: Exception) {
             GB.toast(
                 requireContext(),
@@ -785,7 +790,8 @@ class WorkoutDetailsFragment : Fragment(), MenuProvider {
             AndroidUtils.shareBytesAsFile(
                 requireContext(),
                 filename,
-                workout.data.toString().toByteArray(StandardCharsets.UTF_8)
+                workout.data.toString().toByteArray(StandardCharsets.UTF_8),
+                "application/json"
             )
         } catch (e: Exception) {
             GB.toast(
diff --git a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/AndroidUtils.java b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/AndroidUtils.java
index e65df2b6f1..683347003c 100644
--- a/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/AndroidUtils.java
+++ b/app/src/main/java/nodomain/freeyourgadget/gadgetbridge/util/AndroidUtils.java
@@ -286,7 +286,10 @@ public class AndroidUtils {
         }
     }
 
-    public static void shareBytesAsFile(final Context context, final String name, final byte[] bytes) throws IOException {
+    public static void shareBytesAsFile(final Context context,
+                                        final String name,
+                                        final byte[] bytes,
+                                        final String type) throws IOException {
         final File cacheDir = context.getCacheDir();
         final File rawCacheDir = new File(cacheDir, "raw");
         rawCacheDir.mkdir();
@@ -299,19 +302,15 @@ public class AndroidUtils {
             return;
         }
 
-        shareFile(context, file);
+        shareFile(context, file, type);
     }
 
-    public static void shareFile(final Context context, final File file) throws IOException {
+    public static void shareFile(final Context context, final File file, final String type) throws IOException {
         if (!file.exists()) {
             LOG.warn("File {} does not exist", file.getPath());
             return;
         }
 
-        shareFile(context, file, "*/*");
-    }
-
-    public static void shareFile(final Context context, final File file, final String type) throws IOException {
         final Uri contentUri = FileProvider.getUriForFile(
                 context,
                 context.getApplicationContext().getPackageName() + ".screenshot_provider",