プロジェクト

全般

プロフィール

Backport(バックポート) #3185

メールアドレス設定など、確認欄がある場合のエラー表示が適切ではない

Yuma Sakata約5年前に追加. 11ヶ月前に更新.

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

100%


説明

Overview (現象)

  • PCメールアドレス設定
  • 携帯メールアドレス設定
  • パスワード設定

において、本体(例:PCメールアドレス)と確認欄それぞれにバリデーションがかかっているため
両方が空欄だった場合などはそれぞれ「必須項目です。」と表示され(動作として間違ってはいないものの)適切ではないように思われる。

Way to fix (修正内容)

A: メールアドレス
B: メールアドレス(確認)
とした場合、

  • Aにエラーがある → Bの内容に関わらず、Aのフィールドにエラーを表示
  • Aにエラーがなく、AとBが一致しない → Bの内容に関わらず、Bのフィールドにエラーを表示
  • Aにエラーがなく、AとBが一致する → OK

確認バージョン

OpenPNE 3.7.0-dev (master)
OpenPNE 3.6.1 (stable-3.6.x)
OpenPNE 3.4.17 (stable-3.4.x)

fix-unique-memberconfig.patch 表示 (5.48 KB) Yuma Sakata, 2012-08-29 11:17


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #2489: メールアドレス設定など、確認欄がある場合のエラー表示が適切ではない Won't fix(対応せず) 2011-10-14
関連している OpenPNE 3 - Backport(バックポート) #4044: MemberConfigFormのIsUnique制約に対するエラーメッセージが適切に出力されていない Pending Review(レビュー待ち) 2016-11-15
関連している OpenPNE 3 - Backport(バックポート) #4046: MemberConfigFormとMemberProfileFormでIsUnique制約に対するエラーメッセージが統一されていない Pending Review(レビュー待ち) 2016-11-15

関係しているリビジョン

リビジョン 02a6afcb (差分)
isao sano約3年前に追加

(fixes #3185, BP from #2489) changed MemberConfigForm::setMemberConfigWidget()

confirm fields must be used only for comparison check
IsUnique validation must be performed only valid data

リビジョン 6bb5e813
kaoru nishizoe11ヶ月前に追加

Merge pull request #214 from isaosano/t-3185

(fixes #3185, BP from #2489) changed MemberConfigForm::setMemberConfigWidget()

履歴

#1 Yuma Sakata約5年前に更新

修正パッチ添付します。

#2 isao sano約3年前に更新

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

#3 isao sano約3年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#4 isao sano約3年前に更新

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

https://github.com/openpne/OpenPNE3/pull/214
にてプルリクエストを行いました

#5 isao sanoほぼ3年前に更新

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

#6 kaoru nishizoe約2年前に更新

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

#7 kaoru nishizoeほぼ2年前に更新

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

#8 kaoru nishizoeほぼ2年前に更新

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

#9 kaoru nishizoe1年以上前に更新

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

#10 kaoru nishizoe約1年前に更新

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

#11 kaoru nishizoe約1年前に更新

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

#12 Youichi Kimura約1年前に更新

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

レビューOKです

#13 isao sano11ヶ月前に更新

#14 isao sano11ヶ月前に更新

#15 kaoru nishizoe11ヶ月前に更新

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

#16 Chiharu Nakajima11ヶ月前に更新

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

テストOKです

#17 kaoru nishizoe11ヶ月前に更新

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

マージしました

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