feat: remove passenger fields

TODO: disallow removal of 0th passenger, set max to flight.available_seats
sean-garwood · Nov 12, 2024 · 18fd830 · 18fd830
1 parent d0aec92
commit 18fd830
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/app/javascript/controllers/passenger_controller.js b/app/javascript/controllers/passenger_controller.js
@@ -14,4 +14,16 @@ export default class extends Controller {
     newField.innerHTML = newField.innerHTML.replace(/\[[0-9]*\]/g, `[${newIndex}]`).replace(/_[0-9]*_/g, `_${newIndex}_`);
     this.fieldsTarget.insertAdjacentElement('beforebegin', newField);
   }
+
+  remove(event) {
+    event.preventDefault()
+    const lastIndex = parseInt(this.fieldsTarget.querySelector('input').name.match(/\d+/)[0]);
+    // if lastIndex == 0, flash error--we don't want to remove last set of fields
+    if (lastIndex == 0) {
+      alert("You must have at least one passenger.");
+      return;
+    };
+
+    this.fieldsTarget.remove();
+  }
 }
diff --git a/app/views/bookings/_passenger_info.html.erb b/app/views/bookings/_passenger_info.html.erb
@@ -12,6 +12,7 @@
       </div>
     <% end %>
     <button type="button" data-action="passenger#add">Add Passenger</button>
+    <button type="button" data-action="passenger#remove">Remove Passenger</button>
   </div>
   <%= booking_form.submit "Book Flight" %>
 <% end %>