From 74f5eca2f34ca0dc33207c2423b5f5c65bf8d160 Mon Sep 17 00:00:00 2001
From: Raphael Odini <raphodn@users.noreply.github.com>
Date: Thu, 22 Aug 2024 11:37:25 +0200
Subject: [PATCH] refactor(Country detail): add map display option

---
 src/views/CountryDetail.vue | 48 +++++++++++++++++++++++++++++--------
 1 file changed, 38 insertions(+), 10 deletions(-)

diff --git a/src/views/CountryDetail.vue b/src/views/CountryDetail.vue
index 100cf9236fc..0448b8fadf0 100644
--- a/src/views/CountryDetail.vue
+++ b/src/views/CountryDetail.vue
@@ -12,14 +12,26 @@
       </h2>
       <v-progress-circular v-if="loading" indeterminate :size="30" />
       <LoadedCountChip v-if="!loading" :loadedCount="countryLocationList.length" :totalCount="countryLocationTotal" />
+      <DisplayMenu kind="price" :currentDisplay="currentDisplay" @update:currentDisplay="selectPriceDisplay($event)" />
     </v-col>
   </v-row>
 
-  <v-row class="mt-0">
-    <v-col v-for="location in countryLocationList" :key="location" cols="12" sm="6" md="4">
-      <LocationCard :location="location" :hideLocationOSMID="true" :hideCountryCity="true" height="100%" />
-    </v-col>
-  </v-row>
+  <v-window v-model="currentDisplay">
+    <v-window-item value="list">
+      <v-row class="mt-0 mb-1">
+        <v-col v-for="location in countryLocationList" :key="location" cols="12" sm="6" md="4">
+          <LocationCard :location="location" :hideLocationOSMID="true" :hideCountryCity="true" height="100%" />
+        </v-col>
+      </v-row>
+    </v-window-item>
+    <v-window-item value="map">
+      <v-row class="mt-0 mb-1">
+        <v-col style="height:400px">
+          <LeafletMap :locations="countryLocationList" />
+        </v-col>
+      </v-row>
+    </v-window-item>
+  </v-window>
 
   <v-row v-if="countryLocationList.length < countryLocationTotal" class="mb-2">
     <v-col align="center">
@@ -32,18 +44,22 @@
 
 <script>
 import { defineAsyncComponent } from 'vue'
+import constants from '../constants'
 import api from '../services/api'
 
 export default {
   components: {
     CountryCard: defineAsyncComponent(() => import('../components/CountryCard.vue')),
     LoadedCountChip: defineAsyncComponent(() => import('../components/LoadedCountChip.vue')),
-    LocationCard: defineAsyncComponent(() => import('../components/LocationCard.vue'))
+    DisplayMenu: defineAsyncComponent(() => import('../components/DisplayMenu.vue')),
+    LocationCard: defineAsyncComponent(() => import('../components/LocationCard.vue')),
+    LeafletMap: defineAsyncComponent(() => import('../components/LeafletMap.vue')),
   },
   data() {
     return {
-      // filter & order
+      // order & display
       currentOrder: '-price_count',
+      currentDisplay: constants.LOCATION_DISPLAY_LIST[0].key,
       // data
       country: null,  // see init
       countryLocationList: [],
@@ -59,13 +75,20 @@ export default {
     },
   },
   watch: {
-    $route (newCountry, oldCountry) {
-      if (oldCountry && newCountry && newCountry.name == 'country-detail' && oldCountry.fullPath != newCountry.fullPath) {
-        this.initCountry()
+    $route (newRoute, oldRoute) {
+      // only called when query changes to avoid having an API call when the path changes
+      // but ignore 'display' changes
+      if (oldRoute.path === newRoute.path) {
+        const oldRouteQueryFiltered = Object.fromEntries(Object.entries(oldRoute.query).filter(([key, value]) => key !== constants.DISPLAY_PARAM))  // eslint-disable-line no-unused-vars
+        const newRouteQueryFiltered = Object.fromEntries(Object.entries(newRoute.query).filter(([key, value]) => key !== constants.DISPLAY_PARAM))  // eslint-disable-line no-unused-vars
+        if (JSON.stringify(oldRouteQueryFiltered) !== JSON.stringify(newRouteQueryFiltered)) {
+          this.initCountryCity()
+        }
       }
     }
   },
   mounted() {
+    this.currentDisplay = this.$route.query[constants.DISPLAY_PARAM] || this.currentDisplay
     this.initCountry()
   },
   methods: {
@@ -86,6 +109,11 @@ export default {
           this.loading = false
         })
     },
+    selectPriceDisplay(displayKey) {
+      this.currentDisplay = displayKey
+      this.$router.push({ query: { ...this.$route.query, [constants.DISPLAY_PARAM]: this.currentDisplay } })
+      // this.initCountryCity() will NOT be called in watch $route
+    },
   }
 }
 </script>