EC-CUBE / ec-cube2

EC-CUBE official repository version 2
https://www.ec-cube.net
Other
86 stars 97 forks source link

adminテンプレートにあるペイジェントモジュール連携用のコードは意図したもの? #967

Closed clicktx closed 1 month ago

clicktx commented 1 month ago

https://github.com/EC-CUBE/ec-cube2/blob/5cfe357db66d876607742ceabcb4603674f8aa96/data/Smarty/templates/admin/order/index.tpl#L323-L327

相当前からこのコードが残っているのですが、これは意図して残してあるものなのでしょうか? 511cc24781d295b8c99c33515fdb7b18485be5eb までは無かったみたいですが、 7f423c22c8faf4d9844344da078d6b25f81342b5 から入っているようです。

seasoftjapan commented 1 month ago

直近のモジュール (2023-08 頃) だと mdl_paygent\files\templates\admin\paygent_order_index.tpl に移動している様子なので、空回りしていそうですね。

2011-10 頃のモジュール (2.4向け?) では存在しますね。

clicktx commented 1 month ago

2.4向け

そうなのですね。2.4用のペイジェントモジュールのコードは入手できないので確認できませんが、2.17用のコードを斜め読みした感じだと該当のページのテンプレートはそっくり置き換えているっぽいので必要なさそうに見えますね。

nanasess commented 1 month ago

@clicktx @seasoftjapan 他にも過去のモジュール連携用のコードがいくつかあるのですが、すべて不要だと思いますので PR いただけると助かります🙏 https://github.com/search?q=repo%3AEC-CUBE%2Fec-cube2%20smarty.const.MODULE_REALDIR&type=code

nanasess commented 1 month ago

972 でマージしましたのでクローズします