spree_admin 5.1.7 → 5.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1b5fef55ebb0f54365b38f656dd7dcd5d9add0dc89198f524945c539d8c9febd
-  data.tar.gz: d092feb207665dc4682dda6fa130f5d7981cb946f818ea9306c66b46db368b56
+  metadata.gz: 8387831f06788935b81121023a2c83d0f2930eb03b3e9a4a17861c8db8c6acd0
+  data.tar.gz: 1684b2c66579ada1e3faf72867b4ac72a942c9df44cc96dada5459d72b68a2f0
 SHA512:
-  metadata.gz: 69996445df30c272c3093ffafcc7987ff3fb334b9ba4e06e729c4916b02435aa107b39e4da64fe7ab703a424526964b31dc936a1111fac1f904b19a7c2b98d97
-  data.tar.gz: e25035318a0cba4e0ae9e305722a879cb59a2518d60b0ede374ab872694ca3c30e61196b1c31a3e21d96d964c982d3e599f854ee7d9c667c39be2cae1af309e0
+  metadata.gz: 4dc24174671f9cb9d62e65e1bd07bc6207a71fd044085283e4e17c52fcb78a7fbc0b8bc71ca2899f0d85857b0c9ac2d9c82548ec9ebf32691845f962aa105e28
+  data.tar.gz: 8ebb3cdb8185136fe868f7f5a347bca8de8eace8b82d29c8db5d0d4ece42f1af4568cb6f86e1dec1640532e6d54dcc959db05750e5690e86bb1db9010fef5721

data/app/controllers/spree/admin/payment_methods_controller.rb

@@ -32,10 +32,6 @@ module Spree
         @collection = @search.result.page(params[:page]).per(params[:per_page])
       end
 
-      def update_turbo_stream_enabled?
-        true
-      end
-
       def require_payment_type
         redirect_to spree.admin_payment_methods_path unless params.dig(:payment_method, :type).present?
       end

data/app/controllers/spree/admin/translations_controller.rb

@@ -1,12 +1,26 @@
 module Spree
   module Admin
     class TranslationsController < Spree::Admin::BaseController
+      # Set the resource being translated and any related data
       before_action :set_resource, only: [:edit, :update]
       before_action :load_data, only: [:edit]
       before_action :set_translation_locale, only: [:edit, :update]
 
+      helper_method :normalized_locale
+      helper_method :resource_class
+
+      # GET /admin/translations/:id/edit
+      # Renders the edit translation page for the resource
       def edit; end
 
+      # Normalizes a locale string by replacing '-' with '_' and downcasing
+      def normalized_locale(locale)
+        locale.to_s.downcase.tr('-', '_')
+      end
+
+      # PUT /admin/translations/:id
+      # Updates the translations for the resource
+      # Sets flash messages for success or failure
       def update
         @resource.update!(permitted_translation_params)
 
@@ -34,10 +48,12 @@ module Spree
         end
       end
 
+      # Build translation field names with normalized locale suffix
       def translation_fields(klass)
-        klass.translatable_fields.map { |field| "#{field}_#{@selected_translation_locale}" }
+        klass.translatable_fields.map { |field| "#{field}_#{normalized_locale(@selected_translation_locale)}" }
       end
 
+      # Determine the class of the resource
       def resource_class
         @resource_class ||= begin
           klass = params[:resource_type]
@@ -46,10 +62,12 @@ module Spree
         end
       end
 
+      # Allowed translatable resources configured in Spree
       def allowed_resource_class
         Rails.application.config.spree.translatable_resources
       end
 
+      # Set the resource object using friendly find if available
       def set_resource
         @resource = if resource_class.respond_to?(:friendly)
                       resource_class.friendly.find(params[:id])
@@ -58,10 +76,17 @@ module Spree
                     end
       end
 
+      # Determine the translation locale to use
       def set_translation_locale
-        @selected_translation_locale = params[:translation_locale].presence || @locales&.first || current_store.supported_locales_list.first
+        raw_locale = params[:translation_locale].presence ||
+        @locales&.first ||
+        current_store.default_locale ||
+        current_store.supported_locales_list.first
+
+        @selected_translation_locale = normalized_locale(raw_locale)
       end
 
+      # Load available locales for this resource, excluding default
       def load_data
         @locales = (current_store.supported_locales_list - [@default_locale]).sort
         @resource_name = @resource.try(:name)

data/app/helpers/spree/admin/orders_helper.rb

@@ -20,7 +20,15 @@ module Spree
       def order_payment_state(order, options = {})
         return if order.payment_state.blank?
 
-        content_tag :span, class: "badge  #{options[:class]} badge-#{order.partially_refunded? ? 'warning' : order.payment_state}" do
+        badge_class = if order.order_refunded?
+                        'badge-danger'
+                      elsif order.partially_refunded?
+                        'badge-warning'
+                      else
+                        "badge-#{order.payment_state}"
+                      end
+
+        content_tag :span, class: "badge #{options[:class]} #{badge_class}" do
           if order.order_refunded?
             icon('credit-card-refund') + Spree.t('payment_states.refunded')
           elsif order.partially_refunded?

data/app/views/spree/admin/gift_cards/show.html.erb

@@ -67,7 +67,7 @@
                       <% end %>
                     </td>
                     <td>
-                      <%= order.user.email %>
+                      <%= render 'spree/admin/orders/customer_summary', order: order %>
                     </td>
                     <td class="text-right">
                       <%= order.display_gift_card_total %>

data/app/views/spree/admin/promotion_rules/forms/_currency.html.erb

@@ -1,9 +1,6 @@
 <div class="form-group">
   <%= f.select :preferred_currency,
-    currency_options_for_select(
-      f.object.preferred_currency,
-      current_store.supported_currencies.split(',')
-    ),
+    options_for_select(supported_currency_options, f.object.preferred_currency),
     {},
     { data: { controller: 'autocomplete-select' } } 
   %>

data/app/views/spree/admin/shared/_head.html.erb

@@ -37,9 +37,13 @@
 <%= tinymce_assets %>
 <link rel="stylesheet" type="text/css" href="https://unpkg.com/trix@2.1.12/dist/trix.css">
 
-<script async src="https://ga.jspm.io/npm:es-module-shims@1.8.2/dist/es-module-shims.js" data-turbo-track="reload"></script>
+<% if Rails.application.importmap.packages["application"].present? && Spree::Admin::RuntimeConfig.include_application_importmap %>
+  <%= javascript_importmap_tags "application" %>
+<% else %>
+  <script async src="https://ga.jspm.io/npm:es-module-shims@1.8.2/dist/es-module-shims.js" data-turbo-track="reload"></script>
+<% end %>
 
-<%= javascript_importmap_tags 'application-spree-admin', importmap: Rails.application.config.spree_admin.importmap %>
+<%= javascript_importmap_tags "application-spree-admin", importmap: Rails.application.config.spree_admin.importmap %>
 
 <%= yield :head %>
 

data/app/views/spree/admin/store_credits/_form.html.erb

@@ -6,11 +6,11 @@
 
 <div class="form-group">
   <%= f.label :currency, raw(Spree.t(:currency) + required_span_tag), class: 'form-label' %>
-  <%= f.select :currency, preferred_currencies_select_options, {}, { data: { controller: 'autocomplete-select', enable_button_target: 'input', store_form_target: 'currency' } } %>
+  <%= f.select :currency, options_for_select(supported_currency_options, f.object.currency), {}, { data: { controller: 'autocomplete-select', enable_button_target: 'input', store_form_target: 'currency' } } %>
 </div>
 
 <div class="form-group">
 <%= f.label :memo, Spree.t(:memo) %>
 <%= f.text_area :memo, class: 'form-control' %>
 <%= f.error_message_on :memo %>
-</div>
+</div>

data/config/importmap.rb

@@ -7,7 +7,7 @@ pin "@hotwired/stimulus-loading", to: "stimulus-loading.js"
 pin '@rails/actioncable', to: 'actioncable.esm.js', preload: ['application-spree-admin']
 pin '@rails/activestorage', to: 'activestorage.esm.js', preload: ['application-spree-admin']
 pin '@rails/actiontext', to: '@rails--actiontext@7.2.201.js', preload: ['application-spree-admin']
-pin 'trix', to: 'trix@2.1.12.js', preload: ['application-spree-admin']
+pin 'trix', to: 'trix@2.1.15.js', preload: ['application-spree-admin']
 
 pin 'sortablejs', preload: ['application-spree-admin'] # @1.15.6
 pin 'chartkick', to: 'chartkick.js', preload: ['application-spree-admin']

data/lib/spree/admin/runtime_configuration.rb

@@ -7,6 +7,8 @@ module Spree
       preference :admin_records_per_page, :integer, default: Kaminari.config.default_per_page
       preference :admin_products_per_page, :integer, default: Kaminari.config.default_per_page
       preference :admin_orders_per_page, :integer, default: Kaminari.config.default_per_page
+
+      preference :include_application_importmap, :boolean, default: false
     end
   end
 end