TODO #18276

ログイン中にログイン画面にアクセスできる

村上 彬約1ヶ月前に追加. 29日前に更新.

ステータス:担当開始日:
優先度:通常期日:
担当者:村上 彬進捗率:

0%

カテゴリ:-
対象バージョン:4.1.0

説明

フォーラムに寄せられた問い合わせとなります。
http://forum.basercms.net/modules/newbb/viewtopic.php?topic_id=2957&forum=8

バグではないかと思いますが管理画面ログイン中にログイン画面にアクセスできる必要は無いと思いますので記載の修正案の仕様でも良いと思われます。

履歴

#1 江頭 竜二約1ヶ月前に更新

  • 対象バージョン188 から 4.1.0 に変更

過去にコードを意図的に消した経緯があるようですね。
https://github.com/baserproject/basercms/commit/df9ac3b581a5020f03f1fdd0239f902447956303

改修候補のコードを示しますが、複数ログイン状態を再現した上での動作確認が必要そうです。

        if ($this->request->data) {
            $this->BcAuth->login();
            $user = $this->BcAuth->user();
            $userModel = $this->BcAuth->authenticate['Form']['userModel'];
            if ($user && $this->isAuthorized($user)) {
                if (!empty($this->request->data[$userModel]['saved'])) {
                    if (!$this->request->is('mobile')) {
                        $this->setAuthCookie($this->request->data);
                    } else {
                        $this->BcAuth->saveSerial();
                    }
                    unset($this->request->data[$userModel]['save']);
                } else {
                    $this->Cookie->destroy();
                }
                App::uses('BcBaserHelper', 'View/Helper');
                $BcBaser = new BcBaserHelper(new View());
                $this->setMessage("ようこそ、" . $BcBaser->getUserName($user) . " さん。");
                $this->redirect($this->BcAuth->redirectUrl());
            } else {
                $this->setMessage('アカウント名、パスワードが間違っています。', true);
            }
        } else {
            $user = $this->BcAuth->user();
            if ($user && $this->isAuthorized($user)) {
                $this->redirect($this->BcAuth->redirectUrl());
            }
        }

リリースまで時間がないので次のリリースに見送ります。

#2 村上 彬30日前に更新

ご確認ありがとうございます!

改修候補のコードを示しますが、複数ログイン状態を再現した上での動作確認が必要そうです。

私のほうでも調査を進めたいと思います。

#3 江頭 竜二29日前に更新

マイページを作ってのテストが必要になると思います。
おそらく提供したコードでカバーできているとは思いますが、念のため。

#4 村上 彬29日前に更新

ありがとうございます!
テスト完了しましたので改修してpushしております。
https://github.com/baserproject/basercms/pull/679

宜しくお願い致します。

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