Fix usage of "ui/forms/select" in few components

Following the change in UI select component, forcing `choices` to be an
array, needed to change few store credit components passing their
options as a string.

Also removes blank option for currency selector, since there is a
default selected value, and currency cannot be blank, so there is no
reason to allow selecting a blank option there.

Author: chaimann

admin/app/components/solidus_admin/users/store_credits/edit_amount/component.html.erb

@@ -6,8 +6,8 @@
         <%= render component("ui/forms/field").select(
           f,
           :store_credit_reason_id,
-          store_credit_reasons_select_options.html_safe,
-          include_blank: t('spree.choose_reason'),
+          @store_credit_reasons.map { [_1.name, _1.id] },
+          include_blank: t('.choose_reason'),
           html: { required: true }
         ) %>
       </div>

admin/app/components/solidus_admin/users/store_credits/edit_amount/component.rb

@@ -14,9 +14,4 @@ def form_id
   def form_url
     solidus_admin.update_amount_user_store_credit_path(@user, @store_credit, **search_filter_params)
   end
-
-  def store_credit_reasons_select_options
-    # Placeholder + Store Credit Reasons
-    "<option value>#{t('.choose_reason')}</option>" + options_from_collection_for_select(@store_credit_reasons, :id, :name)
-  end
 end

admin/app/components/solidus_admin/users/store_credits/edit_validity/component.html.erb

@@ -5,8 +5,8 @@
         <%= render component("ui/forms/field").select(
           f,
           :store_credit_reason_id,
-          store_credit_reasons_select_options.html_safe,
-          include_blank: t('spree.choose_reason'),
+          @store_credit_reasons.map { [_1.name, _1.id] },
+          include_blank: t('.choose_reason'),
           html: { required: true }
         ) %>
       </div>

admin/app/components/solidus_admin/users/store_credits/edit_validity/component.rb

@@ -14,9 +14,4 @@ def form_id
   def form_url
     solidus_admin.invalidate_user_store_credit_path(@user, @store_credit, **search_filter_params)
   end
-
-  def store_credit_reasons_select_options
-    # Placeholder + Store Credit Reasons
-    "<option value>#{t('.choose_reason')}</option>" + options_from_collection_for_select(@store_credit_reasons, :id, :name)
-  end
 end

admin/app/components/solidus_admin/users/store_credits/new/component.html.erb

@@ -6,15 +6,16 @@
         <%= render component("ui/forms/field").select(
           f,
           :currency,
-          currency_select_options.html_safe,
-          include_blank: t("spree.currency"),
+          Spree::Config.available_currencies.map { [_1.iso_code, _1.iso_code] },
+          placeholder: t("spree.currency"),
+          value: Spree::Config.currency,
           html: { required: true }
         ) %>
         <%= render component("ui/forms/field").select(
           f,
           :category_id,
-          store_credit_categories_select_options.html_safe,
-          include_blank: t("spree.category"),
+          @store_credit_categories.map { [_1.name, _1.id] },
+          include_blank: t(".choose_category"),
           html: { required: true }
         ) %>
         <%= render component("ui/forms/field").text_field(f, :memo) %>

admin/app/components/solidus_admin/users/store_credits/new/component.rb

@@ -10,13 +10,4 @@ def initialize(user:, store_credit:, categories:)
   def form_url
     solidus_admin.user_store_credits_path(@user, **search_filter_params)
   end
-
-  def currency_select_options
-    options_from_collection_for_select(Spree::Config.available_currencies, :iso_code, :iso_code, Spree::Config.currency)
-  end
-
-  def store_credit_categories_select_options
-    # Placeholder + Store Credit Categories
-    "<option value>#{t('.choose_category')}</option>" + options_from_collection_for_select(@store_credit_categories, :id, :name)
-  end
 end