From 35542ba1b9303f6a98bf1602dd03a19b801cb1d7 Mon Sep 17 00:00:00 2001 From: Rick Barrette Date: Thu, 9 Apr 2026 09:42:45 -0400 Subject: [PATCH] Fix vehicle name fallback not applied in select by replacing pluck with model instances --- lib/vehicles/hooks/issues_form_hook_listener.rb | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/lib/vehicles/hooks/issues_form_hook_listener.rb b/lib/vehicles/hooks/issues_form_hook_listener.rb index 62b2241..7c8ba9b 100644 --- a/lib/vehicles/hooks/issues_form_hook_listener.rb +++ b/lib/vehicles/hooks/issues_form_hook_listener.rb @@ -18,14 +18,18 @@ module Vehicles def view_issues_form_details_bottom(context={}) # Load the customer's vehicles for selection in the issue form. + vehicles = context[:issue].customer&.vehicles || [] + context[:controller].send(:render_to_string, { partial: 'issues/form_hook_vehicles', locals: { - vehicle: context[:form].select( :vehicle_id, - context[:issue].customer ? context[:issue].customer.vehicles.pluck(:name, :id) : [], - selected: context[:issue].vehicle ? context[:issue].vehicle.id : nil, - include_blank: true ) - } + vehicle: context[:form].select( + :vehicle_id, + vehicles.map { |v| [v.name, v.id] }, + selected: context[:issue].vehicle&.id, + include_blank: true + ) + } }) end