Project

General

Profile

Backport(バックポート) #2571

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

Added by Maki Takahashi almost 8 years ago. Updated almost 3 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2011-10-14
Due date:
% Done:

100%


Description

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)


Related issues

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

Associated revisions

Revision 0a05e5d2 (diff)
Added by isao sano almost 5 years ago

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

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

Revision 79961ba0
Added by kaoru n almost 3 years ago

Merge pull request #206 from isaosano/t-2571

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

History

#1 Updated by Maki Takahashi almost 8 years ago

  • Target version changed from OpenPNE 3.6.1 to OpenPNE 3.6.x

#2 Updated by isao sano almost 5 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.17

#3 Updated by isao sano almost 5 years ago

  • Status changed from New(新規) to Accepted(着手)

#4 Updated by isao sano almost 5 years ago

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

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

#5 Updated by isao sano almost 5 years ago

  • Assignee changed from Maki Takahashi to isao sano

#6 Updated by isao sano almost 5 years ago

  • Target version changed from OpenPNE 3.6.17 to OpenPNE 3.6.x

#7 Updated by kaoru n almost 4 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.21

#8 Updated by kaoru n almost 4 years ago

  • Target version changed from OpenPNE 3.6.21 to OpenPNE 3.6.x

#9 Updated by kaoru n almost 4 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.22

#10 Updated by kaoru n over 3 years ago

  • Target version changed from OpenPNE 3.6.22 to OpenPNE 3.6.x

#11 Updated by kaoru n about 3 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.24

#12 Updated by kaoru n about 3 years ago

  • Target version changed from OpenPNE 3.6.24 to OpenPNE 3.6.x

#13 Updated by Youichi Kimura about 3 years ago

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

レビューOKです

#14 Updated by isao sano almost 3 years ago

#15 Updated by isao sano almost 3 years ago

#16 Updated by kaoru n almost 3 years ago

  • Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.25

#17 Updated by Chiharu Nakajima almost 3 years ago

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

テストOKです

#18 Updated by kaoru n almost 3 years ago

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

マージしました。

Also available in: Atom PDF