From 37365f077a5646d3455c0b81286ecd29a7381b73 Mon Sep 17 00:00:00 2001
From: wronny <github@wron.de>
Date: Fri, 20 Oct 2023 08:28:51 +0200
Subject: [PATCH 1/2] added "Download Track" button to AppNavigationTrackItem

Signed-off-by: wronny <github@wron.de>
---
 lib/Controller/PublicTracksController.php |  5 ++++-
 src/components/AppNavigationTrackItem.vue | 26 ++++++++++++++++++++++-
 2 files changed, 29 insertions(+), 2 deletions(-)

diff --git a/lib/Controller/PublicTracksController.php b/lib/Controller/PublicTracksController.php
index 1cc925641..c10707a80 100644
--- a/lib/Controller/PublicTracksController.php
+++ b/lib/Controller/PublicTracksController.php
@@ -45,6 +45,7 @@
 use OCA\Maps\Service\TracksService;
 use OCP\Share\IManager as ShareManager;
 
+
 /**
  * @param string $text
  * @return string
@@ -151,6 +152,7 @@ private function getShareNode() {
 	 */
     public function getTracks(): DataResponse {
 		$share = $this->getShare();
+		$hideDownload = (bool) $share->getHideDownload();
 		$permissions = $share->getPermissions();
 		$folder = $this->getShareNode();
 		$isReadable = (bool) ($permissions & (1 << 0));
@@ -158,12 +160,13 @@ public function getTracks(): DataResponse {
 			$owner = $share->getShareOwner();
 			$pre_path = $this->root->getUserFolder($owner)->getPath();
 			$tracks = $this->tracksService->getTracksFromDB($owner, $folder, true, false, false);
-			$new_tracks = array_map(function ($track) use ($folder, $permissions, $pre_path) {
+			$new_tracks = array_map(function ($track) use ($folder, $permissions, $pre_path, $hideDownload) {
 				$track['isCreatable'] = ($permissions & (1 << 2)) && $track['isCreatable'];
 				$track['isUpdateable'] = ($permissions & (1 << 1)) && $track['isUpdateable'];
 				$track['isDeletable'] = ($permissions & (1 << 3)) && $track['isDeletable'];
 				$track['path'] = $folder->getRelativePath($pre_path.$track['path']);
 				$track['filename'] = $track['path'];
+				$track['hideDownload'] = $hideDownload;
 				return $track;
 			}, $tracks);
 		} else {
diff --git a/src/components/AppNavigationTrackItem.vue b/src/components/AppNavigationTrackItem.vue
index 20f46dfa0..ec79828f3 100644
--- a/src/components/AppNavigationTrackItem.vue
+++ b/src/components/AppNavigationTrackItem.vue
@@ -60,6 +60,21 @@
 				@click="$emit('add-to-map-track', track)">
 				{{ t('maps', 'Copy to map') }}
 			</NcActionButton>
+			<NcActionLink v-if="parentEnabled && track.enabled && !isPublic()"
+                                target="_self"
+                                :href="downloadTrackUrl"
+				icon="icon-download"
+                                :close-after-click="true">
+                                {{ t('maps', 'Download track') }}
+                        </NcActionLink>
+			<NcActionLink v-if="parentEnabled && track.enabled && isPublic() && !(track.hideDownload)"
+                                target="_self"
+                                :href="downloadTrackShareUrl"
+				icon="icon-download"
+                                :close-after-click="true">
+                                {{ t('maps', 'Download track') }}
+                        </NcActionLink>
+
 		</template>
 	</NcAppNavigationItem>
 </template>
@@ -67,7 +82,9 @@
 <script>
 import NcAppNavigationItem from '@nextcloud/vue/dist/Components/NcAppNavigationItem'
 import NcActionButton from '@nextcloud/vue/dist/Components/NcActionButton'
-import {isPublic} from "../utils/common";
+import NcActionLink from '@nextcloud/vue/dist/Components/NcActionLink'
+import {isPublic, getToken} from "../utils/common";
+import { generateUrl } from "@nextcloud/router";
 
 export default {
 	name: 'AppNavigationTrackItem',
@@ -75,6 +92,7 @@ export default {
 	components: {
 		NcAppNavigationItem,
 		NcActionButton,
+		NcActionLink,
 	},
 
 	props: {
@@ -94,6 +112,12 @@ export default {
 	},
 
 	computed: {
+		downloadTrackUrl() {
+			return OCA.Files.App.fileList.filesClient.getBaseUrl() + this.track.file_path
+		},
+		downloadTrackShareUrl() {
+			return generateUrl('s/' + getToken() + '/download' + '?path=/&files=' + this.track.file_name )
+		},
 	},
 
 	methods: {

From f6ec01df655935a1e0a939852369a6dc9546d5b5 Mon Sep 17 00:00:00 2001
From: wronny <github@wron.de>
Date: Sun, 22 Oct 2023 01:24:50 +0200
Subject: [PATCH 2/2] added "Download track" button to track context menu

Signed-off-by: wronny <github@wron.de>
---
 src/components/map/TrackLayer.vue | 40 ++++++++++++++++++++++++++++++-
 1 file changed, 39 insertions(+), 1 deletion(-)

diff --git a/src/components/map/TrackLayer.vue b/src/components/map/TrackLayer.vue
index a5ed3de6f..2897f9208 100644
--- a/src/components/map/TrackLayer.vue
+++ b/src/components/map/TrackLayer.vue
@@ -17,6 +17,30 @@
 				@click="$emit('add-to-map-track', track)">
 				{{ t('maps', 'Copy to map') }}
 			</NcActionButton>
+			<NcActionLink v-if="!isPublic()"
+				:href="downloadTrackUrl"
+				target="_self"
+				icon="icon-download"
+				:close-after-click="true"
+				@click="closeafterclickworkaround">
+				<!-- 
+				looks like close-after-click not working in this popovermenu
+				therefore added workaround closeafterclickworkaround
+				-->
+				{{ t('maps', 'Download track') }}
+			</NcActionLink>
+			<NcActionLink v-if="isPublic() && !(track.hideDownload)" 
+				target="_self"
+				:href="downloadTrackShareUrl"
+				icon="icon-download"
+				:close-after-click="true"
+				@click="closeafterclickworkaround">
+				<!-- 
+				looks like close-after-click not working in this popovermenu
+				therefore added workaround closeafterclickworkaround
+				-->
+				{{ t('maps', 'Download track') }}
+			</NcActionLink>
 		</LPopup>
 		<LTooltip :options="tooltipOptions">
 			<div class="tooltip-track-wrapper"
@@ -57,10 +81,12 @@ import L from 'leaflet'
 import { LMarker, LTooltip, LPopup, LFeatureGroup, LPolyline } from 'vue2-leaflet'
 
 import NcActionButton from '@nextcloud/vue/dist/Components/NcActionButton'
+import NcActionLink from '@nextcloud/vue/dist/Components/NcActionLink'
 import moment from '@nextcloud/moment'
+import { generateUrl } from "@nextcloud/router";
 
 import optionsController from '../../optionsController'
-import {binSearch, isPublic} from '../../utils/common'
+import {binSearch, isPublic, getToken} from '../../utils/common'
 
 const TRACK_MARKER_VIEW_SIZE = 40
 const WAYPOINT_MARKER_VIEW_SIZE = 30
@@ -74,6 +100,7 @@ export default {
 		LFeatureGroup,
 		LPolyline,
 		NcActionButton,
+		NcActionLink,
 	},
 
 	props: {
@@ -116,6 +143,12 @@ export default {
 	},
 
 	computed: {
+		downloadTrackUrl() {
+			return OCA.Files.App.fileList.filesClient.getBaseUrl() + this.track.file_path
+		},
+		downloadTrackShareUrl() {
+			return generateUrl('s/' + getToken() + '/download' + '?path=/&files=' + this.track.file_name )
+		},
 		dateBegin() {
 			return this.track.metadata?.begin
 				? moment.unix(this.track.metadata.begin).format('LLL')
@@ -233,6 +266,11 @@ export default {
 	},
 
 	methods: {
+		// looks like close-after-click not working for NcAcionLink 
+		// added working closeafterclickworkaround
+		closeafterclickworkaround() {
+			this.$refs.featgroup.mapObject.closePopup()
+		},
 		onFGReady(f) {
 			// avoid left click popup
 			L.DomEvent.on(f, 'click', (ev) => {