Ticket #545 (closed バグ指摘: 修正済)
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 14 years ago by Yammy
- Owner changed from somebody to Yammy
- Status changed from new to assigned
comment:2 Changed 14 years ago by Yammy
- Status changed from assigned to closed
- Resolution set to 修正済
商品一覧、詳細ページからの遷移でも無視されている。 そもそも定数自体使用されていないので、使用するように変更。 それにともない、全く使用されていない、ページクラスの変数などを削除。
r18465 としてコミット
comment:4 in reply to: ↑ 3 ; follow-up: ↓ 5 Changed 14 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 14 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:7 Changed 14 years ago by kajiwara
- Owner changed from Yammy to kajiwara
- Status changed from reopened to new
comment:10 Changed 14 years ago by nanasess
デフォルトで登録されるアイスクリームの購入制限フラグが NULL になっていたため, デフォルトの状態で SALE_LIMIT_MAX を設定しても反映されませんでした r18704 で, アイスクリームのデフォルト値を修正しました.
comment:11 Changed 14 years ago by nanasess
- Status changed from assigned to closed
- Resolution set to 修正済
version-2_4 r18705 でマージしました