spree_admin 5.3.3 → 5.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f53aab5bf5de0f40206a80c2a2375cc915973139dec1a0ca669a36f4d98a6469
-  data.tar.gz: c3c5c6babf054c33dc7027464c9c38bc5e27eb1fcb41d7ff4891a223291dcde0
+  metadata.gz: 50a1165ee32d40280a53a301cc00059012380a2c0fd27e5ef6933ee074bd50b8
+  data.tar.gz: 4282e3887a4f1a32e13ec9611cf9d02377c5fd98ec153361d0f635bbd25e64b0
 SHA512:
-  metadata.gz: 30e6161df15f8ba37bfe8b41c7920f63352f344cf7176941a44ce0d43952347ac22aca27d2ddb9209e6423bebb5f089b0b9c9cf51238752ff25552e9ca8440f6
-  data.tar.gz: 6a14594a4adf0b59a58f0dcae7607e0e65bc945bd99134895889ba312e26a56702712f25f148879b48d250d2edb27c026f89d9ad2b3824a6aac50930852d9c77
+  metadata.gz: 74520ef7ba4d6f284ecf05f024bdfb835ccfa27b91fb564330bd5d45c42c857527d24adc0aa76e54403685d3387280ebfc431205544f59e34cc1318575ad3adc
+  data.tar.gz: d1fc1d04175d7baed961400407b43a9efa040a79f7e94d3902597bce0712258df8418737a4ba9539409350ed73933e9d435c4c65f6339c50a2a68df727a7c660

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

@@ -230,7 +230,7 @@ module Spree
       # @return [String] JSON string
       def query_builder_fields_json(table)
         query_builder = Spree::Admin::Table::QueryBuilder.new(table)
-        query_builder.available_fields.to_json
+        query_builder.available_fields(self).to_json
       end
 
       # Build available operators JSON for Stimulus controller

data/app/javascript/spree/admin/controllers/bulk_editor_controller.js

@@ -79,21 +79,16 @@ export default class extends Controller {
     this.boundHandleBeforeUnload = this.handleBeforeUnload.bind(this)
     this.boundHandleMouseUp = this.handleMouseUp.bind(this)
     this.boundHandleMouseMove = this.handleMouseMove.bind(this)
-    this.boundNormalizeBeforeSubmit = this.normalizeBeforeSubmit.bind(this)
 
     window.addEventListener('beforeunload', this.boundHandleBeforeUnload)
     document.addEventListener('mouseup', this.boundHandleMouseUp)
     document.addEventListener('mousemove', this.boundHandleMouseMove)
-
-    // Listen for form submission to normalize values
-    this.element.addEventListener('submit', this.boundNormalizeBeforeSubmit)
   }
 
   disconnect() {
     window.removeEventListener('beforeunload', this.boundHandleBeforeUnload)
     document.removeEventListener('mouseup', this.boundHandleMouseUp)
     document.removeEventListener('mousemove', this.boundHandleMouseMove)
-    this.element.removeEventListener('submit', this.boundNormalizeBeforeSubmit)
   }
 
   // ==================== Fill Handle ====================
@@ -783,85 +778,4 @@ export default class extends Controller {
     return this.dirtyInputs.size > 0
   }
 
-  // ==================== Locale-aware Number Handling ====================
-
-  /**
-   * Normalizes a locale-formatted number string to standard decimal format
-   * e.g., "1.234,56" (German) -> "1234.56"
-   * e.g., "1,234.56" (English) -> "1234.56"
-   * @param {string} value - The locale-formatted number string
-   * @returns {string} The normalized number string with "." as decimal separator
-   */
-  normalizeNumber(value) {
-    if (value === null || value === undefined) return ''
-
-    let stringValue = String(value).trim()
-    if (stringValue === '') return ''
-
-    // Detect the decimal separator by finding the last separator character
-    // This handles both "1,234.56" (en) and "1.234,56" (de/pl) formats
-    const lastComma = stringValue.lastIndexOf(',')
-    const lastDot = stringValue.lastIndexOf('.')
-
-    let decimalSeparator = '.'
-    let thousandsSeparator = ','
-
-    // If comma comes after dot, comma is the decimal separator (European format)
-    // Also treat comma as decimal if there's no dot and comma has 1-2 digits after it
-    if (lastComma > lastDot) {
-      decimalSeparator = ','
-      thousandsSeparator = '.'
-    } else if (lastDot === -1 && lastComma !== -1) {
-      // No dot present, check if comma looks like a decimal separator
-      // (has 1-3 digits after it, typical for currency)
-      const afterComma = stringValue.substring(lastComma + 1)
-      if (/^\d{1,3}$/.test(afterComma)) {
-        decimalSeparator = ','
-        thousandsSeparator = '.'
-      }
-    }
-
-    // Remove thousands separators
-    stringValue = stringValue.split(thousandsSeparator).join('')
-
-    // Replace decimal separator with standard "."
-    if (decimalSeparator !== '.') {
-      stringValue = stringValue.replace(decimalSeparator, '.')
-    }
-
-    // Remove any non-numeric characters except "." and "-"
-    stringValue = stringValue.replace(/[^0-9.\-]/g, '')
-
-    return stringValue
-  }
-
-  /**
-   * Formats a number for display using the configured locale
-   * @param {number|string} value - The number to format
-   * @returns {string} The formatted number string
-   */
-  formatNumber(value) {
-    if (value === null || value === undefined || value === '') return ''
-
-    const number = parseFloat(value)
-    if (!Number.isFinite(number)) return ''
-
-    return number.toLocaleString(this.localeValue, {
-      minimumFractionDigits: 2,
-      maximumFractionDigits: 2,
-      useGrouping: false
-    })
-  }
-
-  /**
-   * Normalizes all cell values before form submission
-   * @param {Event} event - The form submit event
-   */
-  normalizeBeforeSubmit(event) {
-    this.cellTargets.forEach((cell) => {
-      if (cell.value) {
-        cell.value = this.normalizeNumber(cell.value)
-      }
-    })
-  }
 }

data/app/javascript/spree/admin/controllers/money_field_controller.js

@@ -4,11 +4,10 @@ import { Controller } from '@hotwired/stimulus'
  * MoneyFieldController
  *
  * A Stimulus controller for locale-aware money/price input fields.
- * Handles formatting for display and normalizing values before form submission.
+ * Handles formatting values for display in the user's locale format.
  *
  * Features:
  * - Displays amounts in the user's locale format (e.g., "1.234,56" for German, "1,234.56" for English)
- * - Normalizes values to standard decimal format (with "." as decimal separator) before form submission
  * - Supports optional currency symbol display
  *
  * Usage:
@@ -32,23 +31,10 @@ export default class extends Controller {
   }
 
   connect() {
-    this.form = this.element.closest('form')
-
-    if (this.form) {
-      this.boundNormalizeBeforeSubmit = this.normalizeBeforeSubmit.bind(this)
-      this.form.addEventListener('submit', this.boundNormalizeBeforeSubmit)
-    }
-
     // Format the initial value for display
     this.formatForDisplay()
   }
 
-  disconnect() {
-    if (this.form && this.boundNormalizeBeforeSubmit) {
-      this.form.removeEventListener('submit', this.boundNormalizeBeforeSubmit)
-    }
-  }
-
   /**
    * Called on blur to format the value for display
    */
@@ -139,12 +125,4 @@ export default class extends Controller {
     return stringValue
   }
 
-  /**
-   * Normalizes the value before form submission
-   * @param {Event} event - The form submit event
-   */
-  normalizeBeforeSubmit(event) {
-    const normalizedValue = this.normalizeValue(this.element.value)
-    this.element.value = normalizedValue
-  }
 }

data/app/javascript/spree/admin/controllers/variants_form_controller.js

@@ -33,7 +33,6 @@ export default class extends CheckboxSelectAll {
     prices: Object,
     currentCurrency: String,
     currencies: Array,
-    currencyFormats: Object,
     variantIds: Object,
     currentStockLocationId: String,
     stockLocations: Array,
@@ -71,19 +70,6 @@ export default class extends CheckboxSelectAll {
     }
 
     this.inventoryFormTarget = document.querySelector('.inventory-form');
-
-    // Add form submit listener to normalize price inputs before submission
-    this.form = this.element.closest('form')
-    if (this.form) {
-      this.boundNormalizePricesBeforeSubmit = this.normalizePricesBeforeSubmit.bind(this)
-      this.form.addEventListener('submit', this.boundNormalizePricesBeforeSubmit)
-    }
-  }
-
-  disconnect() {
-    if (this.form && this.boundNormalizePricesBeforeSubmit) {
-      this.form.removeEventListener('submit', this.boundNormalizePricesBeforeSubmit)
-    }
   }
 
   toggleQuantityTracked() {
@@ -387,6 +373,16 @@ export default class extends CheckboxSelectAll {
     }
   }
 
+  formatPrice(event) {
+    const value = event.target.value
+    if (value === '' || value === null) return
+
+    const number = this.parseLocaleNumber(value)
+    if (!Number.isFinite(number)) return
+
+    event.target.value = this.formatNumber(number)
+  }
+
   replaceBlankWithZero(event) {
     if (event.target.value === '') {
       event.target.value = 0
@@ -1100,13 +1096,13 @@ export default class extends CheckboxSelectAll {
     if (existingPrice) {
       return {
         ...existingPrice,
-        amount: existingPrice.amount ? parseFloat(existingPrice.amount) : existingPrice.amount
+        amount: existingPrice.amount ? this.parseLocaleNumber(existingPrice.amount) : existingPrice.amount
       }
     } else {
       const parentName = variantName.split('/')[0]
       const parentPrices = Object.entries(this.pricesValue)
         .filter(([internalName, prices]) => internalName.startsWith(parentName) && prices[currency.toLowerCase()] !== undefined)
-        .map(([_key, prices]) => parseFloat(prices[currency.toLowerCase()].amount))
+        .map(([_key, prices]) => this.parseLocaleNumber(prices[currency.toLowerCase()].amount))
         .sort((priceAmountA, priceAmountB) => priceAmountA - priceAmountB)
 
       return {
@@ -1124,7 +1120,7 @@ export default class extends CheckboxSelectAll {
         ...this.pricesValue[variantName],
         [currency.toLowerCase()]: {
           ...existingPrice,
-          amount: parseFloat(newPrice)
+          amount: this.parseLocaleNumber(newPrice)
         }
       }
     }
@@ -1148,23 +1144,13 @@ export default class extends CheckboxSelectAll {
     }
   }
 
-  formatNumber(value, currency = null) {
+  formatNumber(value) {
     if (value === null) return ''
     if (typeof value === 'string' && value.trim() === '') return ''
 
     const number = Number(value)
     if (!Number.isFinite(number)) return ''
 
-    // Use currency-specific decimal mark if available
-    const currencyCode = currency || this.currentCurrencyValue
-    const currencyFormat = this.currencyFormatsValue?.[currencyCode]
-
-    if (currencyFormat?.decimal_mark) {
-      // Format with 2 decimal places and replace . with currency's decimal mark
-      return number.toFixed(2).replace('.', currencyFormat.decimal_mark)
-    }
-
-    // Fallback to locale-based formatting
     return number.toLocaleString(this.localeValue, {
       minimumFractionDigits: 2,
       maximumFractionDigits: 2,
@@ -1172,70 +1158,22 @@ export default class extends CheckboxSelectAll {
     })
   }
 
-  /**
-   * Normalizes a locale-formatted number string to standard decimal format
-   * e.g., "1.234,56" (German) -> "1234.56"
-   * e.g., "30,99" (Polish) -> "30.99"
-   * @param {string} value - The locale-formatted number string
-   * @returns {string} The normalized number string with "." as decimal separator
-   */
-  normalizeNumber(value) {
-    if (value === null || value === undefined) return ''
-
-    let stringValue = String(value).trim()
-    if (stringValue === '') return ''
-
-    // Detect the decimal separator by finding the last separator character
-    // This handles both "1,234.56" (en) and "1.234,56" (de/pl) formats
-    const lastComma = stringValue.lastIndexOf(',')
-    const lastDot = stringValue.lastIndexOf('.')
-
-    let decimalSeparator = '.'
-    let thousandsSeparator = ','
-
-    // If comma comes after dot, comma is the decimal separator (European format)
-    // Also treat comma as decimal if there's no dot and comma has 1-3 digits after it
-    if (lastComma > lastDot) {
-      decimalSeparator = ','
-      thousandsSeparator = '.'
-    } else if (lastDot === -1 && lastComma !== -1) {
-      // No dot present, check if comma looks like a decimal separator
-      // (has 1-3 digits after it, typical for currency)
-      const afterComma = stringValue.substring(lastComma + 1)
-      if (/^\d{1,3}$/.test(afterComma)) {
-        decimalSeparator = ','
-        thousandsSeparator = '.'
-      }
-    }
+  parseLocaleNumber(value) {
+    if (value === null || value === undefined) return NaN
 
-    // Remove thousands separators
-    stringValue = stringValue.split(thousandsSeparator).join('')
+    let str = String(value).trim()
+    if (str === '') return NaN
 
-    // Replace decimal separator with standard "."
-    if (decimalSeparator !== '.') {
-      stringValue = stringValue.replace(decimalSeparator, '.')
-    }
+    const lastComma = str.lastIndexOf(',')
+    const lastDot = str.lastIndexOf('.')
 
-    // Remove any non-numeric characters except "." and "-"
-    stringValue = stringValue.replace(/[^0-9.\-]/g, '')
+    if (lastComma > lastDot) {
+      str = str.replace(/\./g, '').replace(',', '.')
+    } else if (lastDot === -1 && lastComma !== -1 && /^\d{1,3}$/.test(str.substring(lastComma + 1))) {
+      str = str.replace(',', '.')
+    }
 
-    return stringValue
+    return parseFloat(str)
   }
 
-  /**
-   * Normalizes all price inputs in the variants form before submission
-   * @param {Event} event - The form submit event
-   */
-  normalizePricesBeforeSubmit(event) {
-    // Find all price inputs in the variants container
-    const priceInputs = this.variantsContainerTarget.querySelectorAll(
-      'input[data-slot*="[prices_attributes]"][data-slot*="[amount]_input"]'
-    )
-
-    priceInputs.forEach((input) => {
-      if (input.value) {
-        input.value = this.normalizeNumber(input.value)
-      }
-    })
-  }
 }

data/app/models/spree/admin/table/column.rb

@@ -42,7 +42,7 @@ module Spree
         attribute :ransack_attribute, :string
         attribute :operators, default: -> { [] }
         attribute :value_options
-        attribute :search_url, :string
+        attribute :search_url
         attribute :sort_scope_asc
         attribute :sort_scope_desc
 

data/app/models/spree/admin/table/query_builder.rb

@@ -70,8 +70,9 @@ module Spree
         end
 
         # Get available fields for filtering based on table configuration
+        # @param view_context [ActionView::Base] optional view context for resolving dynamic URLs
         # @return [Array<Hash>]
-        def available_fields
+        def available_fields(view_context = nil)
           @table.filterable_columns.map do |column|
             {
               key: column.ransack_attribute,
@@ -79,7 +80,7 @@ module Spree
               type: column.filter_type.to_s,
               operators: column.operators.map(&:to_s),
               value_options: format_value_options(column.value_options),
-              search_url: column.search_url
+              search_url: resolve_search_url(column.search_url, view_context)
             }
           end
         end
@@ -92,6 +93,10 @@ module Spree
 
         private
 
+        def resolve_search_url(search_url, view_context)
+          search_url.is_a?(Proc) ? search_url.call(view_context) : search_url
+        end
+
         def format_value_options(options)
           return nil if options.blank?
 

data/app/views/spree/admin/products/form/_variants.html.erb

@@ -5,21 +5,11 @@
     </div>
     <% option_types_for_select = @option_types.pluck(:presentation, :id) %>
     <div class="card-body p-0">
-      <%# Build currency format info for JavaScript %>
-      <% currency_formats = supported_currencies.each_with_object({}) do |currency, hash|
-           currency_obj = ::Money::Currency.find(currency)
-           hash[currency.to_s] = {
-             decimal_mark: currency_obj.decimal_mark,
-             thousands_separator: currency_obj.thousands_separator,
-             symbol: currency_obj.symbol
-           }
-         end %>
       <div
         data-controller="variants-form"
         class="variants-form"
         data-action="product-form:toggle-quantity-tracked@window->variants-form#toggleQuantityTracked"
         data-variants-form-locale-value="<%= I18n.locale %>"
-        data-variants-form-currency-formats-value="<%= currency_formats.to_json %>"
         <% if @product.persisted? %> data-variants-form-product-id-value="<%= @product.slug %>" <% end %>
         data-variants-form-current-currency-value="<%= current_currency %>"
         data-variants-form-currencies-value="<%= supported_currencies.map(&:to_s).to_json %>"

data/app/views/spree/admin/products/form/variants/_variant_template.html.erb

@@ -18,7 +18,7 @@
     <div class="variants-table__body__cell column-price mr-2">
       <% supported_currencies.each_with_index do |currency, i| %>
         <div class="input-group price-input-container <%= current_currency != currency ? 'hidden' : 'flex' %>" <%= !can_manage_prices ? 'readonly' : '' %>>
-          <%= text_field_tag "product[variants_attributes][prices_attributes][#{i}][amount]", '',  class: 'border-0 focus:ring-0 focus:outline-none', data: { slot: "[prices_attributes][#{currency}][amount]_input", action: 'variants-form#updatePrice', currency: currency.to_s }, readonly: !can_manage_prices %>
+          <%= text_field_tag "product[variants_attributes][prices_attributes][#{i}][amount]", '',  class: 'border-0 focus:ring-0 focus:outline-none', data: { slot: "[prices_attributes][#{currency}][amount]_input", action: 'variants-form#updatePrice blur->variants-form#formatPrice', currency: currency.to_s }, readonly: !can_manage_prices %>
           <span class="px-3"><%= currency_symbol(currency) %></span>
         </div>
         <%= hidden_field_tag "product[variants_attributes][prices_attributes][#{i}][currency]", currency, data: {slot: "[prices_attributes][#{currency}][currency]_input"} %>

data/app/views/spree/admin/tables/_table.html.erb

@@ -6,11 +6,10 @@
 <% search_placeholder = (table.respond_to?(:search_placeholder) && table.search_placeholder) || [Spree.t(:search), controller_name.humanize.downcase].join(' ') %>
 <% date_range_param = table.date_range_param %>
 
-<% if export_type.present? %>
-  <%= render 'spree/admin/shared/export_modal', export_type: export_type %>
-<% end %>
-
 <%= turbo_frame_tag frame_name, data: { turbo_action: 'advance' } do %>
+  <% if export_type.present? %>
+    <%= render 'spree/admin/shared/export_modal', export_type: export_type %>
+  <% end %>
   <div class="<%= local_assigns[:container_class] || 'card-lg' %>" data-controller="table" data-table-url-value="<%= request.path %>">
     <div class="p-3 border-b border-gray-200">
       <div class="flex flex-col lg:flex-row gap-3 items-start lg:items-center justify-between">

data/config/initializers/spree_admin_tables.rb

@@ -111,7 +111,7 @@ Rails.application.config.after_initialize do
                                         position: 80,
                                         ransack_attribute: 'taxons_id',
                                         operators: %i[in],
-                                        search_url: '/admin/taxons/select_options.json',
+                                        search_url: ->(view_context) { view_context.spree.admin_taxons_select_options_path(format: :json) },
                                         method: ->(product) { product.taxons.pluck(:pretty_name).to_sentence if product.classification_count.positive? }
 
   # Tags - displayed as comma-separated list, filtered via autocomplete
@@ -125,14 +125,14 @@ Rails.application.config.after_initialize do
                                         position: 85,
                                         ransack_attribute: 'tags_name',
                                         operators: %i[in],
-                                        search_url: '/admin/tags/select_options.json?taggable_type=Spree::Product',
+                                        search_url: ->(view_context) { view_context.spree.admin_tags_select_options_path(format: :json, taggable_type: 'Spree::Product') },
                                         method: ->(product) { product.tag_list.to_sentence }
 
   # Products bulk actions
   Spree.admin.tables.products.add_bulk_action :set_active,
                                                     label: 'admin.bulk_ops.products.title.set_active',
                                                     icon: 'circle-check',
-                                                    action_path: '/admin/products/bulk_status_update?status=active',
+                                                    action_path: ->(view_context) { view_context.spree.bulk_status_update_admin_products_path(status: 'active') },
                                                     body: 'admin.bulk_ops.products.body.set_active',
                                                     position: 10,
                                                     condition: -> { can?(:activate, Spree::Product) }
@@ -140,21 +140,21 @@ Rails.application.config.after_initialize do
   Spree.admin.tables.products.add_bulk_action :set_draft,
                                                     label: 'admin.bulk_ops.products.title.set_draft',
                                                     icon: 'circle-dotted',
-                                                    action_path: '/admin/products/bulk_status_update?status=draft',
+                                                    action_path: ->(view_context) { view_context.spree.bulk_status_update_admin_products_path(status: 'draft') },
                                                     body: 'admin.bulk_ops.products.body.set_draft',
                                                     position: 20
 
   Spree.admin.tables.products.add_bulk_action :set_archived,
                                                     label: 'admin.bulk_ops.products.title.set_archived',
                                                     icon: 'archive',
-                                                    action_path: '/admin/products/bulk_status_update?status=archived',
+                                                    action_path: ->(view_context) { view_context.spree.bulk_status_update_admin_products_path(status: 'archived') },
                                                     body: 'admin.bulk_ops.products.body.set_archived',
                                                     position: 30
 
   Spree.admin.tables.products.add_bulk_action :add_to_taxons,
                                                     label: 'admin.bulk_ops.products.title.add_to_taxons',
                                                     icon: 'category-plus',
-                                                    action_path: '/admin/products/bulk_add_to_taxons',
+                                                    action_path: ->(view_context) { view_context.spree.bulk_add_to_taxons_admin_products_path },
                                                     body: 'admin.bulk_ops.products.body.add_to_taxons',
                                                     form_partial: 'spree/admin/bulk_operations/forms/taxon_picker',
                                                     position: 40,
@@ -163,7 +163,7 @@ Rails.application.config.after_initialize do
   Spree.admin.tables.products.add_bulk_action :remove_from_taxons,
                                                     label: 'admin.bulk_ops.products.title.remove_from_taxons',
                                                     icon: 'category-minus',
-                                                    action_path: '/admin/products/bulk_remove_from_taxons',
+                                                    action_path: ->(view_context) { view_context.spree.bulk_remove_from_taxons_admin_products_path },
                                                     body: 'admin.bulk_ops.products.body.remove_from_taxons',
                                                     form_partial: 'spree/admin/bulk_operations/forms/taxon_picker',
                                                     position: 50,
@@ -172,7 +172,7 @@ Rails.application.config.after_initialize do
   Spree.admin.tables.products.add_bulk_action :add_tags,
                                                     label: 'admin.bulk_ops.products.title.add_tags',
                                                     icon: 'tag-plus',
-                                                    action_path: '/admin/products/bulk_add_tags',
+                                                    action_path: ->(view_context) { view_context.spree.bulk_add_tags_admin_products_path },
                                                     body: 'admin.bulk_ops.products.body.add_tags',
                                                     form_partial: 'spree/admin/bulk_operations/forms/tag_picker',
                                                     form_partial_locals: { allow_create: true },
@@ -182,7 +182,7 @@ Rails.application.config.after_initialize do
   Spree.admin.tables.products.add_bulk_action :remove_tags,
                                                     label: 'admin.bulk_ops.products.title.remove_tags',
                                                     icon: 'tag-minus',
-                                                    action_path: '/admin/products/bulk_remove_tags',
+                                                    action_path: ->(view_context) { view_context.spree.bulk_remove_tags_admin_products_path },
                                                     body: 'admin.bulk_ops.products.body.remove_tags',
                                                     form_partial: 'spree/admin/bulk_operations/forms/tag_picker',
                                                     form_partial_locals: { allow_create: false },
@@ -348,7 +348,7 @@ Rails.application.config.after_initialize do
                                       position: 150,
                                       ransack_attribute: 'promotions_id',
                                       operators: %i[in],
-                                      search_url: '/admin/promotions/select_options.json'
+                                      search_url: ->(view_context) { view_context.spree.select_options_admin_promotions_path(format: :json) }
 
   # Register Checkouts table (draft orders)
   Spree.admin.tables.register(:checkouts, model_class: Spree::Order, search_param: :multi_search, date_range_param: :created_at, new_resource: false)
@@ -462,7 +462,7 @@ Rails.application.config.after_initialize do
                                      position: 30,
                                      ransack_attribute: 'addresses_country_name',
                                      operators: %i[eq],
-                                     search_url: '/admin/countries/select_options.json',
+                                     search_url: ->(view_context) { view_context.spree.admin_countries_select_options_path(format: :json) },
                                      partial: 'spree/admin/tables/columns/user_location'
 
   # Number of orders
@@ -533,13 +533,13 @@ Rails.application.config.after_initialize do
                                      position: 90,
                                      ransack_attribute: 'tags_name',
                                      operators: %i[in],
-                                     search_url: '/admin/tags/select_options.json?taggable_type=Spree::User'
+                                     search_url: ->(view_context) { view_context.spree.admin_tags_select_options_path(format: :json, taggable_type: 'Spree::User') }
 
   # Users bulk actions
   Spree.admin.tables.users.add_bulk_action :add_tags,
                                                  label: 'admin.bulk_ops.users.title.add_tags',
                                                  icon: 'tag-plus',
-                                                 action_path: '/admin/users/bulk_add_tags',
+                                                 action_path: ->(view_context) { view_context.spree.bulk_add_tags_admin_users_path },
                                                  body: 'admin.bulk_ops.users.body.add_tags',
                                                  form_partial: 'spree/admin/bulk_operations/forms/tag_picker',
                                                  form_partial_locals: { allow_create: true },
@@ -550,7 +550,7 @@ Rails.application.config.after_initialize do
   Spree.admin.tables.users.add_bulk_action :remove_tags,
                                                  label: 'admin.bulk_ops.users.title.remove_tags',
                                                  icon: 'tag-minus',
-                                                 action_path: '/admin/users/bulk_remove_tags',
+                                                 action_path: ->(view_context) { view_context.spree.bulk_remove_tags_admin_users_path },
                                                  body: 'admin.bulk_ops.users.body.remove_tags',
                                                  form_partial: 'spree/admin/bulk_operations/forms/tag_picker',
                                                  form_partial_locals: { allow_create: false },
@@ -1080,7 +1080,7 @@ Rails.application.config.after_initialize do
                                           position: 70,
                                           ransack_attribute: 'user_id',
                                           operators: %i[eq],
-                                          search_url: '/admin/users/select_options.json',
+                                          search_url: ->(view_context) { view_context.spree.admin_users_select_options_path(format: :json) },
                                           method: ->(gc) { gc.user&.email }
 
   Spree.admin.tables.gift_cards.add :created_at,
@@ -1125,7 +1125,7 @@ Rails.application.config.after_initialize do
                                            position: 20,
                                            ransack_attribute: 'stock_location_id',
                                            operators: %i[eq],
-                                           search_url: '/admin/stock_locations/select_options.json',
+                                           search_url: ->(view_context) { view_context.spree.admin_stock_locations_select_options_path(format: :json) },
                                            partial: 'spree/admin/tables/columns/stock_item_location'
 
   # Backorderable (inline editable checkbox)
@@ -1198,7 +1198,7 @@ Rails.application.config.after_initialize do
                                      position: 20,
                                      method: ->(post) { post.post_category_title },
                                      ransack_attribute: 'post_category_id',
-                                     search_url: '/admin/post_categories/select_options.json'
+                                     search_url: ->(view_context) { view_context.spree.select_options_admin_post_categories_path(format: :json) }
 
   Spree.admin.tables.posts.add :author,
                                      label: :author,
@@ -1210,7 +1210,7 @@ Rails.application.config.after_initialize do
                                      position: 30,
                                      method: ->(post) { post.author_name },
                                      ransack_attribute: 'author_id',
-                                     search_url: '/admin/admin_users/select_options.json'
+                                     search_url: ->(view_context) { view_context.spree.select_options_admin_admin_users_path(format: :json) }
 
   Spree.admin.tables.posts.add :published_at,
                                      label: :published_at,

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: spree_admin
 version: !ruby/object:Gem::Version
-  version: 5.3.3
+  version: 5.3.4
 platform: ruby
 authors:
 - Vendo Connect Inc.
@@ -15,14 +15,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.3.3
+        version: 5.3.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 5.3.3
+        version: 5.3.4
 - !ruby/object:Gem::Dependency
   name: active_link_to
   requirement: !ruby/object:Gem::Requirement
@@ -1257,9 +1257,9 @@ licenses:
 - AGPL-3.0-or-later
 metadata:
   bug_tracker_uri: https://github.com/spree/spree/issues
-  changelog_uri: https://github.com/spree/spree/releases/tag/v5.3.3
+  changelog_uri: https://github.com/spree/spree/releases/tag/v5.3.4
   documentation_uri: https://docs.spreecommerce.org/
-  source_code_uri: https://github.com/spree/spree/tree/v5.3.3
+  source_code_uri: https://github.com/spree/spree/tree/v5.3.4
 rdoc_options: []
 require_paths:
 - lib