Ticket #1449 (closed 改善提案: 修正済)

Opened 9 years ago

Last modified 9 years ago

不要な関数・処理の整理

Reported by: shutta Owned by: somebody
Priority: Milestone: EC-CUBE2.11.5
Component: その他 Version: 2.11.2
Keywords: Cc:
修正済み: yes

Description (last modified by Seasoft) (diff)

使用されなくなっている関数や、処理・ロジックを整理する。 ざっくりとした概要ですみません。

但し、下記指針を考慮する。

 ・マイナーバージョンアップではメソッドやファイルを削除しない、
  メソッドの引数の変更といった外部参照方法が変わる変更は行わない。
 (追加や、不具合修正のような内容修正は可)

文言に関しては、#1479, #1526 で扱う。

r21230 r21231 r21232 r21327

Change History

comment:1 Changed 9 years ago by shutta

r21203 にて、SC_Utils:sfReload()を削除。

comment:2 follow-up: ↓ 7 Changed 9 years ago by shutta

どこからも参照されていないSC_Utils::sfSendPostDataを削除。(r21242)

comment:3 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:4 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:5 Changed 9 years ago by kotani

  • Milestone changed from EC-CUBE2.11.3 to EC-CUBE2.11.4(仮)

comment:6 Changed 9 years ago by shutta

SC_Queryクラスのメンバー変数($instance)は使用されていないのでその定義部分を削除。(r21273)

comment:7 in reply to: ↑ 2 Changed 9 years ago by Seasoft

shutta への返信

どこからも参照されていないSC_Utils::sfSendPostDataを削除。(r21242)

複数の決済モジュールで利用があるようです。

コアMLにて2.11系の大きな方向性として「各種公式モジュールやデザインテンプレートの本体への依存度をなくす。(もしくは少なくする)」という、話しもありましたので、方向性としては正しいように感じます。

また、「内部でインスタンスを生成している関数は, Helper クラスへ移動するべき」という指針にも反するメソッドなので、削除は妥当なようには感じます。

ただし、決済モジュールが追随できるか、その辺りが不明確なのでなんとも・・・ しばらくは @deprecated 辺りかなぁ・・・

# まぁ、しかし残すにしても、不可解な UA とか、何とかしたいところですね。EC-CUBE を名乗るべきかと。無難な携帯端末を偽装して抜けれるような脆弱な決済システムでもあったんだろか・・・

comment:8 Changed 9 years ago by shutta

  • Description modified (diff)

r21242 の修正による問題に関しては、[comu-ml:231]にて、イプシロン決済モジュールのみ影響し、そのモジュール側で対応されるとのこと。

comment:9 Changed 9 years ago by shutta

r21308 にて、SC_CustomerListの携帯電話番号での検索処理部分を削除。

携帯電話番号情報(dtb_customerのcell01,cell02,cell03)は、 #628r19669 にて削除されており、検索項目として存在していないので不要。

comment:10 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:11 Changed 9 years ago by Seasoft

  • Description modified (diff)

comment:12 Changed 9 years ago by kajiwara

  • Status changed from new to closed
  • 修正済み set
  • Resolution set to 修正済

2.11系は一旦ここまでで、クローズさせていただきます。

Note: See TracTickets for help on using tickets.