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

SC_CartSession::setPrevURL() の呼び出しを削除する #805

Closed
seasoftjapan opened this issue Dec 21, 2023 · 1 comment · Fixed by #807
Closed

SC_CartSession::setPrevURL() の呼び出しを削除する #805

seasoftjapan opened this issue Dec 21, 2023 · 1 comment · Fixed by #807
Assignees
Milestone

Comments

@seasoftjapan
Copy link
Contributor

ba4a323 以後、相方の getPrevURL() は使われていない。(厳密には、テストのみで使われている。)

処理内容から見て、従来使われていた用途以外には使い所はなく、まるっと削除しても支障無さそうだが、念のため (getPrevURL() も含めて) メソッドは非推奨を記載して残して、呼び出しを削除する方向で考えている。

  • SC_CartSession::setPrevURL() は、ブロックの処理からも呼ばれているが、明らかに無駄である。(TOP画面で実測すると、17回呼び出されていた。) 管理画面・エラー画面・POST・リダイレクトでも呼ばれる。これはバグと判断している。
  • 複数のクラスのコンストラクタを複雑にする要因にもなっていそう。
  • 現在のカゴの中 [戻る]ボタン 購入手続きへ遷移することがある #803 で呼び出しの復活を検討したが、上述の改善を行うと実装が複雑になりそうなので断念した。
@seasoftjapan
Copy link
Contributor Author

決済モジュール・プラグインで有効に1 使われている状況をご存知でしたら、教えてください。

Footnotes

  1. メソッドは残すので、意味なく呼ばれているパターンは無視できる認識です。

@seasoftjapan seasoftjapan self-assigned this Dec 22, 2023
seasoftjapan added a commit to seasoftjapan/eccube-2_13 that referenced this issue Dec 29, 2023
ji-eunsoo added a commit that referenced this issue Feb 14, 2024
SC_CartSession::setPrevURL() の呼び出しを削除する #805
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant