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

Opened 9 years ago

Last modified 9 years ago

検索のページングが動作しない

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 9 years ago by pineray

  • Owner changed from somebody to pineray
  • 修正済み set
  • Status changed from new to assigned

r21940 にて対応しました。

comment:2 follow-up: ↓ 3 Changed 9 years ago by pineray

説明補足

ページャーが動作しないのは、 r21856 にて sakamoto さんが加えた変更が原因でした。

この変更は、一覧ページから商品をカゴに入れた後ブラウザバックで戻ると、商品ID等の hidden を保持したままになることがあり、その状態でページャーをクリックしても、商品をカゴに入れる動作となってしまう不具合を修正するためのものでした。

ただ、コードに不備があり、上記の動作を修正できないことに加え、ページャーも動作しなくなっていました。

r21856 が目的としていた修正を取り込みつつ、ページャーが動作するようにするには、
・ページャーで移動する際に商品IDの hidden を削除する ( r21856 が意図していた動作 )
・form1 からの送信ではなく商品毎のフォームから送信する
といった方法が考えられ、そのうち変更するファイルが最も少ないだろうという理由で、後者の方法を採用し、修正しました。

comment:3 in reply to: ↑ 2 ; follow-up: ↓ 4 Changed 9 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 9 years ago by pineray

h_yoshimoto への返信

ご確認ありがとうございます。

モバイルに関しても修正し、上記チケットにてご報告しております。

ただ、本チケットで手を加えたファイルはPC用の商品一覧画面のテンプレートだけなので、スマートフォンでエラーが発生するのは別の問題だと思います。
私の環境で確認すると、fnSetClassCategories関数が無いというエラーは表示されますが、「もっとみる」は動作していますね。

comment:5 in reply to: ↑ 4 Changed 9 years ago by h_yoshimoto

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

pineray さまへの返信

お世話になっております。

早急なご対応ありがとうございます!

先ほど修正を確認させて頂きました。
スマホの件は私の環境によるものでした。失礼いたしました。

本チケットはこれで閉じさせて頂きます。

ありがとうございました。

Note: See TracTickets for help on using tickets.