Skip to content

名前[姓]/名前[名]に半角・全角スペースが登録できてしまう件の修正 #4453

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
otsuka-star wants to merge 4 commits into
base: 5.3.x
Choose a base branch
from
+36 −0
Open

名前[姓]/名前[名]に半角・全角スペースが登録できてしまう件の修正 #4453

Changes from 2 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
f2b72c9
CU_ALL_TEST-16 No.61
otsuka-star Jul 16, 2026
4fbfab9
バリデーションの条件を半角・全角スペース入力時だけに絞る
otsuka-star Jul 17, 2026
51d961f
名前間のスペースは許容させ、名前の先頭と末尾のスペースを取り除く
otsuka-star Jul 23, 2026
71ef10b
UsersTableTest にスペース入力時の失敗ケースを追加する
otsuka-star Jul 31, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 16 additions & 2 deletions plugins/baser-core/src/Model/Table/UsersTable.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -151,11 +151,25 @@ public function validationDefault(Validator $validator): Validator
->scalar('real_name_1')
->maxLength('real_name_1', 50, __d('baser_core', '名前[姓]は50文字以内で入力してください。'))
->requirePresence('real_name_1', 'create', __d('baser_core', '名前[姓]を入力してください。'))
->notEmptyString('real_name_1', __d('baser_core', '名前[姓]を入力してください。'));
->notEmptyString('real_name_1', __d('baser_core', '名前[姓]を入力してください。'))
->add('real_name_1', [
'noWhitespace' => [

@teratai3 teratai3 Jul 21, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

カスタムでバリデーションルールを実装するより、regexで対応できるのではと考えていますがどうでしょうか？
コアでもregexが多用されている印象があり、複雑なロジックの場合だけカスタムでバリデーションルールが実装されている気がします。

3.8系みたいですが以下がまとまっています。
https://qiita.com/azukiazusa/items/9e8bbe461bdbc7ee572c

'rule' => function ($value) {
return !preg_match('/[ 　]/u', $value);
},
'message' => __d('baser_core', '名前[姓]に半角・全角スペースは使用できません。')

@teratai3 teratai3 Jul 21, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

名前[姓]に半角・全角スペースは使用できません。

上記はすでに多言語化、対応ファイルが追記されているのかと考えましたが、まだのようなので、以下ファイルに多言語化対応などを追記する必要がありそうです。

例：

  • plugins/baser-core/resources/locales/baser_core.pot
  • plugins/baser-core/resources/locales/en/baser_core.po

なお別prで対応いただく形であれば問題ないと思います。

]]);
$validator
->scalar('real_name_2')
->maxLength('real_name_2', 50, __d('baser_core', '名前[名]は50文字以内で入力してください。'))
->allowEmptyString('real_name_2');
->allowEmptyString('real_name_2')
->add('real_name_2', [
'noWhitespace' => [
'rule' => function ($value) {
return !preg_match('/[ 　]/u', $value);
},
'message' => __d('baser_core', '名前[名]に半角・全角スペースは使用できません。')
]]);
$validator
->scalar('nickname')
->maxLength('nickname', 255, __d('baser_core', 'ニックネームは255文字以内で入力してください。'))
Expand Down
Loading