プロジェクト

全般

プロフィール

Bug(バグ) #2489

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

Maki Takahashi12年以上前に追加. 約7年前に更新.

ステータス:
Won't fix(対応せず)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
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)


関連するチケット

関連している OpenPNE 3 - Backport(バックポート) #2571: メールアドレス設定など、確認欄がある場合のエラー表示が適切ではない Fixed(完了) 2011-10-14
関連している OpenPNE 3 - Backport(バックポート) #2572: メールアドレス設定など、確認欄がある場合のエラー表示が適切ではない Won't fix(対応せず) 2011-10-14
関連している OpenPNE 3 - Backport(バックポート) #3185: メールアドレス設定など、確認欄がある場合のエラー表示が適切ではない Fixed(完了) 2011-10-14
関連している OpenPNE 3 - Enhancement(機能追加・改善) #4001: MemberConfigFormに対するテストコードの追加 Rejected(差し戻し) 2016-09-01

関係しているリビジョン

リビジョン 7f3f02fc (差分)
Maki Takahashi12年以上前に追加

(fixes #2489) changed MemberConfigForm::setMemberConfigWidget()

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

履歴

#1 Maki Takahashi12年以上前に更新

  • 説明 を更新 (diff)

#2 Maki Takahashi12年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Maki Takahashi にセット

#3 Maki Takahashi12年以上前に更新

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

更新履歴 7f3f02fc4eec23de48b95722ca365360f003a2a0 で適用されました。

#4 Maki Takahashi12年以上前に更新

追加で、ユニークチェックを行う設定になっている場合は、
(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年前に更新

#9 誠二 天重ほぼ10年前に更新

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

#10 誠二 天重ほぼ10年前に更新

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

#11 Youichi Kimura7年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更
  • 3.6 で発生するかUnknown (未調査) から Yes (はい) に変更
  • 3.8 で発生するかUnknown (未調査) から Yes (はい) に変更

レビューOKです
(レビュー時に動作確認のために作成したテストコードを #4001 にて追加しています)

#12 Youichi Kimura7年以上前に更新

#13 Youichi Kimura7年以上前に更新

#15 isao sano約7年前に更新

  • ステータスPending Testing(テスト待ち) から Won't fix(対応せず) に変更
  • 進捗率70 から 0 に変更

OpenPNE 3.8.22 にて対応済みであったため、対応せずとします。

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