Controller周辺のリファクタリング、型の厳格化、コードのデカップリング等 #108 #256

[uzulla]

rel #108 #256

• Controller 親クラスの責務を分割し、処理を必要なクラスに移譲。
• 全 Controller に declare(strict_types=1); を付与し、暗黙の型変換を抑制、型変換を明示的に記述
• 存在しないactionへのアクセスを404にするために__callをつかっていたのを廃止（すでにRouterでフィルター済み）
• Captchaでしかつかわれていないコードの統廃合
• すでに不要となっていたメソッドの消し込み
• モデルでインスタンスとして多用される配列を、配列としてもアクセスできる構造体的なRowクラスへの変更 #84 配列として扱える、Rowモデルクラス関連の適用拡大と改善
• FatになっていたBlogTemplatesModel からテンプレート関連のコンパイル、文法チェック等のロジックを Fc2BlogTemplate に分割し、静的解析に対する透明性の向上。
• Controller以外も declare(strict_types=1); の範囲を拡大
• Admin系のgetBlogId 不要な引数の削減と、名称変更
• 古いチェックコードから、 Request::isValidSig() に書き換えて揃えた
• Typoの修正
• 一部アクセス修飾子をPrivateやProtectedに是正
• 検討の結果、不要となったTODOの削除
• 500のエラー画面がない #256 500エラー画面を改善、デバッグモード時にtext/html以外の場合はワーニングをださないように修正
• UserController::getBlogId を $request->getBlogId() に変更し、削除
• ディスパッチャとコントローラ（アクション）の分離 #108 Router のリファクタリング、不要なバリデーションを削除。各種Controllerのアクセス修飾子整理とあわせて、Router自体のリファクタリングを一旦完了とする
  • （ 各種URLにおける期待されるhttp methodの明示と厳格化 #279 HTTP methodの厳格化はControllerの大規模リファクタ（分割）が必要なので、バリデーションをControllerにいれる予定）

作業時間 7h