solidus_backend 3.2.9 → 3.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0ebd88037c534ab114b9df0719ef4648d5572501a4adfb326e7a10a58b0490a5
-  data.tar.gz: 4eae2ee341af874256562f9584e5ccd78736b6f9a33d7cfa7c4310571f991544
+  metadata.gz: 167281f642dc52cabba45304f1ec89744ea6583885fc53783303a88d949d83e6
+  data.tar.gz: 1a49a77a9e9c3dedbe2fedbfb5877a61c5fade7e435df5a28b88755bcf8cb6ed
 SHA512:
-  metadata.gz: 5a4dac579a6a7a6e25c714bb6f6c4e1bf60592c714904f533598907651bef8d563a728a0e0a2216ae34b8946dc8dbd7ab62c29a1d985029e28981aa804cfee01
-  data.tar.gz: 640d77bb6c592c45c3abac26201ca8ab6e5c38a5b909751eb0c18fa62ac342745fb9fc20815367afdfb4c1bad1e4bb1d61f729128ded682677edcc6767a946a7
+  metadata.gz: 4715eaba5c97daa199678e78498cb606bf1ce3cfaffe9b209c0dd388175454d91f9c46666c5143dff8244342fc1aa1061dbe18b202482c1e1a8e9531263fbb21
+  data.tar.gz: 5f9bd79cd6ce2c349bbe553889d00ba0fdf0dc3fd06961e8f72cc9ff208db8a427eaac33d038b774d840d074cd34300faef41f46558f101b7179fd354014a0df

data/app/assets/javascripts/spree/backend/orders/edit.js

@@ -1,6 +1,8 @@
 Spree.ready(function () {
   'use strict';
 
-  $('[data-hook="add_product_name"]').find('.variant_autocomplete').variantAutocomplete({ suppliable_only: true });
+  $('[data-hook="add_product_name"]').find('.variant_autocomplete').variantAutocomplete({
+    searchParameters: function (_term) { return { suppliable_only: true } }
+  });
   $("[data-hook='admin_orders_index_search']").find(".variant_autocomplete").variantAutocomplete();
 });

data/app/assets/javascripts/spree/backend/product_picker.js

@@ -1,9 +1,32 @@
+/**
+  * Make the element a select2 dropdown used for finding Products. By default,
+  * it allows the Products to be found by its name and its Variants' SKUs.
+  * @param  {Object} options Options
+  * @param  {Boolean} [options.multiple=true] Allow multiple products to be selectable
+  * @param  {Function|undefined} options.searchParameters Returns a hash object for params to merge on the select2 ajax request
+  *                                                       Accepts an argument of the select2 search term. To use custom Ransack
+  *                                                       define q on the hash and add your custom terms. Note, you need to
+  *                                                       ensure that the attributes are allowed to be Ransacked.
+  */
 $.fn.productAutocomplete = function (options) {
   'use strict';
 
   // Default options
   options = options || {}
   var multiple = typeof(options['multiple']) !== 'undefined' ? options['multiple'] : true
+  function extraParameters(term) {
+    if (typeof(options['searchParameters']) === 'function') {
+      return options['searchParameters'](term)
+    }
+
+    return {
+      q: {
+        name_cont: term,
+        variants_including_master_sku_start: term,
+        m: 'or'
+      }
+    }
+  }
 
   function formatProduct(product) {
     return Select2.util.escapeMarkup(product.name);
@@ -26,15 +49,11 @@ $.fn.productAutocomplete = function (options) {
       datatype: 'json',
       params: { "headers": {  'Authorization': 'Bearer ' + Spree.api_key } },
       data: function (term, page) {
-        return {
-          q: {
-            name_cont: term,
-            variants_including_master_sku_start: term,
-            m: 'or'
-          },
+        const params = {
           token: Spree.api_key,
           page: page
         };
+        return _.extend(params, extraParameters(term));
       },
       results: function (data, page) {
         var products = data.products ? data.products : [];

data/app/assets/javascripts/spree/backend/variant_autocomplete.js

@@ -7,10 +7,38 @@
     });
   };
 
-  $.fn.variantAutocomplete = function(searchOptions) {
-    if (searchOptions == null) {
-      searchOptions = {};
+  /**
+    * Make the element a select2 dropdown used for finding Variants. By default, the search term will be
+    * passed to the defined Spree::Config.variant_search_class by the controller with its defined scope.
+    * @param  {Object|undefined|null} options Options
+    * @param  {Function|undefined} options.searchParameters Returns a hash object for params to merge on the select2 ajax request
+    *                                                       Accepts an argument of the select2 search term. To use Ransack, define
+    *                                                       variant_search_term as a falsy value, and q as the Ransack query. Note,
+    *                                                       you need to ensure that the attributes are allowed to be Ransacked.
+    */
+  $.fn.variantAutocomplete = function(options) {
+    function extraParameters(term) {
+      if (typeof(options) === 'object') {
+        if (typeof(options['searchParameters']) === 'function') {
+          return options['searchParameters'](term)
+        } else {
+          console.warn(
+            "Solidus deprecation: Passing an object of parameters to variantAutocomplete is deprecated. Instead, on the options object, please declare `searchParameters` as a function returning the parameters.\n\n",
+            "Deprecated usage:\n",
+            "$('#id').variantAutocomplete({\n",
+            "  suppliable_only: true\n",
+            "})",
+            "\n\nNew usage:\n",
+            "$('#id').variantAutocomplete({\n",
+            "  searchParameters: function (_selectSearchTerm) { return { suppliable_only: true  } }\n",
+            "})"
+          )
+        }
+      }
+
+      return {}
     }
+
     this.select2({
       placeholder: Spree.translations.variant_placeholder,
       minimumInputLength: 3,
@@ -35,7 +63,7 @@
             token: Spree.api_key,
             page: page
           };
-          return _.extend(searchData, searchOptions);
+          return _.extend(searchData, extraParameters(term));
         },
 
         results: function(data, page) {

data/app/assets/javascripts/spree/backend/views/cart/line_item_row.js

@@ -84,6 +84,8 @@ Spree.Views.Cart.LineItemRow = Backbone.View.extend({
       noCancel: this.model.isNew() && this.model.collection.length == 1
     });
     this.$el.html(html);
-    this.$("[name=variant_id]").variantAutocomplete({ suppliable_only: true });
+    this.$("[name=variant_id]").variantAutocomplete({
+      searchParameters: function (_term) { return { suppliable_only: true  } }
+    });
   }
 });

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

@@ -69,6 +69,12 @@ module Spree
       def payment_method_params
         params.require(:payment_method).permit!
       end
+
+      def build_resource
+        model_class.new(
+          store_ids: [Spree::Store.default.id].compact
+        )
+      end
     end
   end
 end

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

@@ -82,9 +82,7 @@ class Spree::Admin::ResourceController < Spree::Admin::BaseController
       records = model_class.where(id: positions.keys).to_a
 
       positions.each do |id, index|
-        # To permit the use of UUID as id, we compare models id in database
-        # and in params under the string representation instead of integer
-        records.find { |r| r.id.to_s == id }&.set_list_position(index)
+        records.find { |r| r.id == id.to_i }&.set_list_position(index)
       end
     end
 

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

@@ -32,6 +32,7 @@ module Spree
 
       def load_data
         @tax_categories = Spree::TaxCategory.order(:name)
+        @shipping_categories = Spree::ShippingCategory.order(:name)
       end
 
       def variant_includes

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

@@ -142,14 +142,7 @@
             </div>
           <% end %>
         </div>
-        <div data-hook="admin_product_form_track_inventory">
-          <%= f.field_container :track_inventory do %>
-            <label>
-              <%= f.check_box :track_inventory %>
-              <%= Spree::Variant.human_attribute_name(:track_inventory) %>
-            </label>
-          <% end %>
-        </div>
+
       <% end %>
 
       <div data-hook="admin_product_form_shipping_categories">

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

@@ -1,4 +1,4 @@
-<div data-hook="admin_<%= model_name %>_form_generate_vat_prices" class="<%= local_assigns[:wrapper_class] %> checkbox">
+<div data-hook="admin_<%= model_name %>_form_generate_vat_prices" class="checkbox">
   <label>
     <%= form.check_box :rebuild_vat_prices, checked: form.object.prices.size <= 1 %>
     <%= Spree::Variant.human_attribute_name(:rebuild_vat_prices) %>

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

@@ -19,6 +19,20 @@
               <%= t('spree.included_in_price') %>
             </label>
           </div>
+          <div data-hook="level" class="field">
+            <%= f.label :level, t('spree.tax_rate_level') %>
+            <%= admin_hint t('spree.tax_rate_level'), t(:tax_rate_level, scope: [:spree, :hints, "spree/tax_rate"]) %>
+            <ul>
+              <% Spree::TaxRate.levels.keys.each do |level| %>
+                <li>
+                  <label>
+                    <%= f.radio_button :level, level %>
+                    <%= t("spree.#{level}_level") %>
+                  </label>
+                </li>
+              <% end %>
+            </ul>
+          </div>
         </div>
 
         <div class="col-5">

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

@@ -12,7 +12,7 @@
         <% if @taxon.parent %>
           <div class="input-group-prepend">
             <span class="input-group-text">
-              <%= @taxon.permalink.split('/')[0...-1].join('/') + '/' %>
+              <%= @taxon.permalink.split('/')[...-1].join('/') + '/' %>
             </span>
           </div>
         <% end %>

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

@@ -70,14 +70,11 @@
           <%= f.label :price %>
           <%= render "spree/admin/shared/number_with_currency", f: f, amount_attr: :price, currency: @variant.default_price_or_build.currency %>
         </div>
+        <% if show_rebuild_vat_checkbox? %>
+          <%= render "spree/admin/shared/rebuild_vat_prices_checkbox", form: f, model_name: "variant" %>
+        <% end %>
       </div>
 
-      <% if show_rebuild_vat_checkbox? %>
-        <div class="col-3">
-          <%= render "spree/admin/shared/rebuild_vat_prices_checkbox", form: f, model_name: "variant", wrapper_class: "field" %>
-        </div>
-      <% end %>
-
       <div class="col-3">
         <div class="field" data-hook="cost_price">
           <%= f.label :cost_price %>
@@ -97,6 +94,19 @@
                                   { class: 'custom-select fullwidth' } %>
         </div>
       </div>
+
+      <div class="col-3">
+        <div class="field" data-hook="shipping_category">
+          <%= f.label :shipping_category %>
+          <%= f.field_hint :shipping_category %>
+          <%= f.collection_select :shipping_category_id,
+                                  @shipping_categories,
+                                  :id,
+                                  :name,
+                                  { include_blank: t('.use_product_shipping_category') },
+                                  { class: 'custom-select fullwidth' } %>
+        </div>
+      </div>
     </div>
   </fieldset>
 </div>

data/solidus_backend.gemspec

@@ -20,7 +20,7 @@ Gem::Specification.new do |s|
     f.match(%r{^(spec|script)/})
   end
 
-  s.required_ruby_version = '>= 2.5.0'
+  s.required_ruby_version = '>= 2.7.0'
   s.required_rubygems_version = '>= 1.8.23'
 
   s.add_dependency 'solidus_api', s.version