[TEMP] 管理者はメンテナンスモードを回避可能に変更

[izayoi256]

概要(Overview・Refs Issue)

#4961 を参考までに実装

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

新機能追加に伴う競合確認

• プラグインとの競合がないか
• プラグインからのマイグレーションが必要か

[chihiro-adachi]

@izayoi256
ありがとうございます。メンテナンス中のアラートがでるのいいですね。

[chihiro-adachi]

メンテナンス管理からの動作は確認しました。
#4960 の記載通り動作しています。

プラグインインストール時の動作を確認していきます。

[okazy]

管理画面ログアウト時の挙動が変わっています。
今まではログアウト後にログイン画面に遷移していましたが、フロント画面のロップページに遷移するようになっています。

E2Eテストの以下のところで落ちてしまいます。

ec-cube/codeception/_support/AcceptanceTester.php

Line 72 in 23da54c

$I->see('ログイン', '#form1 > button');

特に理由がなければ遷移先は変えたくありません。

[matsuoshi]

既存のメンテナンスプラグインをインストール・有効化した状態で動作させてみましたが、特に問題はありませんでした

[nanasess]

public メソッドの互換性維持のため、 既存メソッドに対する返り値の型指定追加は避けた方が良いと思われます

Suggested change

	public function switchMaintenance($isEnable = false, $mode = self::AUTO_MAINTENANCE, bool $force = false): void
	public function switchMaintenance($isEnable = false, $mode = self::AUTO_MAINTENANCE, bool $force = false)

[izayoi256]

修正してpushしました。

[okazy]

対応ありがとうございます！
画面遷移に関わる変更なので、E2Eテストも実装しておきたいと思いました。

[izayoi256]

@okazy

案件が入り、立て込んでしまっているため、E2Eテストの着手までしばらくかかりそうです。

[chihiro-adachi]

@izayoi256

#5123 のPRを取り込ませていただきました。
ご対応ありがとうございました。