# HG changeset patch
# User Markus Kottlaender <markus@intevation.de>
# Date 1558097939 -7200
# Node ID c7b48272e11ea5ef52ae699135c9eaaade2004cd
# Parent  d61b88439c8524ea3aaf533d19f5acd4a99c9bce
client: fairway availability: removed unnecessary computed property

Since entries() was only used for stretches/sections the check for type=bottlenecks was not necessary.
To make think even more clear and straight forward, I just removed this computed entries property and just
added a new select, so that now every type has it's own select. Bit more html but less bloated component logic.

diff -r d61b88439c85 -r c7b48272e11e client/src/components/fairway/AvailableFairwayDepthDialogue.vue
--- a/client/src/components/fairway/AvailableFairwayDepthDialogue.vue	Fri May 17 14:41:28 2019 +0200
+++ b/client/src/components/fairway/AvailableFairwayDepthDialogue.vue	Fri May 17 14:58:59 2019 +0200
@@ -30,7 +30,6 @@
           </div>
         </div>
         <select
-          :key="1"
           v-if="type === $options.BOTTLENECKS"
           @change="entrySelected"
           class="form-control font-weight-bold"
@@ -52,19 +51,34 @@
           </optgroup>
         </select>
         <select
-          :key="2"
-          v-else
+          v-else-if="type === $options.STRETCHES"
           @change="entrySelected"
           class="form-control font-weight-bold"
           v-model="selectedEntry"
         >
           <option :value="null">{{ placeholder }}</option>
           <option
-            v-for="(entry, index) in entries"
-            :value="entry"
-            :key="index"
-            >{{ entry.properties.name }}</option
+            v-for="stretch in stretches"
+            :value="stretch"
+            :key="stretch.id"
           >
+            {{ stretch.properties.name }}
+          </option>
+        </select>
+        <select
+          v-else-if="type === $options.SECTIONS"
+          @change="entrySelected"
+          class="form-control font-weight-bold"
+          v-model="selectedEntry"
+        >
+          <option :value="null">{{ placeholder }}</option>
+          <option
+            v-for="section in sections"
+            :value="section"
+            :key="section.id"
+          >
+            {{ section.properties.name }}
+          </option>
         </select>
         <div class="d-flex mt-2">
           <div class="d-flex flex-column w-50 mr-1">
@@ -253,12 +267,6 @@
         );
       }
     },
-    entries() {
-      if (this.type === this.$options.BOTTLENECKS) return this.bottlenecksList;
-      if (this.type === this.$options.STRETCHES) return this.stretches;
-      if (this.type === this.$options.SECTIONS) return this.sections;
-      return [];
-    },
     label() {
       return this.$gettext("Available fairway depth");
     },