From 78c26cb669ce0a547970691803e4f829d92ad410 Mon Sep 17 00:00:00 2001
From: thrillfall <thrillfall@users.noreply.github.com>
Date: Tue, 8 Sep 2020 08:50:07 +0200
Subject: [PATCH] unify export filename logic

---
 .../fragments/ExportProgressDialogFragment.java        |  4 ++--
 .../dennisguse/opentracks/io/file/ExportFileName.java  | 10 ++++++++++
 2 files changed, 12 insertions(+), 2 deletions(-)
 create mode 100644 src/main/java/de/dennisguse/opentracks/io/file/ExportFileName.java

diff --git a/src/main/java/de/dennisguse/opentracks/fragments/ExportProgressDialogFragment.java b/src/main/java/de/dennisguse/opentracks/fragments/ExportProgressDialogFragment.java
index a907c31b3..f1cf66da0 100644
--- a/src/main/java/de/dennisguse/opentracks/fragments/ExportProgressDialogFragment.java
+++ b/src/main/java/de/dennisguse/opentracks/fragments/ExportProgressDialogFragment.java
@@ -38,6 +38,7 @@ import de.dennisguse.opentracks.R;
 import de.dennisguse.opentracks.content.data.Track;
 import de.dennisguse.opentracks.content.data.TracksColumns;
 import de.dennisguse.opentracks.content.provider.ContentProviderUtils;
+import de.dennisguse.opentracks.io.file.ExportFileName;
 import de.dennisguse.opentracks.io.file.TrackFileFormat;
 import de.dennisguse.opentracks.io.file.exporter.TrackExporter;
 import de.dennisguse.opentracks.util.DialogUtils;
@@ -208,8 +209,7 @@ public class ExportProgressDialogFragment extends DialogFragment {
         private Boolean exportTrack(Context context, Track track) {
             TrackExporter trackExporter = trackFileFormat.newTrackExporter(context);
 
-            //TODO Move to helper function
-            String fileName = track.getId().getId() + "." + trackFileFormat.getExtension();
+            String fileName = ExportFileName.getExportFileNameByTrackId(track.getId(), trackFileFormat.getExtension());
 
             // Overwrite a file if it exists; DocumentFile.createFile() creates a new file appending a suffix if the displayname already exists.
             DocumentFile file = directory.findFile(fileName);
diff --git a/src/main/java/de/dennisguse/opentracks/io/file/ExportFileName.java b/src/main/java/de/dennisguse/opentracks/io/file/ExportFileName.java
new file mode 100644
index 000000000..c3eb6b038
--- /dev/null
+++ b/src/main/java/de/dennisguse/opentracks/io/file/ExportFileName.java
@@ -0,0 +1,10 @@
+package de.dennisguse.opentracks.io.file;
+
+import de.dennisguse.opentracks.content.data.Track;
+
+public class ExportFileName {
+
+    public static String getExportFileNameByTrackId(Track.Id trackId, String trackFileFormatExtension) {
+        return trackId.getId() + "." + trackFileFormatExtension;
+    }
+}