Ticket #545 (closed バグ指摘: 修正済)

Opened 11 years ago

Last modified 11 years ago

SALE_LIMIT_MAX が無視されている

Reported by: Seasoft Owned by: nanasess
Priority: Milestone: EC-CUBE2.4.4
Component: フロント Version: 2.4.1
Keywords: Cc:
修正済み:

Description

パラメータ「SALE_LIMIT_MAX」の設定値が、少なくとも「現在のかごの中」画面の[+]ボタンで、無視されることを確認。

Change History

comment:1 Changed 11 years ago by Yammy

  • Owner changed from somebody to Yammy
  • Status changed from new to assigned

comment:2 Changed 11 years ago by Yammy

  • Status changed from assigned to closed
  • Resolution set to 修正済

商品一覧、詳細ページからの遷移でも無視されている。 そもそも定数自体使用されていないので、使用するように変更。 それにともない、全く使用されていない、ページクラスの変数などを削除。

r18465 としてコミット

comment:3 follow-up: ↓ 4 Changed 11 years ago by nanasess

r18516 で version-2_4 へマージ

comment:4 in reply to: ↑ 3 ; follow-up: ↓ 5 Changed 11 years ago by shutta

  • Status changed from closed to reopened
  • Resolution 修正済 deleted

r18465 だと、

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=flat&topic_id=5477&forum=6[[BR]]

のように不具合があるので、r18583 にて修正してみました。

ただ、SALE_LIMIT_MAXパラメータの説明が「購入制限なしの場合の最大購入個数」とだけで、 どういう仕様を意図したものかが不明なので、実装方法を間違えているかもしれません。

・在庫制限あり&購入制限あり
-> 制限の小さい方が購入可能最大数
・在庫制限なし&購入制限あり
-> 購入制限が購入可能最大数
・在庫制限あり&購入制限なし
-> SALE_LIMIT_MAXと在庫数の小さい方が購入可能最大数
・在庫制限なし&購入制限なし
-> SALE_LIMIT_MAXが購入可能最大数

となるように実装しましたが、意図としては、SALE_LIMIT_MAXを利用するのは、 在庫制限なし&購入制限なし(=何も制限がない)の時だけを想定してましたでしょうか?

comment:5 in reply to: ↑ 4 Changed 11 years ago by shutta

[comu-ml:87]の仕様に沿うように修正。(r18601)

【SALE_LIMIT_MAX】

■説明文:購入制限なしの場合の最大購入個数(0の場合は無制限)

■初期値:0

■使用方法:以下が初心者の方でも使いやすいかと思いました。
  if (SALE_LIMIT_MAX > 0) {
      SALE_LIMIT_MAX を購入制限数として使用するように設定
  }

■実際の処理:
①在庫制限あり&購入制限あり
  -> 制限の小さい方が購入可能最大数
②在庫制限なし&購入制限あり
  -> 購入制限が購入可能最大数
  
③在庫制限あり&購入制限なし
  -> SALE_LIMIT_MAXと在庫数の小さい方が購入可能最大数
   (上記「使用方法」のif文を使用)
④在庫制限なし&購入制限なし
  -> SALE_LIMIT_MAXが購入可能最大数
   (上記「使用方法」のif文を使用)

comment:6 Changed 11 years ago by kajiwara

  • Milestone changed from EC-CUBE2.4.3 to EC-CUBE2.4.4

comment:7 Changed 11 years ago by kajiwara

  • Owner changed from Yammy to kajiwara
  • Status changed from reopened to new

comment:8 Changed 11 years ago by nanasess

  • Owner changed from kajiwara to nanasess

comment:9 Changed 11 years ago by nanasess

  • Status changed from new to assigned

comment:10 Changed 11 years ago by nanasess

デフォルトで登録されるアイスクリームの購入制限フラグが NULL になっていたため, デフォルトの状態で SALE_LIMIT_MAX を設定しても反映されませんでした r18704 で, アイスクリームのデフォルト値を修正しました.

comment:11 Changed 11 years ago by nanasess

  • Status changed from assigned to closed
  • Resolution set to 修正済

version-2_4 r18705 でマージしました

Note: See TracTickets for help on using tickets.