Bug（バグ） #1836
完了
Yuma Sakata さんがほぼ14年前に追加.
約9年前に更新.
説明
管理画面でログイン停止にしてもPCでログイン可能。
バージョン3.4と3.2では再現できませんでした。
※今回、URLは省略させていただきます。
OpenPNEのバージョン 3.6beta7-dev
再現手順¶
- 管理画面でログイン停止状態にして有効にする。
- ログイン停止状態のIDでログインする。
- ログイン停止状態のはずなのに、ログイン可能。
修正内容¶
関連するチケット
2 (0件未完了 — 2件完了)
- 優先度 を Normal（通常） から High（高め） に変更
- 対象バージョン を OpenPNE 3.7.0 にセット
- ステータス を New（新規） から Accepted（着手） に変更
- 担当者 を Masato Nagasawa にセット
- ステータス を Accepted（着手） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
以下のテストを行っています。
| ログイン状態 |
ログイン停止対象 |
ログイン対象 |
期待結果 |
| 未ログイン |
メンバーA |
メンバーA |
ログインできない |
| 未ログイン |
メンバーB |
メンバーB |
ログインできない |
| 未ログイン |
メンバーA |
メンバーB |
ログインできる |
| 未ログイン |
メンバーB |
メンバーA |
ログインできる |
| メンバーA |
メンバーA |
メンバーA |
ログアウトされる |
| メンバーB |
メンバーB |
メンバーB |
ログアウトされる |
| メンバーB |
メンバーA |
メンバーB |
ログイン継続 |
| メンバーA |
メンバーB |
メンバーA |
ログイン継続 |
isActive() が false であった場合に、
setAuthenticated() の値が false になるような変更を行ったので、
影響のありそうな以下の動作テストも行いました。
- 友達招待で登録からログインまでできるか
- 新規登録で登録からログインまでできるか
変更点確認しました。OKです。
私見ですが、opSecurityUser クラスにおける "isSNSMember" という名称の使われ方が曖昧なので、
- isSNSMember() → isValidSNSMember()
- setIsSNSMember(bool) → setSNSMemberCredential(bool)
のように改名したらよいのではないかと思いました。(tozuka)
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
- 3.8 で発生するか を Unknown (未調査) にセット
他の形式にエクスポート: Atom
PDF
