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

商品を廃止状態にしても管理者の場合URLで直接アクセスすると閲覧可能 #5737

Closed
kurozumi opened this issue Aug 31, 2022 · 4 comments
Labels
improvement 機能改善
Milestone

Comments

@kurozumi
Copy link
Contributor

概要(Overview)

商品を廃止状態にしても管理者の場合URLで直接アクセスすると閲覧可能です。

期待する内容(Expect) or 要望 (Requirement)

廃止した商品を管理者のみですが閲覧できるのは正常でしょうか?
改善してほしいというお問い合わせがあったので確認です。

再現手順(Procedure)

・チェリーアイスサンドを廃止
・管理画面ログイン
https://domain/products/detail/2
・チェリーアイスサンドが閲覧可能

環境 (environment)

  • EC-CUBE: 4.0.x
  • PHP: 7.x.x
  • DB:
    • PostgreSQL x.x.x
    • MySQL x.x.x

関連情報 (Ref)

@chihiro-adachi
Copy link
Contributor

chihiro-adachi commented Aug 31, 2022

@kurozumi

同様の改善要望Issueはあがってますね。
#5006

@kurozumi
Copy link
Contributor Author

@chihiro-adachi

非表示の場合は商品一覧で表示され、廃止の場合は商品一覧で表示されないので、
非表示はメンテナンスモードのように表示させて、
廃止の場合は表示させないようにするのはどうでしょうか?

@chihiro-adachi chihiro-adachi added the improvement 機能改善 label Aug 31, 2022
@chihiro-adachi chihiro-adachi added this to the 4.2.0 milestone Aug 31, 2022
@chihiro-adachi
Copy link
Contributor

chihiro-adachi commented Aug 31, 2022

@kurozumi

非表示の場合は商品一覧で表示され、廃止の場合は商品一覧で表示されないので、

あれ、一覧はどちらも表示されないような気もします。。

非表示と廃止で実装分けるの面倒ではないですか?
商品のステータスが判別つかないのが課題だと思うので、管理者アクセスのメッセージ+商品のステータスを表示とかでもよいのかなと思いました。

※そこまでこだわりがあるわけではないので、実装が複雑にならなければご提案いただいた挙動でもよいかと思います。

@kurozumi
Copy link
Contributor Author

@chihiro-adachi

あ、ごめんなさい。
商品一覧とは管理画面の商品一覧のことです。
言葉足らずでした。

こだわりがないのでしたらどうするか考えてPRします。

@chihiro-adachi chihiro-adachi modified the milestones: 4.2.0, 4.2.x Sep 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement 機能改善
Projects
None yet
Development

No branches or pull requests

2 participants