Ticket #1879 (closed バグ指摘: 修正済)
検索のページングが動作しない
Reported by: | pineray | Owned by: | pineray |
---|---|---|---|
Priority: | 中 | Milestone: | EC-CUBE2.12.1 |
Component: | フロント | Version: | 2.12.0 |
Keywords: | Cc: | ||
修正済み: | yes |
Description
フロントの検索結果が複数ページにわたるとき、ページャーをクリックすると全商品の一覧になってしまい、検索条件が引き継がれない。
http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=10849&forum=8
Change History
comment:1 Changed 12 years ago by pineray
- Owner changed from somebody to pineray
- 修正済み set
- Status changed from new to assigned
comment:2 follow-up: ↓ 3 Changed 12 years ago by pineray
説明補足
ページャーが動作しないのは、 r21856 にて sakamoto さんが加えた変更が原因でした。
この変更は、一覧ページから商品をカゴに入れた後ブラウザバックで戻ると、商品ID等の hidden を保持したままになることがあり、その状態でページャーをクリックしても、商品をカゴに入れる動作となってしまう不具合を修正するためのものでした。
ただ、コードに不備があり、上記の動作を修正できないことに加え、ページャーも動作しなくなっていました。
r21856 が目的としていた修正を取り込みつつ、ページャーが動作するようにするには、
・ページャーで移動する際に商品IDの hidden を削除する ( r21856 が意図していた動作 )
・form1 からの送信ではなく商品毎のフォームから送信する
といった方法が考えられ、そのうち変更するファイルが最も少ないだろうという理由で、後者の方法を採用し、修正しました。
comment:3 in reply to: ↑ 2 ; follow-up: ↓ 4 Changed 12 years ago by h_yoshimoto
pineray さまへの返信
ご対応ありがとうございます!
PCで正常に動作する事が確認出来ました。
モバイルについては下記のチケットに記載しています。
http://svn.ec-cube.net/open_trac/ticket/1726
本チケットが原因かは不明ですがスマホで同様の動作をした場合、parserエラーが発生する様です。
こちらも併せてご確認よろしくお願いいたします。
comment:4 in reply to: ↑ 3 ; follow-up: ↓ 5 Changed 12 years ago by pineray
h_yoshimoto への返信
ご確認ありがとうございます。
モバイルに関しても修正し、上記チケットにてご報告しております。
ただ、本チケットで手を加えたファイルはPC用の商品一覧画面のテンプレートだけなので、スマートフォンでエラーが発生するのは別の問題だと思います。
私の環境で確認すると、fnSetClassCategories関数が無いというエラーは表示されますが、「もっとみる」は動作していますね。
r21940 にて対応しました。