プロジェクト

全般

プロフィール

Backport(バックポート) #4046

完了

Bug(バグ) #3999: MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていない

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

isao sano さんが約8年前に追加. 6年以上前に更新.

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

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


関連するチケット 1 (0件未完了1件完了)

関連している OpenPNE 3 - Backport(バックポート) #3185: メールアドレス設定など、確認欄がある場合のエラー表示が適切ではないFixed(完了)Maki Takahashi2011-10-14

操作

isao sano さんが約8年前に更新

  • 関連している Bug(バグ) #3999: MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていない を追加

isao sano さんが約8年前に更新

https://redmine.openpne.jp/issues/3185
このチケットでの修正とコンフリクトします。 #3185 は master では取り込まれている修正です。

また、本チケット #4046 を手動でコンフリクト解決し、後から #3185 を取り込んだ場合にもコンフリクトします。

以上の理由から #3185 の修正を取り込んだ後、本チケットのプルリクエストを作成します。

isao sano さんが約8年前に更新

isao sano さんが約8年前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.22 に変更

kaoru n さんが約8年前に更新

  • 対象バージョンOpenPNE 3.8.22 から OpenPNE 3.8.x に変更

Youichi Kimura さんがほぼ8年前に更新

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

下記 Pull Request にて修正しました (#4044, #4046 の修正を含んでいます)
https://github.com/openpne/OpenPNE3/pull/442

kaoru n さんが約7年前に更新

  • 親チケット#3999 にセット

Rimpei Ogawa さんが6年以上前に更新

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

isao sano さんが6年以上前に更新

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

試験完了しました。
問題ありません。

isao sano さんが6年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.28 に変更

kaoru n さんが6年以上前に更新

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

マージしました

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