Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

商品一覧。存在しないカテゴリへアクセス時のエラーメッセージ #1658

Closed
tomothumb opened this issue Aug 2, 2016 · 6 comments
Assignees
Labels
enhancement 機能追加
Milestone

Comments

@tomothumb
Copy link
Contributor

商品一覧で、存在しないカテゴリへリンクさせた場合、
/products/list?category_id=xxxx

エラーメッセージは「有効な値ではありません」と出る。

文言例

  • ご指定のカテゴリは存在しません。
@ryo-endo
Copy link
Contributor

ryo-endo commented Aug 9, 2016

ありがとうございます。
ご提案いただいた文言(ご指定のカテゴリは存在しません。)が、2系と同じなので、それでいいと思います。

@ryo-endo ryo-endo added this to the 3.0.11 milestone Aug 9, 2016
@ryo-endo ryo-endo added the enhancement 機能追加 label Aug 9, 2016
@chihiro-adachi
Copy link
Contributor

chihiro-adachi commented Aug 23, 2016

こちら確認しましたが、このエラーメッセージは意図せず表示されている様子。

  • 検索パラメータをhiddenで保持するために、freeze_type_text=falseでフォームの描画を非表示にしている
  • が、エラーメッセージだけは、freeze_type_text=falseでも表示されてしまっている

状態のようです。※こちらは #1701

修正としては、

  • カテゴリが存在しない場合は、ページがありませんのエラー画面を表示(商品詳細画面に合わせる)
  • 現状では適切な表示ではないですが、404エラーページのカスタマイズ #1697 が取り込まれれば、2系と同様のレイアウト表示となります。

のように対応しようと思います。

@Yangsin
Copy link

Yangsin commented Nov 30, 2016

#1697 は採用を見送っているため実装方針から再検討が必要

@chihiro-adachi
Copy link
Contributor

存在しないカテゴリが指定された場合は、全てのカテゴリでの検索になっているため、こちらも考慮が必要

  • ProductControllerで、$searchForm['category_id']のエラーチェックを行う
  • エラーが有る場合は、ProductRepository::getQueryBuilderBySearchDataで、Categoryはnullでの検索を行う
  • list.twigで、表示の制御を行う
    • ○件の商品が見つかりました-> ご指定のカテゴリはありません

くらいの実装で可能かと思われます。

@ryo-endo
Copy link
Contributor

期待結果をまとめます

  • 「有効な値ではありません。」というメッセージは表示されない。
  • 「ご指定のカテゴリは存在しません。」という新規メッセージを表示する。
  • 検索結果は、紛らわしいので表示しない。

@ryo-endo
Copy link
Contributor

対応してmasterへの取り込み完了しました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 機能追加
Projects
None yet
Development

No branches or pull requests

5 participants