Backport(バックポート) #4046
完了Bug(バグ) #3999: MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていない
MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていない
100%
説明
Overview (現象)¶
MemberConfigForm と MemberProfileForm には、例えばメールアドレス設定のように SNS 内の他のメンバーと設定値が重複することを禁止するための IsUnique 制約が備わっている。
両者はほぼ同一の機能であるがそれぞれ別のエラーメッセージが使われており、MemberConfigForm では「Invalid %name%.」(携帯メールアドレス設定では ja_JP で「メールアドレスが無効です。」と表示される) に対して MemberProfileForm では「The inputted value is already exist.」(ja_JP で「すでに登録されています。」と表示される) が使用されている。
MemberConfigForm のエラーメッセージ:
MemberProfileForm のエラーメッセージ:
これらのメッセージは他メンバーとの重複という同じ問題に対するエラーメッセージであるため、いずれか一方に揃えるべきである。
また、「Invalid %name%.」に対して「The inputted value is already exist.」の方がより的確にエラーの原因を説明していること、前者のメッセージは OpenPNE 内部で使用する設定名をそのままメッセージに使用しておりユーザーに対して表示するメッセージとしては適切でないことから、統一するとすれば後者のメッセージが望ましいと考える。
Way to fix (修正内容)¶
MemberConfigForm::isUnique()
で使用するエラーメッセージを「The inputted value is already exist.」に変更する。また、これに伴い不要となった翻訳文を i18n/messages.ja.xml
から削除する。
isao sano さんが約8年前に更新
- 関連している Bug(バグ) #3999: MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていない を追加
isao sano さんがほぼ8年前に更新
- 関連している Backport(バックポート) #3185: メールアドレス設定など、確認欄がある場合のエラー表示が適切ではない を追加
Youichi Kimura さんが7年以上前に更新
- ステータス を New(新規) から Pending Review(レビュー待ち) に変更
- 担当者 を isao sano から Youichi Kimura に変更
- 進捗率 を 0 から 50 に変更
下記 Pull Request にて修正しました (#4044, #4046 の修正を含んでいます)
https://github.com/openpne/OpenPNE3/pull/442
Rimpei Ogawa さんが6年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更