solidus_backend 4.6.2 → 4.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2fb2b4c9596648ba66d5ceffe7e339e7f9bca3bc5dc0863feb80b2a8dcf9d472
-  data.tar.gz: 22389566c4aa44ea193beeed9e65db2d64c46f0d5b8290a07594e2520c8d7f8b
+  metadata.gz: 4958f03ffc21587a25b09f2c740c02c65620763fe7dbe2f7c7af03beb34d871b
+  data.tar.gz: f6fd946e53ea567c45df3e320df786155f5d25fc213737310b452f8bb39df19a
 SHA512:
-  metadata.gz: 7fbad55955f91c6334def19f2f8202a6619a2b3db6d655cc2eff5ce4d480ec76d9a9b5509b80428984d8a545e7c9ac659de00adeba0d8549f7f11d46b3e34dd0
-  data.tar.gz: 669532feea711782cf622022d99de0a12317a994601f50320025c0f3d174131b1ec19f8f89de5b727c0c60da351b7d6d2090dc3d9b288e3d5de8aa1a8b9f95f9
+  metadata.gz: 8a218405e4b56232325c49b1f9e508216bfc38bc469c8b3b704be0d44206ba77919ede568341abdc4f35a70ebc13ea40fdc8d46e0509dcef62f895fe7420837a
+  data.tar.gz: 352496ad965b70ed839de9779048c8c49ecf67cf92cc83e374dad6cc7b15b1f3d1ce94e777829aa76c09cbde580df2bde5e1964e5ce4f77d8ca5886f803411f1

data/Rakefile

@@ -1,32 +1,32 @@
 # frozen_string_literal: true
 
-require 'rubygems'
-require 'rake'
-require 'rake/testtask'
-require 'rspec/core/rake_task'
-require 'bundler/gem_tasks'
+require "rubygems"
+require "rake"
+require "rake/testtask"
+require "rspec/core/rake_task"
+require "bundler/gem_tasks"
 
-require 'solidus_backend'
-require 'spree/testing_support/dummy_app/rake_tasks'
+require "solidus_backend"
+require "spree/testing_support/dummy_app/rake_tasks"
 
 RSpec::Core::RakeTask.new
-task default: [:spec, 'spec:js']
+task default: [:spec, "spec:js"]
 
 DummyApp::RakeTasks.new(
   gem_root: File.expand_path(__dir__),
-  lib_name: 'solidus_backend'
+  lib_name: "solidus_backend"
 )
 
 task :teaspoon do
-  require 'teaspoon'
-  Rake::Task['dummy_environment'].invoke
+  require "teaspoon"
+  Rake::Task["dummy_environment"].invoke
   require "teaspoon/console"
 
   options = {
     files: ENV["files"].nil? ? [] : ENV["files"].split(","),
     suite: ENV["suite"],
     coverage: ENV["coverage"],
-    driver_options: ENV["driver_options"],
+    driver_options: ENV["driver_options"]
   }
 
   options.delete_if { |_k, v| v.nil? }
@@ -35,13 +35,13 @@ task :teaspoon do
 end
 
 desc "Run javascript specs"
-task 'spec:js' => :teaspoon
+task "spec:js" => :teaspoon
 
 namespace :teaspoon do
   task :server do
-    require 'teaspoon'
-    Rake::Task['dummy_environment'].invoke
-    require 'teaspoon/server'
+    require "teaspoon"
+    Rake::Task["dummy_environment"].invoke
+    require "teaspoon/server"
 
     server = Teaspoon::Server.new
     server.start
@@ -50,4 +50,4 @@ namespace :teaspoon do
   end
 end
 
-task test_app: 'db:reset'
+task test_app: "db:reset"

data/app/assets/javascripts/spree/backend/components/tabs.js

@@ -30,7 +30,6 @@ Tabs = (function() {
 
   Tabs.prototype.overflowTabs = function() {
     var containerWidth = this.el.offsetWidth;
-    var dropdownWidth = this.dropdown.offsetWidth;
 
     for (var i = 0; i < this.tabs.length; i++) {
       var tab = this.tabs[i];
@@ -41,7 +40,6 @@ Tabs = (function() {
       this.el.classList.remove("tabs-overflowed");
     } else {
       this.el.classList.add("tabs-overflowed");
-      remainingWidth -= dropdownWidth;
     }
 
     var remainingWidth = containerWidth;

data/app/assets/javascripts/spree/backend/views/tables/selectable_table/sum_return_item_amount.js

@@ -16,7 +16,7 @@ Spree.Views.Tables.SelectableTable.SumReturnItemAmount = Backbone.View.extend({
 
   totalSelectedReturnItemAmount: function () {
     var totalAmount = 0;
-    var selectedItems = [];
+    var selectedItems;
     var decimals = 0;
     var separator = Spree.t('currency_separator');
     var amount, decimalAmount;

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

@@ -3,7 +3,7 @@
 module Spree
   module Admin
     class AdjustmentsController < ResourceController
-      belongs_to 'spree/order', find_by: :number
+      belongs_to "spree/order", find_by: :number
 
       create.after :update_totals
       destroy.after :update_totals

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

@@ -3,8 +3,8 @@
 module Spree
   module Admin
     class BaseController < Spree::BaseController
-      helper 'spree/admin/navigation'
-      layout 'spree/layouts/admin'
+      helper "spree/admin/navigation"
+      layout "spree/layouts/admin"
 
       before_action :authorize_admin
 
@@ -21,10 +21,10 @@ module Spree
       end
 
       def authorize_admin
-        if respond_to?(:model_class, true) && model_class
-          record = model_class
+        record = if respond_to?(:model_class, true) && model_class
+          model_class
         else
-          record = controller_name.to_sym
+          controller_name.to_sym
         end
         authorize! :admin, record
         authorize! action, record
@@ -39,13 +39,13 @@ module Spree
       end
 
       def flash_message_for(object, event_sym)
-        resource_desc  = object.class.model_name.human
+        resource_desc = object.class.model_name.human
         resource_desc += " \"#{object.name}\"" if object.respond_to?(:name) && object.name.present?
-        t(event_sym, resource: resource_desc, scope: 'spree')
+        t(event_sym, resource: resource_desc, scope: "spree")
       end
 
       def render_js_for_destroy
-        render partial: '/spree/admin/shared/destroy'
+        render partial: "/spree/admin/shared/destroy"
       end
 
       def config_locale
@@ -55,7 +55,7 @@ module Spree
       def lock_order
         Spree::OrderMutex.with_lock!(@order) { yield }
       rescue Spree::OrderMutex::LockFailed
-        flash[:error] = t('spree.order_mutex_admin_error')
+        flash[:error] = t("spree.order_mutex_admin_error")
         redirect_to order_mutex_redirect_path
       end
 

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

@@ -14,15 +14,15 @@ module Spree
         inventory_units = Spree::InventoryUnit.where(id: inventory_unit_ids)
 
         if inventory_units.size != inventory_unit_ids.size
-          flash[:error] = t('spree.unable_to_find_all_inventory_units')
+          flash[:error] = t("spree.unable_to_find_all_inventory_units")
           redirect_to admin_order_cancellations_path(@order)
         elsif inventory_units.empty?
-          flash[:error] = t('spree.no_inventory_selected')
+          flash[:error] = t("spree.no_inventory_selected")
           redirect_to admin_order_cancellations_path(@order)
         else
           @order.cancellations.short_ship(inventory_units, created_by:)
 
-          flash[:success] = t('spree.inventory_canceled')
+          flash[:success] = t("spree.inventory_canceled")
           redirect_to edit_admin_order_url(@order)
         end
       end

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

@@ -3,13 +3,13 @@
 module Spree
   module Admin
     class CustomerReturnsController < ResourceController
-      helper 'spree/admin/reimbursement_type'
-      belongs_to 'spree/order', find_by: :number
+      helper "spree/admin/reimbursement_type"
+      belongs_to "spree/order", find_by: :number
 
       before_action :parent # ensure order gets loaded to support our pseudo parent-child relationship
       before_action :load_form_data, only: [:new, :edit]
       before_action :build_return_items_from_params, only: [:create]
-      create.fails  :load_form_data
+      create.fails :load_form_data
       create.after :order_process_return
 
       def edit
@@ -62,18 +62,18 @@ module Spree
       end
 
       def permitted_resource_params
-        @permitted_resource_params ||= params.require('customer_return').permit(permitted_customer_return_attributes)
+        @permitted_resource_params ||= params.require("customer_return").permit(permitted_customer_return_attributes)
       end
 
       def build_return_items_from_params
         return_items_params = permitted_resource_params.delete(:return_items_attributes).values
         @customer_return.return_items = return_items_params.map do |item_params|
-          next unless item_params.delete('returned') == '1'
+          next unless item_params.delete("returned") == "1"
           return_item = item_params[:id] ? Spree::ReturnItem.find(item_params[:id]) : Spree::ReturnItem.new
           return_item.assign_attributes(item_params)
 
           if item_params[:reception_status_event].blank?
-            return redirect_to(new_object_url, flash: { error: 'Reception status choice required' })
+            return redirect_to(new_object_url, flash: {error: "Reception status choice required"})
           end
           return_item
         end.compact

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

@@ -22,13 +22,13 @@ module Spree
         @variants = @product.variants.collect do |variant|
           [variant.sku_and_options_text, variant.id]
         end
-        @variants.insert(0, [t('spree.all'), @product.master.id])
+        @variants.insert(0, [t("spree.all"), @product.master.id])
       rescue ActiveRecord::RecordNotFound
         resource_not_found(flash_class: Spree::Product, redirect_url: admin_products_path)
       end
 
       def set_viewable
-        @image.viewable_type = 'Spree::Variant'
+        @image.viewable_type = "Spree::Variant"
         @image.viewable_id = params[:image][:viewable_id]
       end
     end

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

@@ -12,13 +12,13 @@ module Spree
           I18n.locale = requested_locale
           session[set_user_language_locale_key] = requested_locale
           respond_to do |format|
-            format.json { render json: { locale: requested_locale, location: spree.admin_url } }
-            format.html { redirect_back_or_to spree.admin_url, notice: t('spree.locale_changed') }
+            format.json { render json: {locale: requested_locale, location: spree.admin_url} }
+            format.html { redirect_back_or_to spree.admin_url, notice: t("spree.locale_changed") }
           end
         else
           respond_to do |format|
-            format.json { render json: { locale: I18n.locale }, status: 404 }
-            format.html { redirect_back_or_to spree.admin_url, error: t('spree.error') }
+            format.json { render json: {locale: I18n.locale}, status: 404 }
+            format.html { redirect_back_or_to spree.admin_url, error: t("spree.error") }
           end
         end
       end

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

@@ -31,7 +31,7 @@ module Spree
 
       def set_available_option_types
         @available_option_types = if @product.option_type_ids.any?
-          Spree::OptionType.where('id NOT IN (?)', @product.option_type_ids)
+          Spree::OptionType.where("id NOT IN (?)", @product.option_type_ids)
         else
           Spree::OptionType.all
         end

data/app/controllers/spree/admin/orders/customer_details_controller.rb

@@ -35,7 +35,7 @@ module Spree
               @order.refresh_shipment_rates
             end
 
-            flash[:success] = t('spree.customer_details_updated')
+            flash[:success] = t("spree.customer_details_updated")
             redirect_to edit_admin_order_url(@order)
           else
             render action: :edit
@@ -74,7 +74,7 @@ module Spree
         end
 
         def insufficient_stock_error
-          flash[:error] = t('spree.insufficient_stock_for_order')
+          flash[:error] = t("spree.insufficient_stock_for_order")
           redirect_to edit_admin_order_customer_url(@order)
         end
       end

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

@@ -16,10 +16,10 @@ module Spree
 
       def index
         params[:q] ||= {}
-        params[:q][:completed_at_not_null] ||= '1' if Spree::Config[:show_only_complete_orders_by_default]
-        @show_only_completed = params[:q][:completed_at_not_null] == '1'
-        params[:q][:s] ||= @show_only_completed ? 'completed_at desc' : 'created_at desc'
-        params[:q][:completed_at_not_null] = '' unless @show_only_completed
+        params[:q][:completed_at_not_null] ||= "1" if Spree::Config[:show_only_complete_orders_by_default]
+        @show_only_completed = params[:q][:completed_at_not_null] == "1"
+        params[:q][:s] ||= @show_only_completed ? "completed_at desc" : "created_at desc"
+        params[:q][:completed_at_not_null] = "" unless @show_only_completed
 
         # As date params are deleted if @show_only_completed, store
         # the original date so we can restore them into the params
@@ -31,18 +31,18 @@ module Spree
 
         if params[:q][:created_at_gt].present?
           params[:q][:created_at_gt] = begin
-                                         Time.zone.parse(params[:q][:created_at_gt]).beginning_of_day
-                                       rescue StandardError
-                                         ""
-                                       end
+            Time.zone.parse(params[:q][:created_at_gt]).beginning_of_day
+          rescue
+            ""
+          end
         end
 
         if params[:q][:created_at_lt].present?
           params[:q][:created_at_lt] = begin
-                                         Time.zone.parse(params[:q][:created_at_lt]).end_of_day
-                                       rescue StandardError
-                                         ""
-                                       end
+            Time.zone.parse(params[:q][:created_at_lt]).end_of_day
+          rescue
+            ""
+          end
         end
 
         if @show_only_completed
@@ -51,9 +51,9 @@ module Spree
         end
 
         @search = Spree::Order.accessible_by(current_ability, :index).ransack(params[:q])
-        @orders = @search.result.includes([:user]).
-          page(params[:page]).
-          per(params[:per_page] || Spree::Config[:orders_per_page])
+        @orders = @search.result.includes([:user])
+          .page(params[:page])
+          .per(params[:per_page] || Spree::Config[:orders_per_page])
 
         # Restore dates
         params[:q][:created_at_gt] = created_at_gt
@@ -82,15 +82,15 @@ module Spree
 
       def advance
         if @order.completed?
-          flash[:notice] = t('spree.order_already_completed')
+          flash[:notice] = t("spree.order_already_completed")
           redirect_to edit_admin_order_url(@order)
         else
           @order.contents.advance
 
           if @order.can_complete?
-            flash[:success] = t('spree.order_ready_for_confirm')
+            flash[:success] = t("spree.order_ready_for_confirm")
           else
-            flash[:error] = @order.errors.full_messages.join(', ')
+            flash[:error] = @order.errors.full_messages.join(", ")
           end
           redirect_to confirm_admin_order_url(@order)
         end
@@ -101,14 +101,14 @@ module Spree
         if @order.completed?
           redirect_to edit_admin_order_url(@order)
         elsif !@order.can_complete?
-          render template: 'spree/admin/orders/confirm_advance'
+          render template: "spree/admin/orders/confirm_advance"
         end
       end
 
       # PUT
       def complete
         @order.complete!
-        flash[:success] = t('spree.order_completed')
+        flash[:success] = t("spree.order_completed")
         redirect_to edit_admin_order_url(@order)
       rescue StateMachines::InvalidTransition => error
         flash[:error] = error.message
@@ -117,25 +117,25 @@ module Spree
 
       def cancel
         @order.canceled_by(spree_current_user)
-        flash[:success] = t('spree.order_canceled')
+        flash[:success] = t("spree.order_canceled")
         redirect_to(spree.edit_admin_order_path(@order))
       end
 
       def resume
         @order.resume!
-        flash[:success] = t('spree.order_resumed')
+        flash[:success] = t("spree.order_resumed")
         redirect_to(spree.edit_admin_order_path(@order))
       end
 
       def approve
         @order.contents.approve(user: spree_current_user)
-        flash[:success] = t('spree.order_approved')
+        flash[:success] = t("spree.order_approved")
         redirect_to(spree.edit_admin_order_path(@order))
       end
 
       def resend
         Spree::Config.order_mailer_class.confirm_email(@order, true).deliver_later
-        flash[:success] = t('spree.order_email_resent')
+        flash[:success] = t("spree.order_email_resent")
 
         redirect_to(spree.edit_admin_order_path(@order))
       end
@@ -143,7 +143,7 @@ module Spree
       def unfinalize_adjustments
         adjustments = @order.all_adjustments.finalized
         adjustments.each(&:unfinalize!)
-        flash[:success] = t('spree.all_adjustments_unfinalized')
+        flash[:success] = t("spree.all_adjustments_unfinalized")
 
         respond_with(@order) { |format| format.html { redirect_to(spree.admin_order_adjustments_path(@order)) } }
       end
@@ -151,7 +151,7 @@ module Spree
       def finalize_adjustments
         adjustments = @order.all_adjustments.not_finalized
         adjustments.each(&:finalize!)
-        flash[:success] = t('spree.all_adjustments_finalized')
+        flash[:success] = t("spree.all_adjustments_finalized")
 
         respond_with(@order) { |format| format.html { redirect_to(spree.admin_order_adjustments_path(@order)) } }
       end
@@ -175,7 +175,7 @@ module Spree
 
       # Used for extensions which need to provide their own custom event links on the order details view.
       def initialize_order_events
-        @order_events = %w{approve cancel resume}
+        @order_events = %w[approve cancel resume]
       end
 
       def model_class
@@ -183,13 +183,13 @@ module Spree
       end
 
       def insufficient_stock_error
-        flash[:error] = t('spree.insufficient_stock_for_order')
+        flash[:error] = t("spree.insufficient_stock_for_order")
         redirect_to cart_admin_order_url(@order)
       end
 
       def require_ship_address
         if @order.ship_address.nil?
-          flash[:notice] = t('spree.fill_in_customer_info')
+          flash[:notice] = t("spree.fill_in_customer_info")
           redirect_to edit_admin_order_customer_url(@order)
         end
       end

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

@@ -15,7 +15,7 @@ module Spree
         invoke_callbacks(:create, :before)
         if @payment_method.save
           invoke_callbacks(:create, :after)
-          flash[:success] = t('spree.successfully_created', resource: t('spree.payment_method'))
+          flash[:success] = t("spree.successfully_created", resource: t("spree.payment_method"))
           redirect_to edit_admin_payment_method_path(@payment_method)
         else
           invoke_callbacks(:create, :fails)
@@ -36,7 +36,7 @@ module Spree
 
         if @payment_method.update(attributes)
           invoke_callbacks(:update, :after)
-          flash[:success] = t('spree.successfully_updated', resource: t('spree.payment_method'))
+          flash[:success] = t("spree.successfully_updated", resource: t("spree.payment_method"))
           redirect_to edit_admin_payment_method_path(@payment_method)
         else
           invoke_callbacks(:update, :fails)
@@ -61,7 +61,7 @@ module Spree
         end
 
         if !@payment_method_type
-          flash[:error] = t('spree.invalid_payment_method_type')
+          flash[:error] = t("spree.invalid_payment_method_type")
           redirect_to new_admin_payment_method_path
         end
       end

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

@@ -27,7 +27,7 @@ module Spree
 
       def create
         @payment = PaymentCreate.new(@order, object_params).build
-        if @payment.payment_method.source_required? && params[:card].present? && params[:card] != 'new'
+        if @payment.payment_method.source_required? && params[:card].present? && params[:card] != "new"
           @payment.source = @payment.payment_method.payment_source_class.find_by(id: params[:card])
         end
 
@@ -45,7 +45,7 @@ module Spree
             flash[:success] = flash_message_for(@payment, :successfully_created)
             redirect_to admin_order_payments_path(@order)
           else
-            flash[:error] = t('spree.payment_could_not_be_created')
+            flash[:error] = t("spree.payment_could_not_be_created")
             render :new
           end
         rescue Spree::Core::GatewayError => error
@@ -60,9 +60,9 @@ module Spree
         # Because we have a transition method also called void, we do this to avoid conflicts.
         event = "void_transaction" if event == "void"
         if @payment.send("#{event}!")
-          flash[:success] = t('spree.payment_updated')
+          flash[:success] = t("spree.payment_updated")
         else
-          flash[:error] = t('spree.cannot_perform_operation')
+          flash[:error] = t("spree.cannot_perform_operation")
         end
       rescue Spree::Core::GatewayError => ge
         flash[:error] = ge.message.to_s
@@ -83,10 +83,10 @@ module Spree
       def load_data
         @amount = params[:amount] || load_order.total
         @payment_methods = Spree::PaymentMethod.active.available_to_admin.ordered_by_position
-        if @payment && @payment.payment_method
-          @payment_method = @payment.payment_method
+        @payment_method = if @payment&.payment_method
+          @payment.payment_method
         else
-          @payment_method = @payment_methods.first
+          @payment_methods.first
         end
       end
 
@@ -113,13 +113,13 @@ module Spree
 
       def require_bill_address
         if Spree::Config[:order_bill_address_used] && @order.bill_address.nil?
-          flash[:notice] = t('spree.fill_in_customer_info')
+          flash[:notice] = t("spree.fill_in_customer_info")
           redirect_to edit_admin_order_customer_url(@order)
         end
       end
 
       def insufficient_stock_error
-        flash[:error] = t('spree.insufficient_stock_for_order')
+        flash[:error] = t("spree.insufficient_stock_for_order")
         redirect_to new_admin_order_payment_url(@order)
       end
     end

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

@@ -3,7 +3,7 @@
 module Spree
   module Admin
     class PricesController < ResourceController
-      belongs_to 'spree/product', find_by: :slug
+      belongs_to "spree/product", find_by: :slug
 
       def index
         params[:q] ||= {}

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

@@ -3,7 +3,7 @@
 module Spree
   module Admin
     class ProductPropertiesController < ResourceController
-      belongs_to 'spree/product', find_by: :slug, includes: { product_properties: :property }
+      belongs_to "spree/product", find_by: :slug, includes: {product_properties: :property}
       before_action :find_properties
       before_action :setup_property, only: :index, if: -> { can?(:create, model_class) }
       before_action :setup_variant_property_rules, only: :index

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

@@ -3,7 +3,7 @@
 module Spree
   module Admin
     class ProductsController < ResourceController
-      helper 'spree/products'
+      helper "spree/products"
 
       before_action :load_data, except: [:index]
       update.before :update_before
@@ -25,7 +25,7 @@ module Spree
         @product = Spree::Product.friendly.find(params[:id])
         @product.discard
 
-        flash[:success] = t('spree.notice_messages.product_deleted')
+        flash[:success] = t("spree.notice_messages.product_deleted")
 
         respond_with(@product) do |format|
           format.html { redirect_to collection_url }
@@ -37,9 +37,9 @@ module Spree
         @new = @product.duplicate
 
         if @new.save
-          flash[:success] = t('spree.notice_messages.product_cloned')
+          flash[:success] = t("spree.notice_messages.product_cloned")
         else
-          flash[:error] = t('spree.notice_messages.product_not_cloned')
+          flash[:error] = t("spree.notice_messages.product_not_cloned")
         end
 
         redirect_to edit_admin_product_url(@new)
@@ -53,10 +53,10 @@ module Spree
 
       def split_params
         if params[:product][:taxon_ids].present?
-          params[:product][:taxon_ids] = params[:product][:taxon_ids].split(',')
+          params[:product][:taxon_ids] = params[:product][:taxon_ids].split(",")
         end
         if params[:product][:option_type_ids].present?
-          params[:product][:option_type_ids] = params[:product][:option_type_ids].split(',')
+          params[:product][:option_type_ids] = params[:product][:option_type_ids].split(",")
         end
       end
 
@@ -89,11 +89,11 @@ module Spree
         params[:q][:s] ||= "name asc"
         # @search needs to be defined as this is passed to search_form_for
         @search = super.ransack(params[:q])
-        @collection = @search.result.
-              order(id: :asc).
-              includes(product_includes).
-              page(params[:page]).
-              per(Spree::Config[:admin_products_per_page])
+        @collection = @search.result
+          .order(id: :asc)
+          .includes(product_includes)
+          .page(params[:page])
+          .per(Spree::Config[:admin_products_per_page])
       end
 
       def update_before
@@ -104,7 +104,7 @@ module Spree
       end
 
       def product_includes
-        [:variant_images, { variants: [:images], master: [:images, :prices] }]
+        [:variant_images, {variants: [:images], master: [:images, :prices]}]
       end
 
       def clone_object_url(resource)
@@ -127,14 +127,14 @@ module Spree
         # Stops people submitting blank slugs, causing errors when they try to
         # update the product again
         @product.slug = @product.slug_was if @product.slug.blank?
-        render action: 'edit'
+        render action: "edit"
       end
 
       def normalize_variant_property_rules
         return unless updating_variant_property_rules?
 
         params[:product][:variant_property_rules_attributes].each_value do |param_attrs|
-          param_attrs[:option_value_ids] = param_attrs[:option_value_ids].split(',')
+          param_attrs[:option_value_ids] = param_attrs[:option_value_ids].split(",")
         end
       end
     end