Ticket #1852 (closed バグ指摘: 修正済)
1規格のみ設定された商品をカートに入れると、選択した分類が無視される
Reported by: | shutta | Owned by: | shutta |
---|---|---|---|
Priority: | 中 | Milestone: | EC-CUBE2.12.1 |
Component: | フロント | Version: | 2.12.0 |
Keywords: | Cc: | ||
修正済み: | yes |
Description
コミュニティよりバグ報告あり。
Ver2.12で選択した分類の商品がカートに入らない(規格1のみの設定) http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=10708&forum=10
1商品に分類を複数登録して、プルダウンで選択しカートに入れる際、選択した分類ではなく、先頭の分類が選択されます。 EC-CUBE Ver2.11.5では、選択した分類が正しくカートに入ります。 商品詳細ページのPHP側は、特に改修は行なっていません。
Change History
comment:1 follow-up: ↓ 2 Changed 9 years ago by Seasoft
- Status changed from new to closed
- Resolution set to 既出
comment:2 in reply to: ↑ 1 ; follow-ups: ↓ 5 ↓ 6 Changed 9 years ago by shutta
- Status changed from closed to reopened
- Resolution 既出 deleted
comment:3 Changed 9 years ago by shutta
- Owner changed from somebody to shutta
- Status changed from reopened to new
comment:6 in reply to: ↑ 2 ; follow-up: ↓ 7 Changed 9 years ago by Seasoft
ひとまず、フロント側(本チケット)は、 r21893 にて修正してみました。
$sele2.val() == undefined だと、冗長な評価となりそうですので、他と同様に length プロパティで評価する方がストレートかなと感じました。
もしかすると、実装意図を正しく組み取れていない部分があるかもしれませんが、所感にて。
comment:7 in reply to: ↑ 6 ; follow-up: ↓ 8 Changed 9 years ago by shutta
$sele2.val() == undefined だと、冗長な評価となりそうですので、他と同様に length プロパティで評価する方がストレートかなと感じました。
もしかすると、実装意図を正しく組み取れていない部分があるかもしれませんが、所感にて。
ご指摘有難うございます。 lengthを使用するように修正しておきました。(r21895)
comment:8 in reply to: ↑ 7 Changed 9 years ago by h_yoshimoto
shutta さまへの返信
いつもお世話になっております。
株式会社ロックオン吉本です。
こちら動作確認しましたところ、モバイルでも同様の現象が発生しています。
取り急ぎ共有させて頂きます。
PC、スマホでの修正は確認できております。
引き続き調査させて頂きます。
comment:9 Changed 9 years ago by h_yoshimoto
r21899 にて対応しました。
また規格1のみの場合は、setClassCategories() を呼ばない様に修正入れています。
comment:10 Changed 9 years ago by h_yoshimoto
- Status changed from assigned to closed
- 修正済み set
- Resolution set to 修正済
- Milestone changed from EC-CUBE2.12.2 to EC-CUBE2.12.1
#1815