Project

General

Profile

Bug(バグ) #3999

MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていない

Added by Youichi Kimura almost 3 years ago. Updated 10 months ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2016-11-15
Due date:
% Done:

100%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)

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 から削除する。

スクリーンショット_2016-08-31_15.31.43.png View (61.1 KB) Youichi Kimura, 2016-08-31 15:36

スクリーンショット_2016-08-31_15.35.48.png View (116 KB) Youichi Kimura, 2016-08-31 15:36


Subtasks

Backport(バックポート) #4045: MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていないFixed(完了)Youichi Kimura

Backport(バックポート) #4046: MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていないFixed(完了)Youichi Kimura


Related issues

Related to OpenPNE 3 - Bug(バグ) #3900: 携帯版の携帯メールアドレス設定エラーが全角カタカナで表示される Invalid(無効) 2016-03-09

History

#1 Updated by Youichi Kimura almost 3 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • Assignee set to Youichi Kimura
  • % Done changed from 0 to 50

下記 Pull Request にて修正しました (#3999, #4000 の修正を含んでいます)

https://github.com/openpne/OpenPNE3/pull/367

#2 Updated by kaoru n almost 3 years ago

  • Related to Bug(バグ) #3900: 携帯版の携帯メールアドレス設定エラーが全角カタカナで表示される added

#3 Updated by isao sano over 2 years ago

#4 Updated by isao sano over 2 years ago

#5 Updated by kaoru n over 2 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Target version changed from OpenPNE 3.9.0-old to OpenPNE 3.9.0

対象バージョン変更により修正内容の確認が必要であるため差し戻します。

#7 Updated by isao sano over 1 year ago

  • Status changed from Rejected(差し戻し) to Accepted(着手)
  • % Done changed from 50 to 0

#8 Updated by isao sano over 1 year ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/498 にてプルリクエストを行いました。(#3999, #4000 の修正を含んでいます)

#9 Updated by Rimpei Ogawa about 1 year ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#10 Updated by kaoru n about 1 year ago

  • Target version changed from OpenPNE 3.9.0 to OpenPNE-3.9.2

#11 Updated by isao sano about 1 year ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

試験行いました。
問題ありません。

#12 Updated by kaoru n about 1 year ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF