Bug(バグ) #2489
メールアドレス設定など、確認欄がある場合のエラー表示が適切ではない
開始日:
2011-10-14
期日:
進捗率:
0%
3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Yes (はい)
説明
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)
関連するチケット
関係しているリビジョン
(fixes #2489) changed MemberConfigForm::setMemberConfigWidget()
confirm fields must be used only for comparison check
IsUnique validation must be performed only valid data
履歴
#1 Maki Takahashi が12年以上前に更新
- 説明 を更新 (diff)
#2 Maki Takahashi が12年以上前に更新
- ステータス を New(新規) から Accepted(着手) に変更
- 担当者 を Maki Takahashi にセット
#3 Maki Takahashi が12年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 7f3f02fc4eec23de48b95722ca365360f003a2a0 で適用されました。
#4 Maki Takahashi が12年以上前に更新
追加で、ユニークチェックを行う設定になっている場合は、
(DBに登録されているのは他のバリデーションを通過していると判断できることから)
他のバリデーションチェックを全て通過できたときだけユニークチェックをかけるように修正しました。
#5 Shouta Kashiwagi が約12年前に更新
- 対象バージョン を OpenPNE 3.7.0 から 252 に変更
#6 Shouta Kashiwagi が約12年前に更新
- 対象バージョン を 252 から OpenPNE 3.8.x に変更
#7 Shinichi Urabe がほぼ10年前に更新
- 対象バージョン を OpenPNE 3.8.x から OpenPNE 3.8.12 に変更
- 3.6 で発生するか を Unknown (未調査) にセット
- 3.8 で発生するか を Unknown (未調査) にセット
#8 誠二 天重 がほぼ10年前に更新
- コピー先 Backport(バックポート) #3638: メールアドレス設定など、確認欄がある場合のエラー表示が適切ではない を追加
#11 Youichi Kimura が7年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
- 3.6 で発生するか を Unknown (未調査) から Yes (はい) に変更
- 3.8 で発生するか を Unknown (未調査) から Yes (はい) に変更
レビューOKです
(レビュー時に動作確認のために作成したテストコードを #4001 にて追加しています)
#12 Youichi Kimura が7年以上前に更新
- 関連している Enhancement(機能追加・改善) #4001: MemberConfigFormに対するテストコードの追加 を追加
#13 Youichi Kimura が7年以上前に更新
- コピー先 を削除 (Backport(バックポート) #3638: メールアドレス設定など、確認欄がある場合のエラー表示が適切ではない)