Ticket #1449 (closed 改善提案: 修正済)
不要な関数・処理の整理
Reported by: | shutta | Owned by: | somebody |
---|---|---|---|
Priority: | 低 | Milestone: | EC-CUBE2.11.5 |
Component: | その他 | Version: | 2.11.2 |
Keywords: | Cc: | ||
修正済み: | yes |
Change History
comment:2 follow-up: ↓ 7 Changed 12 years ago by shutta
どこからも参照されていないSC_Utils::sfSendPostDataを削除。(r21242)
comment:7 in reply to: ↑ 2 Changed 12 years ago by Seasoft
shutta への返信
どこからも参照されていないSC_Utils::sfSendPostDataを削除。(r21242)
複数の決済モジュールで利用があるようです。
- http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=9139&forum=10
- http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=9106&forum=8
コアMLにて2.11系の大きな方向性として「各種公式モジュールやデザインテンプレートの本体への依存度をなくす。(もしくは少なくする)」という、話しもありましたので、方向性としては正しいように感じます。
また、「内部でインスタンスを生成している関数は, Helper クラスへ移動するべき」という指針にも反するメソッドなので、削除は妥当なようには感じます。
ただし、決済モジュールが追随できるか、その辺りが不明確なのでなんとも・・・ しばらくは @deprecated 辺りかなぁ・・・
# まぁ、しかし残すにしても、不可解な UA とか、何とかしたいところですね。EC-CUBE を名乗るべきかと。無難な携帯端末を偽装して抜けれるような脆弱な決済システムでもあったんだろか・・・
comment:8 Changed 12 years ago by shutta
- Description modified (diff)
r21242 の修正による問題に関しては、[comu-ml:231]にて、イプシロン決済モジュールのみ影響し、そのモジュール側で対応されるとのこと。
comment:12 Changed 11 years ago by kajiwara
- Status changed from new to closed
- 修正済み set
- Resolution set to 修正済
2.11系は一旦ここまでで、クローズさせていただきます。
Note: See
TracTickets for help on using
tickets.
r21203 にて、SC_Utils:sfReload()を削除。