ba_spree 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 82aeb70ed5e30afcc4c5849eb6daf41f9a311b649e9e3f86adb0ff519feee6c9
-  data.tar.gz: 771f95d599301938053a772c997baca793f9237451568b1072ed127be2770f17
+  metadata.gz: 7ff58a21f768530a02800b2fc55181f69d07e8abf91b2e2da5b0a9064e412b7e
+  data.tar.gz: d16fd26564e35a8a448420fd8f7dc585111c11f4fbb3e5204a73c299bfb9c9ee
 SHA512:
-  metadata.gz: 1e15809db814b48a2211a17dce5ac0e3e2fcd6dd40aaf0dcf0d7a96f0c43664a71eb33ff718eded86546cef5a6384706a35bc0a2c1361eea5219e12c178720c2
-  data.tar.gz: 0a2c180bdf9a3e648affd0f56fe915f87a8ee7c2b790e30d04a80298fbfc52c4f907a511d30ecd0283330bc96e953abad0be333a0625d65e54263bcefa91b75f
+  metadata.gz: bc60da4dd93ffcf3d0d15e823f5f6f264a3368c4614b2c2bcde5527cf4bf0146b45cabfe4277d0e57478a5fdeac1ca131f011e6a3bc924627eba02553b6e7ed5
+  data.tar.gz: dc7bd940663133de589568351e00d546587163bdebe19eb81a2872df644238e2cae1fe3420153bd2ccf7f3f8c4365947487e7a299027464d70f6d8ae10384f54

data/README.md

@@ -69,7 +69,7 @@ are specific to your shop, so set them yourself.
 
 ## What you get
 
-ba_spree itself is a thin wrapper. The substance is these 20 engines, all
+ba_spree itself is a thin wrapper. The substance is these 23 engines, all
 pulled in by a single `gem 'ba_spree'`.
 
 ### Japanese commerce conventions
@@ -99,6 +99,9 @@ pulled in by a single `gem 'ba_spree'`.
 | `spree_auto_capture_digital` | Auto-capture orders containing only digital products |
 | `spree_digital_payment_notice` | Show payment notices for orders containing digital products |
 | `spree_checkout_signup_promotion` | Prompt sign-up / login at the checkout address step |
+| `spree_order_total_discount` | Percentage discount on the order total |
+| `spree_common_shipping_method_splitter` | Ship items in one package when their shipping categories share a shipping method |
+| `spree_promotion_action_create_line_items` | Stop customers adding, changing or removing the items a promotion adds to the cart |
 
 ### Analytics
 

data/app/controllers/ba_spree/checkout_controller_decorator.rb

@@ -11,6 +11,9 @@ module BaSpree
     # @gem-override spree_storefront-5.3.6/app/controllers/spree/checkout_controller.rb#load_order
     # reason: ゲストチェックアウト対応のため super なしで丸ごと再実装。本家のゲスト処理を
     #   handle_guest_checkout / create_guest_order に抽出している (commit d0dbe2b4)
+    #   #1352: 抽出時に落ちていた本家の `elsif !current_store.prefers_guest_checkout?` →
+    #   `require_user(redirect_path: spree_signup_path)` の分岐を復元した。
+    #   これが無いと guest_checkout を OFF にしてもゲストのカートがチェックアウトへ進めてしまう。
     # @see https://github.com/be-agile/giga-repeat/commit/d0dbe2b4524e22c1cf974bbd4829c587ef7cd254
     def load_order
       return @order if @order.present?
@@ -35,6 +38,11 @@ module BaSpree
         elsif try_spree_current_user.nil? && !allow_access_to_complete_order_with_new_user?
           handle_guest_checkout
         end
+      elsif !current_store.prefers_guest_checkout?
+        # 本家にある分岐。ユーザーが紐付いていない（＝純粋なゲストの）カートは、
+        # ゲスト購入が無効なストアではログインを要求する。
+        # @see https://github.com/be-agile/giga-repeat/issues/1352
+        require_user(redirect_path: spree_signup_path)
       end
 
       # completed orders shouldn't be updated anymore

data/app/controllers/ba_spree/storefront_api_products_controller_decorator.rb

@@ -0,0 +1,28 @@
+module BaSpree
+  # storefront API の商品取得を、HTML 側と同じ公開状態の判定に揃える。
+  #
+  # 本家の scope は for_store + accessible_by(:show) だけで、ゲストに適用される
+  # Spree::PermissionSets::DefaultCustomer が `can :read, Spree::Product` を条件なしで
+  # 与えるため、公開状態の絞り込みが効いていない。結果、slug か id を指定すれば
+  # draft(非公開) / archived(取扱対象外) / 論理削除済みの商品が取得できていた。
+  #
+  # 一覧も同じ scope を通るので、filter[show_discontinued] / filter[show_deleted] を
+  # 渡してフィルタを外す経路もここで塞がる（for_user が付ける status = 'active' は
+  # finder の include_discontinued では外れず、not_deleted は生 SQL の where なので
+  # with_deleted の unscope でも消えない）。
+  #
+  # prepend 先は storefront のコントローラに限定する。Spree::Api::V2::ResourceController に
+  # 当てると、それを継承する Spree::Api::V2::Platform::ResourceController 経由で
+  # Platform API 全体が Spree::Variant 等に for_user を呼んで NoMethodError になる。
+  #
+  # @see https://github.com/be-agile/giga-repeat/issues/1352
+  module StorefrontApiProductsControllerDecorator
+    protected
+
+    def scope(skip_cancancan: false)
+      super.for_user(spree_current_user)
+    end
+  end
+end
+
+Spree::Api::V2::Storefront::ProductsController.prepend(BaSpree::StorefrontApiProductsControllerDecorator)

data/app/controllers/ba_spree/storefront_api_variants_controller_decorator.rb

@@ -0,0 +1,27 @@
+module BaSpree
+  # storefront API のバリエーション取得も、商品と同じ公開状態の判定に揃える
+  # （draft / archived / 論理削除済みの商品の SKU・価格が取得できていた）。
+  #
+  # scope の返り値は Spree::Variant のリレーションで、for_user は Spree::Product の
+  # スコープのため後ろにチェーンできない。商品を引く側に挟む必要があるので、
+  # 本家のメソッドを丸ごと再実装している。
+  module StorefrontApiVariantsControllerDecorator
+    private
+
+    # @gem-override spree_api-5.3.6/app/controllers/spree/api/v2/storefront/variants_controller.rb#scope
+    # reason: 本家の product_scope に for_user を挟むため。for_user は Spree::Product の
+    #   スコープで、本家の返り値(Spree::Variant のリレーション)にはチェーンできない。
+    # @see https://github.com/be-agile/giga-repeat/issues/1352
+    def scope(skip_cancancan: false)
+      product_scope = Spree::Product.for_store(current_store)
+      product_scope = product_scope.accessible_by(current_ability, :show) unless skip_cancancan
+      product_scope = product_scope.for_user(spree_current_user)
+      product_scope = product_scope.i18n if model_class.include?(Spree::TranslatableResource)
+
+      product = product_scope.friendly.find(params[:id])
+      product.variants_including_master
+    end
+  end
+end
+
+Spree::Api::V2::Storefront::VariantsController.prepend(BaSpree::StorefrontApiVariantsControllerDecorator)

data/app/finders/ba_spree/products/find_decorator.rb

@@ -0,0 +1,19 @@
+module BaSpree
+  module Products
+    module FindDecorator
+      private
+
+      # @gem-override spree_core-5.3.6/app/finders/spree/products/find.rb#ordered
+      # reason: 「関連性」の並び順で、親カテゴリの一覧に子カテゴリの商品が混ざって並ばないよう、
+      #   カテゴリツリーの縦型探索順に並べる。
+      # @see https://github.com/be-agile/giga-repeat/issues/1503
+      def ordered(products)
+        return super unless %w[default manual].include?(sort_by) && taxons?
+
+        products.ascend_by_taxons_tree_position(taxons)
+      end
+    end
+  end
+end
+
+Spree::Products::Find.prepend(BaSpree::Products::FindDecorator)

data/app/helpers/spree/base_helper_decorator.rb

@@ -1,5 +1,26 @@
 module Spree
   module BaseHelperDecorator
+    # Issue #1153: ストアフロントのフラグメントキャッシュのキーは、本家では
+    #   locale / currency / ログイン中か否か / role_users しか持たず、ユーザごとの差が入らない。
+    #   このままカテゴリ・商品・価格をユーザごとに絞ると、別のユーザ向けに生成された HTML が
+    #   そのまま出るため、顧客グループID(ソート済み)を署名として足して分離する。
+    #   spree_storefront_base_cache_key の中身でもあるため、ナビ・一覧・絞り込みサイドバーが
+    #   これ1箇所で分離される。Issue #1219 で残っていた「一覧の価格が別グループのユーザに
+    #   見える」経路の修正も兼ねる。
+    # 本家と同じく defined?(try_spree_current_user) のガードを維持する
+    #   (Spree::BaseHelper は Spree::MailHelper と Spree::VariantPresenter にも include され、
+    #    そこでは try_spree_current_user が未定義のため)。
+    # @see https://github.com/be-agile/giga-repeat/issues/1153
+    # メモ化には本家と別の ivar を使う。本家も @spree_base_cache_key に ||= するため、同じ
+    # ivar を使うと「||= が左辺を先に読む」という評価順序に依存することになり、本家の実装が
+    # 変わったときに顧客グループIDだけが無音で落ちる(別グループ向けの HTML が出る)。
+    def spree_base_cache_key
+      @ba_spree_base_cache_key ||= [
+        *super,
+        (defined?(try_spree_current_user) && try_spree_current_user.try(:customer_group_ids)&.sort)
+      ].compact
+    end
+
     CHILD_TAXONS_SECTION_DEFAULT_PREFERENCES = {
       heading_size: 'medium',
       heading_alignment: 'center'

data/app/models/ba_spree/product_decorator.rb

@@ -24,6 +24,20 @@ module BaSpree
 
       # ProductProperty -> metafields 移行用の読み書きヘルパー(Spree 5.4 の Property 削除対応)
       base.include Spree::ProductPropertyMetafields
+
+      # 指定カテゴリ群の中で、カテゴリツリーを縦型探索した順(lft) → カテゴリ内の商品の並び順(position) に並べる。
+      # 複数のカテゴリに属する商品は、最初に出てくる位置で並ぶ。
+      # taxon は、絞り込みに使う spree_products_taxons の行からたどり、指定カテゴリ群の行だけで MIN をとる。
+      base.add_search_scope :ascend_by_taxons_tree_position do |taxon_ids|
+        # position(INT)の上の桁に lft を置いた1つの整数にする
+        tree_position_sql = "MIN(#{Spree::Taxon.table_name}.lft * #{2**32} + #{Spree::Classification.table_name}.position)"
+
+        joins(classifications: :taxon).
+          where(Spree::Classification.table_name => { taxon_id: taxon_ids }).
+          select("#{Spree::Product.table_name}.*", "#{tree_position_sql} AS taxon_tree_position").
+          group("#{Spree::Product.table_name}.id").
+          order(Arel.sql("#{tree_position_sql} ASC"))
+      end
     end
 
     def set_slug

data/app/models/spree/promotion_handler/cart_decorator.rb

@@ -18,7 +18,12 @@ module Spree
         end
       end
 
+      # 注文に紐づいたプロモーションは、条件を外れたときに取り消す（deactivate）ため除外しない
       def exclude_promotion_ids
+        unmatched_product_rule_promotion_ids - order.promotion_ids
+      end
+
+      def unmatched_product_rule_promotion_ids
         cart_product_ids = order.line_items.joins(:variant).pluck('spree_variants.product_id').uniq
         # Find all promotions with Product rules
         all_product_rule_promotions = Spree::Promotion

data/app/views/themes/default/spree/orders/_summary.html.erb

@@ -1,6 +1,8 @@
 <%# @gem-override spree_storefront-5.3.6/app/views/themes/default/spree/orders/_summary.html.erb %>
-<%# reason: カートサマリの小計・割引・合計表示を adjustments ベースに組み替え、買い物を続けるボタンを追加している %>
+<%# reason: カートサマリの小計・割引・合計表示を adjustments ベースに組み替え、買い物を続けるボタンを追加している。空のクイックチェックアウト欄を出さず、スマホでボタンを横幅いっぱいにしている %>
+<%#   (「レジに進む」を押したらカートをローディング状態にして連打を防ぐ) %>
 <%# @see https://github.com/be-agile/giga-repeat/issues/698 %>
+<%# @see https://github.com/be-agile/giga-repeat/issues/1390 %>
 <%= turbo_frame_tag :cart_summary do %>
   <% if order.line_items.any? %>
     <div class="cart-summary-container p-4">
@@ -53,7 +55,9 @@
 
 
       <div class="flex <%= !turbo_frame_request? && !turbo_stream_request? ? 'flex-row' : 'flex-col' %> gap-4 mt-4 w-full justify-end items-end">
-        <% if quick_checkout_enabled?(@order) %>
+        <%# quick_checkout_enabled? は注文の条件しか見ないので、Apple Pay 等のボタンのパーシャルが1つも登録されていないと
+            中身が空の w-full の欄だけが出て、隣の「レジに進む」等のボタンの幅を半分奪う。パーシャルがあるときだけ出す %>
+        <% if quick_checkout_enabled?(@order) && Spree.storefront.partials.quick_checkout.any? %>
           <div class="min-h-[46px] w-full has-[turbo-frame]:block group no-separator">
             <% if !turbo_frame_request? && !turbo_stream_request? %>
               <%= render partial: 'spree/checkout/quick_checkout', locals: { height: 46, theme: 'black', button_width: 320, container_class: "w-full flex justify-end" } %>
@@ -62,21 +66,28 @@
             <% end %>
           </div>
         <% end %>
-        <div style="display: flex; flex-direction: column; align-items: center; gap: 1rem;<%= ' width: 100%;' if turbo_frame_request? || turbo_stream_request? %>">
+        <%# 幅を指定しないと中身に合わせて縮み、日本語は1文字ごとに改行できるのでボタンが極端に細くなる。
+            スマホではボタンを横幅いっぱいに広げ、PC では幅 320px（lg:w-80）でページなら右寄せ・スライドオーバーなら中央にする。
+            ba_spree のビューは storefront の Tailwind のビルド対象外なので、クラスは本家のビューで使われていてビルドに含まれるものだけを使う %>
+        <div class="<%= turbo_frame_request? || turbo_stream_request? ? 'lg:items-center' : 'lg:items-end' %>" style="display: flex; flex-direction: column; gap: 1rem; width: 100%;">
+          <%# レジに進むと確認画面まで自動で進める(spree_checkout_auto_advance)分だけ待ち時間が長いので、数量の変更時と同じく
+              カートを操作できなくしてローディングを出す(連打も防ぐ)。ブラウザの戻るで復元されたときは元に戻す (#1390) %>
           <%= link_to Spree.t(:checkout), spree.checkout_path(order.token),
           class: 'btn-primary block w-full lg:w-80 text-center !no-underline !mt-0',
-          data: { 'turbo': 'false', 'cart-target': 'checkoutButton' } %>
+          data: { 'turbo': 'false', 'cart-target': 'checkoutButton', action: 'click->cart#disableCart pageshow@window->cart#enableCart' } %>
 
           <% # Continue Shopping button %>
           <% if turbo_frame_request? || turbo_stream_request? %>
             <%# Modal context: close the slideover %>
             <%= button_tag Spree.t(:continue_shopping),
               type: 'button',
+              class: 'lg:w-80',
               style: 'display: inline-block; text-align: center; text-decoration: none; background-color: white; padding: 0.75rem 1.25rem; font-weight: 600; font-size: 0.875rem; line-height: 1.25rem; text-transform: uppercase; letter-spacing: 0.05em; cursor: pointer; border-radius: var(--button-border-radius); border: var(--button-border-width) solid var(--button-background-color, var(--button)); color: var(--button-background-color, var(--button));',
               data: { action: 'slideover#toggle' } %>
           <% else %>
             <%# Full-page cart context: redirect to products %>
             <%= link_to Spree.t(:continue_shopping), spree.products_path,
+              class: 'lg:w-80',
               style: 'display: inline-block; text-align: center; text-decoration: none; background-color: white; padding: 0.75rem 1.25rem; font-weight: 600; font-size: 0.875rem; line-height: 1.25rem; text-transform: uppercase; letter-spacing: 0.05em; cursor: pointer; border-radius: var(--button-border-radius); border: var(--button-border-width) solid var(--button-background-color, var(--button)); color: var(--button-background-color, var(--button));',
               data: { 'turbo-frame': '_top' } %>
           <% end %>

data/lib/ba_spree/version.rb

@@ -1,3 +1,3 @@
 module BaSpree
-  VERSION = "1.1.0"
+  VERSION = "1.2.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ba_spree
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - be agile Co., Ltd.
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-09-30 00:00:00.000000000 Z
+date: 2026-10-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree
@@ -402,6 +402,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: spree_promotion_action_create_line_items
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: gem_override_marker
   requirement: !ruby/object:Gem::Requirement
@@ -431,6 +445,8 @@ files:
 - app/assets/stylesheets/spree/admin/custom.scss
 - app/controllers/ba_spree/checkout_controller_decorator.rb
 - app/controllers/ba_spree/store_controller_decorator.rb
+- app/controllers/ba_spree/storefront_api_products_controller_decorator.rb
+- app/controllers/ba_spree/storefront_api_variants_controller_decorator.rb
 - app/controllers/concerns/ba_spree/theme_concern_decorator.rb
 - app/controllers/spree/admin/assets_controller_decorator.rb
 - app/controllers/spree/admin/products_controller_decorator.rb
@@ -444,6 +460,7 @@ files:
 - app/decorators/spree/money_decorator.rb
 - app/decorators/spree/order_email_subscriber_decorator.rb
 - app/decorators/spree/page_sections/featured_posts_decorator.rb
+- app/finders/ba_spree/products/find_decorator.rb
 - app/helpers/currency_rounding_helper.rb
 - app/helpers/spree/admin/filter_params_preservation.rb
 - app/helpers/spree/admin/navigation_helper_decorator.rb