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

【インストーラー】InstrationsController内にある一部のコードをBcManagerComponentに持っていくリファクタリング #1167

Open
tnagatomi opened this issue Apr 29, 2019 · 2 comments
Assignees
Labels
Review レビュー要 Ver4 バージョン4

Comments

@tnagatomi
Copy link
Contributor

(このIssueはRedmineのチケットから移行されています)

作成者: 滝下 真玄
作成日: 2013/08/08 03:04

その1:
InstrationsController::deleteAllTables()の内容をBcManagerComponentに持っていけないでしょうか?

その2:
同じく、InstrationsController::index()の内容(tmpファイルのクリーンナップ)をBcManagerComponentに持っていけないでしょうか?

どちらも、環境リセット用として使いたいのですが、アクションに直書きされてしまっているのでコピペするしかありません

その3:
その1でInstrationsController::_readDbSettingFromSession()も必要になるのでBcManagerComponentに持っていけないでしょうか?

その4:
その2の機能の一部がbasics.phpにある「clearAllCache()」と同機能のようなので、clearAllCache()を使用するリファクタリングをした方が良いように思います

@tnagatomi
Copy link
Contributor Author

投稿者: 滝下 真玄
投稿日: 2013/08/07 18:06:58

理由を書くのを忘れました。Seleniumテストで環境をリセットする仕組みが必要になり、インストール時に処理している初期化処理を流用したいためです。

@tnagatomi
Copy link
Contributor Author

投稿者: ナカエ -
投稿日: 2015/03/14 02:16:01

BcManagerComponentは現状でも大きすぎると思います。

Componentじゃなくてインストール専用のクラス(例えばBcInstallerなど)になりませんかね?まあBcInstallerComponentでもいいんですが、これComponentに書く意味そんなにあります?BcManagerShellの処理とcheckenv()の部分も共通化したいです。

@baserproject baserproject deleted a comment from tnagatomi May 12, 2019
@baserproject baserproject deleted a comment from tnagatomi May 12, 2019
@baserproject baserproject deleted a comment from tnagatomi May 12, 2019
@ryuring ryuring added the Review レビュー要 label May 12, 2019
@ryuring ryuring changed the title InstrationsController内にある一部のコードをBcManagerComponentに持っていくリファクタリング 【インストーラー】InstrationsController内にある一部のコードをBcManagerComponentに持っていくリファクタリング May 5, 2020
@ryuring ryuring added the Ver4 バージョン4 label Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review レビュー要 Ver4 バージョン4
Projects
None yet
Development

No branches or pull requests

4 participants