プロジェクト

全般

プロフィール

Bug(バグ) #3999

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

Youichi Kimura7年以上前に追加. 5年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2016-11-15
期日:
進捗率:

100%

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

説明

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 表示 (61.1 KB) Youichi Kimura, 2016-08-31 15:36

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


子チケット

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

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


関連するチケット

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

関係しているリビジョン

リビジョン f531aaf0 (差分)
isao sano約6年前に追加

unifying the message text for IsUnique error used by MemberConfigForm and MemberProfileForm (fixes #3999)

リビジョン 789c4831
kaoru n5年以上前に追加

Merge pull request #498 from isaosano/t-4000-3999

MemberConfigForm::isUnique() のエラーメッセージの翻訳に関する複数の修正 (fixes #3999, #4000)

履歴

#1 Youichi Kimura7年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 担当者Youichi Kimura にセット
  • 進捗率0 から 50 に変更

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

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

#2 kaoru n7年以上前に更新

  • 関連している Bug(バグ) #3900: 携帯版の携帯メールアドレス設定エラーが全角カタカナで表示される を追加

#3 isao sano7年以上前に更新

#4 isao sano7年以上前に更新

#5 kaoru nほぼ7年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更
  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.9.0 に変更

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

#7 isao sano約6年前に更新

  • ステータスRejected(差し戻し) から Accepted(着手) に変更
  • 進捗率50 から 0 に変更

#8 isao sano約6年前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

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

#9 Rimpei Ogawaほぼ6年前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#10 kaoru nほぼ6年前に更新

  • 対象バージョンOpenPNE 3.9.0 から OpenPNE-3.9.2 に変更

#11 isao sano5年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

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

#12 kaoru n5年以上前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

マージしました

他の形式にエクスポート: Atom PDF