solidus_frontend 2.11.7 → 3.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1fde8bbc2769e900496781f11efd3cabe66e289ad144628a836a91d03731f991
-  data.tar.gz: d8ec75e27e0b2160fbc2dafc6a0f45fec1fe8788b01f11ef64892dd07998676e
+  metadata.gz: cf5e18acd4c8ecbcf2cf401eb50c46e6d6b5bd84c7ffe3704c0b298f1308c574
+  data.tar.gz: 5f9cb36908fb080589a39cebf70da10c7ff38d2a023ee418e858aa73457dec58
 SHA512:
-  metadata.gz: a6c0551f408f61e1fb4c4d0e9cccd2cc0c39e31ab14b893f4a0c322edf65cfbdedfd394bee42e40a7ad154cebbb7963d3ee9f2ee2bbba8f75bf6c3aea7343182
-  data.tar.gz: 4af7e9c9c880d27e691524196852522f7703137b5ba2425c81d3ed2b248492a5feedd6fac20ec93b790d3fb5af6e4970c85f5a6d940c2aceaf650e7f1c153e34
+  metadata.gz: 8fd1a9c0f68b3f5e67644ba34f861ecbb5b3ce87b63c5aded1f5c490d6bd8a5dc06f6879e44bb330d14a923b70c05aff4097d85b82926f52bc61c1cd47d79733
+  data.tar.gz: 10372dfccc8cc1885247b8e4063974fbecbde6e54cbe394a5dfe84df9c2a1ceaa9c532fd882114b192adbf7d21e1717db5d7404081372fc90ec2df395afc539e

data/app/controllers/spree/checkout_controller.rb

@@ -17,7 +17,6 @@ module Spree
 
     before_action :associate_user
     before_action :check_authorization
-    before_action :apply_coupon_code, only: [:update]
 
     before_action :setup_for_current_state, only: [:edit, :update]
 
@@ -111,10 +110,6 @@ module Spree
       massaged_params = params.deep_dup
 
       move_payment_source_into_payments_attributes(massaged_params)
-      if massaged_params[:order] && massaged_params[:order][:existing_card].present?
-        Spree::Deprecation.warn("Passing order[:existing_card] is deprecated. Send order[:wallet_payment_source_id] instead.", caller)
-        move_existing_card_into_payments_attributes(massaged_params) # deprecated
-      end
       move_wallet_payment_source_id_into_payments_attributes(massaged_params)
       set_payment_parameters_amount(massaged_params, @order)
 
@@ -160,11 +155,6 @@ module Spree
       end
     end
 
-    def set_state_if_present
-      ensure_order_is_not_skipping_states
-    end
-    deprecate set_state_if_present: :prevent_order_from_skipping_states, deprecator: Spree::Deprecation
-
     def ensure_checkout_allowed
       unless @order.checkout_allowed?
         redirect_to spree.cart_path
@@ -188,24 +178,6 @@ module Spree
       spree.order_path(@order)
     end
 
-    def apply_coupon_code
-      if update_params[:coupon_code].present?
-        Spree::Deprecation.warn('This endpoint is deprecated. Please use `Spree::CouponCodesController#create` endpoint instead.')
-        @order.coupon_code = update_params[:coupon_code]
-
-        handler = PromotionHandler::Coupon.new(@order).apply
-
-        if handler.error.present?
-          flash.now[:error] = handler.error
-        elsif handler.success
-          flash[:success] = handler.success
-        end
-
-        setup_for_current_state
-        respond_with(@order) { |format| format.html { render :edit } } && return
-      end
-    end
-
     def setup_for_current_state
       method_name = :"before_#{@order.state}"
       send(method_name) if respond_to?(method_name, true)
@@ -239,14 +211,6 @@ module Spree
         @wallet_payment_sources = try_spree_current_user.wallet.wallet_payment_sources
         @default_wallet_payment_source = @wallet_payment_sources.detect(&:default) ||
                                          @wallet_payment_sources.first
-
-        @payment_sources = Spree::DeprecatedInstanceVariableProxy.new(
-          self,
-          :deprecated_payment_sources,
-          :@payment_sources,
-          Spree::Deprecation,
-          "Please, do not use @payment_sources anymore, use @wallet_payment_sources instead."
-        )
       end
     end
 
@@ -276,22 +240,5 @@ module Spree
         end
       end
     end
-
-    # This method returns payment sources of the current user. It is no more
-    # used into our frontend. We used to assign the content of this method
-    # into an ivar (@payment_sources) into the checkout payment step. This
-    # method is here only to be able to deprecate this ivar and will be removed.
-    #
-    # DO NOT USE THIS METHOD!
-    #
-    # @return [Array<Spree::PaymentSource>] Payment sources connected to
-    #   current user wallet.
-    # @deprecated This method has been added to deprecate @payment_sources
-    #   ivar and will be removed. Use @wallet_payment_sources instead.
-    def deprecated_payment_sources
-      try_spree_current_user.wallet.wallet_payment_sources
-        .map(&:payment_source)
-        .select { |ps| ps.is_a?(Spree::CreditCard) }
-    end
   end
 end

data/app/controllers/spree/orders_controller.rb

@@ -10,7 +10,6 @@ module Spree
     before_action :assign_order, only: :update
     # note: do not lock the #edit action because that's where we redirect when we fail to acquire a lock
     around_action :lock_order, only: :update
-    before_action :apply_coupon_code, only: :update
     skip_before_action :verify_authenticity_token, only: [:populate]
 
     def show
@@ -123,21 +122,5 @@ module Spree
         redirect_to(root_path) && return
       end
     end
-
-    def apply_coupon_code
-      if order_params[:coupon_code].present?
-        Spree::Deprecation.warn('This endpoint is deprecated. Please use `Spree::CouponCodesController#create` endpoint instead.')
-        @order.coupon_code = order_params[:coupon_code]
-
-        handler = PromotionHandler::Coupon.new(@order).apply
-
-        if handler.error.present?
-          flash.now[:error] = handler.error
-          respond_with(@order) { |format| format.html { render :edit } } && return
-        elsif handler.success
-          flash[:success] = handler.success
-        end
-      end
-    end
   end
 end

data/app/views/spree/address/_form.html.erb

@@ -1,21 +1,9 @@
 <% address_id = address_type.chars.first %>
 <div class="inner" data-hook=<%="#{address_type}_inner" %>>
- <% if Spree::Config.use_combined_first_and_last_name_in_address %>
-    <div class="field field-required" id="<%= "#{address_id}name" %>">
-      <%= form.label :name, t('spree.name') %>
-      <%= form.text_field :name, class: 'required', autocomplete: address_type + ' name', required: true, autofocus: true %>
-    </div>
-  <% else %>
-    <div class="field field-required" id="<%= "#{address_id}firstname" %>">
-      <%= form.label :firstname, t('spree.first_name') %>
-      <%= form.text_field :firstname, class: 'required', autocomplete: address_type + ' given-name', required: true, autofocus: true %>
-    </div>
-
-    <div class="field" id="<%= "#{address_id}lastname" %>">
-      <%= form.label :lastname, t('spree.last_name') %>
-      <%= form.text_field :lastname, autocomplete: address_type + ' family-name' %>
-    </div>
-  <% end %>
+  <div class="field field-required" id="<%= "#{address_id}name" %>">
+    <%= form.label :name, t('spree.name') %>
+    <%= form.text_field :name, class: 'required', autocomplete: address_type + ' name', required: true, autofocus: true %>
+  </div>
 
   <% if Spree::Config[:company] %>
     <div class="field" id=<%="#{address_id}company" %>>

data/app/views/spree/address/_form_hidden.html.erb

@@ -1,9 +1,4 @@
-<% if Spree::Config.use_combined_first_and_last_name_in_address %>
-  <%= form.hidden_field :name %>
-<% else %>
-  <%= form.hidden_field :firstname %>
-  <%= form.hidden_field :lastname %>
-<% end %>
+<%= form.hidden_field :name %>
 <%= form.hidden_field :company %>
 <%= form.hidden_field :address1 %>
 <%= form.hidden_field :address2 %>

data/lib/spree/frontend.rb

@@ -1,6 +1,8 @@
 # frozen_string_literal: true
 
-require 'rails/all'
+require 'solidus_core'
+require 'solidus_api'
+
 require 'jquery-rails'
 require 'canonical-rails'
 require 'sassc-rails'
@@ -8,8 +10,5 @@ require 'font-awesome-rails'
 require 'responders'
 require 'kaminari'
 
-require 'solidus_core'
-require 'solidus_api'
-
 require 'spree/frontend/middleware/seo_assist'
 require 'spree/frontend/engine'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: solidus_frontend
 version: !ruby/object:Gem::Version
-  version: 2.11.7
+  version: 3.0.1
 platform: ruby
 authors:
 - Solidus Team
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-03-18 00:00:00.000000000 Z
+date: 2021-05-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: solidus_api
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.11.7
+        version: 3.0.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.11.7
+        version: 3.0.1
 - !ruby/object:Gem::Dependency
   name: solidus_core
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.11.7
+        version: 3.0.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 2.11.7
+        version: 3.0.1
 - !ruby/object:Gem::Dependency
   name: canonical-rails
   requirement: !ruby/object:Gem::Requirement