Dynamically set shoe select option.

stevepolitodesign · Sep 2, 2020 · 7c47a33 · 7c47a33
1 parent 693f4f7
commit 7c47a33
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 13 deletions.
diff --git a/app/views/activities/_form.html.erb b/app/views/activities/_form.html.erb
@@ -14,9 +14,12 @@
             <%= f.collection_select :difficulty, Activity.difficulties.keys.map{ |activity| [activity, activity.humanize] }, :first, :second, {}, class: "custom-select" %>
         </div>
         <div class="col-auto form-group">
-            <div id="shoes_select_wrapper">
-            <%= render partial: "activities/shoes_select", locals: { f: f } %>
-            </div>
+            <%= f.label :shoe_id, "Shoe" %>
+            <% if f.object.persisted? %>
+                <%= f.collection_select :shoe_id, current_user.shoes.alphabetized, :id, :name_with_miles, { prompt: true }, class: "custom-select" %>
+            <% else %>
+                <%= f.collection_select :shoe_id, current_user.shoes.available.alphabetized, :id, :name_with_miles, { prompt: true }, class: "custom-select" %>
+            <% end %>
             <button type="button" class="btn btn-link" data-toggle="modal" data-target="#shoeFormModal">
                 Add Shoe
             </button>

diff --git a/app/views/activities/_shoes_select.html.erb b/app/views/activities/_shoes_select.html.erb
diff --git a/app/views/shoes/create.js.erb b/app/views/shoes/create.js.erb
@@ -1,6 +1,9 @@
-var wrapper = document.querySelector("#shoes_select_wrapper");
+var shoeSelectList = document.querySelector("#activity_shoe_id");
 
-if(wrapper) {
-    wrapper.innerHTML = null;
-    wrapper.insertAdjacentHTML("afterbegin", <%= escape_javascript( render partial: "activities/shoes_select", locals: { f: @shoe } )%>);
+if(shoeSelectList) {
+    var options = shoeSelectList.querySelectorAll("option");
+    options.forEach((option) => {
+        option.removeAttribute("selected")
+    });
+    shoeSelectList.querySelector("option").insertAdjacentHTML("afterend", "<option value='<%= @shoe.id %>' selected><%= @shoe.name_with_miles %></option>");
 }