[カテゴリ] 表示順に文字列を入力した場合にバリデーションエラーとなるようにしました

[masaton0216]

概要

カテゴリ設定の表示順に「a123」等の文字列を含む値を入力した場合、PHPの intval() により 0 に変換されて登録される、またはPHPバージョンやMySQL strict modeの設定によっては500エラーになる問題がありました。

表示順のバリデーションルールに numeric を追加し、数値以外の入力をエラーとして弾くようにしました。

対象箇所：

• サイト管理＞カテゴリ設定（追加行・既存行）
• 各プラグインのカテゴリ設定（ブログ、FAQ、データベース、リンクリスト、課題管理）の追加行・既存行・共通カテゴリ行

なお、施設予約のカテゴリは独自バリデーションで元から numeric が設定済みのため変更なしです。

手動テストプラン

サイト管理＞カテゴリ設定

• 追加行：表示順に「a123」を入力して保存 → バリデーションエラーになること
• 追加行：表示順に「12a3」を入力して保存 → バリデーションエラーになること
• 追加行：表示順に「10」を入力して保存 → 正常に登録されること
• 既存行：表示順を「abc」に変更して保存 → バリデーションエラーになること
• 既存行：表示順を「5」に変更して保存 → 正常に更新されること

プラグイン＞カテゴリ設定（ブログで代表確認）

• 追加行：表示順に「a123」を入力して保存 → バリデーションエラーになること
• 追加行：表示順に「10」を入力して保存 → 正常に登録されること
• 既存行（プラグイン固有カテゴリ）：表示順を「abc」に変更して保存 → バリデーションエラーになること
• 既存行（プラグイン固有カテゴリ）：表示順を「5」に変更して保存 → 正常に更新されること
• 既存行（共通カテゴリ）：表示順を「abc」に変更して保存 → バリデーションエラーになること
• 既存行（共通カテゴリ）：表示順を「5」に変更して保存 → 正常に更新されること

他プラグインでの確認

• FAQ：カテゴリ追加行で文字列入力 → バリデーションエラーになること
• データベース：カテゴリ追加行で文字列入力 → バリデーションエラーになること
• リンクリスト：カテゴリ追加行で文字列入力 → バリデーションエラーになること
• 課題管理：カテゴリ追加行で文字列入力 → バリデーションエラーになること

施設予約（変更なし・デグレ確認）

• カテゴリ追加行で文字列入力 → 従来通りバリデーションエラーになること
• カテゴリ追加行で数値入力 → 従来通り正常に登録されること

レビュー完了希望日

不具合対応なので急ぎたいです

関連Pull requests/Issues

参考

DB変更の有無

無し

チェックリスト

• プルリクエストにわかりやすいタイトルとラベルを付けました。https://github.com/opensource-workshop/connect-cms/wiki/Pull-requests-Rule