ここの情報は古いです。ご理解頂いた上でお取り扱いください。

Opened 12 years ago

Closed 11 years ago

Last modified 10 years ago

#986 closed defect (fixed)

招待メール送信で、不正なメールアドレスを入力したときにエラーメッセージが表示されない

Reported by: kiwa Owned by: shingo
Priority: trivial Milestone: OpenPNE2.12.6
Component: pne-admin Version: 2.10.x & 2.12.x & 2.14.x
Keywords: OpenPNE2.10.12 OpenPNE2.13.4 Cc:

Description (last modified by kudo)

■現象

不正メールアドレス(2バイト文字を含むなど)入力時に、エラー文言が表示されない。

■原因

コーディングミス。メールアドレスとして正しくない場合、エラー文言なしでエラー画面を表示するようになっていた。

■修正内容

上記を修正。これにより、入力チェックでエラー画面に遷移したさいにエラー文言が出力されるようになった。

■関連情報

元のDescription

招待メール送信(send_invites)で、不正なメールアドレスを入力して確認画面(do_send_invites)に遷移した際、エラーメッセージが表示されず、「修正する」ボタンだけが表示されます。

確認された状態
  • 日本語の文字列が含まれている
  • 空白文字が含まれている
  • メールアドレスではない半角英数
  • 招待メールドメイン制限で制限されているメールアドレス

既に登録されているメールアドレスを記入した場合のみ

以下のアドレスは登録済みのため送信されません。
    ***@****.**.**(メールアドレスが記載されている)

と表示されます。

Attachments (1)

20081106_do_send_invites.png (7.1 KB) - added by kiwa 11 years ago.
バグ報告画像

Download all attachments as: .zip

Change History (23)

comment:1 Changed 12 years ago by ogawa

Priority: minortrivial

comment:2 Changed 11 years ago by ebihara

Version: 2.6.x & 2.8.x & 2.10.x2.10.x & 2.12.x & 2.13.x

comment:3 Changed 11 years ago by ebihara

Component: owner【オーナー向け機能】pne-admin
Keywords: OpenPNE2.10.12 OpenPNE2.13.4 added
Milestone: OpenPNE2.12.6

comment:4 Changed 11 years ago by mori

Owner: changed from nobody to mori
Status: newassigned

担当します。

comment:5 Changed 11 years ago by mori

Keywords: 確認中 added

以下のリビジョンを修正しました。

comment:6 Changed 11 years ago by mori

Description: modified (diff)

comment:7 Changed 11 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed
  • エラーメッセージを、登録済みメールアドレスを入力した際に表示される、「以下のメールアドレスは登録済みのため送信されません。」という文章にあわせてください。
  • 招待メールドメイン制限についても考慮する必要があるのではないでしょうか。

comment:8 Changed 11 years ago by mori

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンを修正しました。

comment:9 Changed 11 years ago by ebihara

Keywords: テスト待ち added; 確認中 removed

comment:10 Changed 11 years ago by kiwa

Keywords: 差し戻し added; テスト待ち removed

Descriptionについての指摘です。下記の修正をお願いします。

  1. 元のDescription(「■現象」の上にある内容)は、■関連情報に「元のDescription」とわかるように記載してください
  2. ■修正内容について、ユーザ側から見てどのような対応になったのかも記載してほしいです
    • 例: #2910 「絞込みをすると1ページ目に移動するように修正。」

※ 今回#2910を例に挙げてますが、条件を満たしたものを適当に選んだだけなので「これが最良例」というわけではありません

comment:11 Changed 11 years ago by mori

Description: modified (diff)
Keywords: テスト待ち added; 差し戻し removed

comment:12 Changed 11 years ago by mori

Description: modified (diff)

Changed 11 years ago by kiwa

バグ報告画像

comment:13 Changed 11 years ago by kiwa

Keywords: 差し戻し added; テスト待ち removed

以下の現象の確認をお願いします

version

  • 2.10.11(r9001時点)
  • 2.12.5(r9001時点)
  • 2.13.3(r9001時点)

場所

  • 招待メール送信確認画面(admin_do_send_invites)

内容

招待メールアドレスを以下のような不正な値に変更して「送信する」を押下した場合、招待メール送信完了時と同じメッセージが表示されてトップ画面に遷移する。

動作テストで確認した不正な値

  • 重複するメールアドレス
  • メールアドレス以外の半角英数
  • 日本語
  • 招待メールドメイン制限を行っている場合、制限から外れたメールアドレスドメイン制限

※実際に招待メールは送信されていませんが、動作として不自然かと思います


バグ報告画像

また画像のように送信できるメールアドレスと送信できないメールアドレスが混在している場合、形式が正しくない場合・招待メールドメインに含まれていない場合のエラーメッセージが「このメールアドレスには招待メールが送信されない」ということを示せていません。

動作には問題ありませんが、分かりづらいので「このメールアドレスには送信されない」とわかるようなメッセージにしてほしいです。

comment:14 Changed 11 years ago by shingo

Owner: changed from mori to shingo
Status: assignednew

修正します。

comment:15 Changed 11 years ago by shingo

以下のリビジョンで comment:13 にあげられた事項について修正しました。 ご確認ください。

comment:16 Changed 11 years ago by shingo

Keywords: 確認中 added; 差し戻し removed
Owner: changed from shingo to mori

comment:17 Changed 11 years ago by ebihara

Keywords: 差し戻し added; 確認中 removed

この入力値チェックとエラー遷移の記述は一ヶ所にまとめられると思います。

comment:18 Changed 11 years ago by shingo

Owner: changed from mori to shingo

comment:19 Changed 11 years ago by shingo

Keywords: 確認中 added; 差し戻し removed

以下のリビジョンで comment:17 にあげられた事項について修正しました。 ご確認ください。

comment:20 Changed 11 years ago by ebihara

Keywords: テスト待ち added; 確認中 removed

comment:21 Changed 11 years ago by kiwa

Keywords: テスト待ち removed
Resolution: fixed
Status: newclosed

確認しました。問題ありません。

comment:22 Changed 10 years ago by kudo

Description: modified (diff)
Note: See TracTickets for help on using tickets.