Backport(バックポート) #4045
Bug(バグ) #3999: MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていない
MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていない
100%
Description
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
から削除する。
Related issues
History
#1
Updated by isao sano about 6 years ago
- Related to Bug(バグ) #3999: MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていない added
#2
Updated by isao sano about 6 years ago
https://redmine.openpne.jp/issues/2571
このチケットでの修正とコンフリクトします。 #2571 は master では取り込まれている修正です。
また、本チケット #4045 を手動でコンフリクト解決し、後から #2571 を取り込んだ場合にもコンフリクトします。
以上の理由から #2571 の修正を取り込んだ後、本チケットのプルリクエストを作成します。
#3
Updated by isao sano about 6 years ago
- Related to Backport(バックポート) #2571: メールアドレス設定など、確認欄がある場合のエラー表示が適切ではない added
#4
Updated by isao sano about 6 years ago
- Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.25
#5
Updated by kaoru n about 6 years ago
- Target version changed from OpenPNE 3.6.25 to OpenPNE 3.6.x
#6
Updated by Youichi Kimura almost 6 years ago
- Status changed from New(新規) to Pending Review(レビュー待ち)
- Assignee changed from isao sano to Youichi Kimura
- % Done changed from 0 to 50
下記 Pull Request にて修正しました (#4043, #4045 の修正を含んでいます)
https://github.com/openpne/OpenPNE3/pull/441
#8
Updated by kaoru n about 5 years ago
- Parent task set to #3999
#9
Updated by Rimpei Ogawa over 4 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- % Done changed from 50 to 70
#10
Updated by isao sano over 4 years ago
- Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
- % Done changed from 70 to 80
試験完了しました。
問題ありません。
#11
Updated by isao sano over 4 years ago
- Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.31
#12
Updated by kaoru n over 4 years ago
- Status changed from Pending Merge(マージ待ち) to Fixed(完了)
- % Done changed from 80 to 100
マージしました