MT::App::Search(::TagSearch) の警告を減らす

Description

現状の MT は仕組み上 init_request がエラーを返してもそこで強制終了せずに次の run を実行するようになっている。MT::App::CMS の場合は init_request 内で問題が起きた場合は die しているので問題になっていないが、MT::App::Search は問題が起きたときに初期化を中断して return してしまうため、後続の処理の各所で uuv などの警告を引き起こしている。単純に die するように変えると Internal Server Error 表示になってしまうので、初期化の順序を入れ替えたり return するのをとりやめたりして必要な設定が確実に行われるようにする。

Checklists

None

Activity

Show:

Kenichi IshigakiDecember 6, 2023 at 2:15 AM

ありがとうございます。基本的なQAはdevelop側で済んでいるものとしてマージしました。

Keita SugamaDecember 6, 2023 at 1:02 AM

ありがとうございます。良さそうです。

Kenichi IshigakiDecember 5, 2023 at 10:46 AM

https://github.com/movabletype/movabletype/pull/2869 をもとに MT8.0 用の PR を作成しました。念のためレビューおねがいします。https://github.com/movabletype/movabletype/pull/2907

Kenichi IshigakiNovember 20, 2023 at 6:53 AM

ありがとうございます。マージしました。

Reiko SatoNovember 20, 2023 at 6:51 AM


2環境とも問題ないように見えます。

Implemented

Details

Assignee

Reporter

Fix versions

Priority

Smart Checklist

Created November 10, 2023 at 3:51 AM
Updated December 6, 2023 at 2:15 AM
Resolved November 20, 2023 at 6:53 AM