名前[姓]/名前[名]に半角・全角スペースが登録できてしまう件の修正 #4453
Conversation
|
@otsuka-star ユニットテストの調整をお願いします。わからなかったら、連絡ください。 |
|
ユニットテストに使用された 
|
|
@otsuka-star はい、合ってます |
|
WordPressはユーザーデフォルトで性名フィールドがあると認識していますが、これはスペースを禁止にするバリデーションは入っていません。 
|
| ->notEmptyString('real_name_1', __d('baser_core', '名前[姓]を入力してください。')); | ||
| ->notEmptyString('real_name_1', __d('baser_core', '名前[姓]を入力してください。')) | ||
| ->add('real_name_1', [ | ||
| 'noWhitespace' => [ |
There was a problem hiding this comment.
カスタムでバリデーションルールを実装するより、regexで対応できるのではと考えていますがどうでしょうか？
コアでもregexが多用されている印象があり、複雑なロジックの場合だけカスタムでバリデーションルールが実装されている気がします。
3.8系みたいですが以下がまとまっています。
https://qiita.com/azukiazusa/items/9e8bbe461bdbc7ee572c
| 'rule' => function ($value) { | ||
| return !preg_match('/[ ]/u', $value); | ||
| }, | ||
| 'message' => __d('baser_core', '名前[姓]に半角・全角スペースは使用できません。') |
There was a problem hiding this comment.
名前[姓]に半角・全角スペースは使用できません。
上記はすでに多言語化、対応ファイルが追記されているのかと考えましたが、まだのようなので、以下ファイルに多言語化対応などを追記する必要がありそうです。
例：
- plugins/baser-core/resources/locales/baser_core.pot
- plugins/baser-core/resources/locales/en/baser_core.po
なお別prで対応いただく形であれば問題ないと思います。
5 participants
@ryuring
以下の改修を行いましたので、レビューお願いします。
改修内容：