ContentFieldType::Numberのバリデーション不具合

Description

小数点以下の桁数を設定せずに、最小値または最大値に文字列を設定するとバリデーションを通過してしまうようです。

管理画面で確認したところ、正常に動作しているようにみえましたが、意図しない形で検証に成功しているようです。(どこかで一度でも正規表現がキャプチャされた場合、`defined $1` がtrueになるため成功する。以下参照)

https://github.com/movabletype/movabletype/blob/master/lib/MT/ContentFieldType/Number.pm#L114-L118

Environment

None

Checklists

None

Activity

Show:

Kenichi IshigakiDecember 12, 2023 at 7:30 AM

ありがとうございます。QAはdevelop側で済んでいるものとしてCI完了後にマージします。

Keita SugamaDecember 12, 2023 at 7:28 AM

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

Kenichi IshigakiDecember 12, 2023 at 7:17 AM

をもとに MT8.0 用の PR を作成しました。念のためレビューおねがいします。

Kenichi IshigakiDecember 12, 2023 at 7:13 AM

ありがとうございます。だいぶ時間がたってしまいましたが QA 済みとのことでマージしました。

Chieko MatsuoNovember 16, 2023 at 7:41 AM

これまでの動作と変わらないことを確認しました。

Fixed

Details

Assignee

Reporter

Fix versions

Priority

Smart Checklist

Created November 9, 2023 at 2:14 AM
Updated December 12, 2023 at 7:30 AM
Resolved November 14, 2023 at 1:21 AM