[ページ管理]固定リンクを空で登録するとシステムエラーとなる問題を修正しました #2184
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要
ページ管理機能において、ページ登録または更新時に固定リンクを空のまま保存しようとするとシステムエラーが発生する問題を修正しました。
変更の目的
固定リンクが未入力の状態で保存操作が行われた際に、予期せぬシステムエラー(
strncmp()
関数の型エラー)が発生するのを防ぎ、アプリケーションの安定性を向上させます。変更内容
app/Plugins/Manage/PageManage/PageManage.php
の 224 行目付近にあるstrncmp()
関数において、第一引数$request->permanent_link
がnull
となる可能性がありました。null
合体演算子 (??
) を使用し、$request->permanent_link
がnull
の場合に空文字列 (''
) をstrncmp()
関数に渡すよう修正しました。レビュー完了希望日
関連Pull requests/Issues
#2183
参考
DB変更の有無
無し
チェックリスト