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

Opened 9 years ago

Last modified 9 years ago

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

Seasoft への返信

#1815

上記チケットは受注管理側で、こちらはフロントのカートに入れる際の不具合なので、少し現象は違うかなと思い、別チケットを切っておりました。

ひとまず、フロント側(本チケット)は、 r21893 にて修正してみました。

続いて、#1815 の方も見てみます。

comment:3 Changed 9 years ago by shutta

  • Owner changed from somebody to shutta
  • Status changed from reopened to new

comment:4 Changed 9 years ago by shutta

  • Status changed from new to assigned

comment:5 in reply to: ↑ 2 Changed 9 years ago by Seasoft

承知いたしました。

ご対応、コメントありがとうございます。

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
Note: See TracTickets for help on using tickets.