MT7で整理され使われなくなった image_defaults による条件分岐に廃止の警告を追加する

Description

https://movabletype.atlassian.net/browse/MTC-29967 より派生。

MT6までは tmpl/cms/dialog/asset_options_image.tmpl にimage_defaults という名のチェックボックスがあり、save_image_defaults という権限があれば MT::Asset::Image の https://github.com/movabletype/movabletype/blob/fe257947fcda8349cf57c46e70d5801d6596e5d8/lib/MT/Asset/Image.pm#L548-L568 を経て、いまはサイトの投稿設定にあるサムネイルのデフォルト値を変更できていた。

ただし、 MT7 でこのあたりの整理が進み、いまでは image_defaults をふくむ tmpl/cms/dialog/asset_options_image.tmpl 自体が削除され、MT::Asset::Image の該当部分の条件分岐も無用のものとなっている。サードパーティが利用している可能性も考慮して、まずは該当部分に MT::Util::Deprecated による警告を追加する。

Checklists

None

Activity

Show:

Kenichi IshigakiNovember 21, 2024 at 3:03 AM

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

Chieko MatsuoNovember 21, 2024 at 3:01 AM

問題は見受けられませんでした。

Kenichi IshigakiNovember 21, 2024 at 2:15 AM

ありがとうございます。

https://github.com/movabletype/movabletype/pull/3451 で内部的に使われていないはずのコードが万一使われた場合に警告を出すようにしました。表に見える変化は一切ないはずですが、念のため記事や旧ブロックエディタを使ったコンテンツデータで画像挿入が問題なく行えることをご確認おねがいします。

Keita SugamaNovember 21, 2024 at 1:50 AM

確認しました。良さそうです。

Kenichi IshigakiNovember 20, 2024 at 8:41 AM

https://github.com/movabletype/movabletype/pull/3451 で対応しました。念のためレビューおねがいします。

Implemented

Details

Assignee

Reporter

Fix versions

Priority

Smart Checklist

Created November 20, 2024 at 8:22 AM
Updated November 21, 2024 at 3:03 AM
Resolved November 21, 2024 at 3:03 AM