OpenPNE 3

修正方針¶

setRememberLoginCookie()はログインしている状態で意味があるので，認証しているかどうかを先に確認する方がいいと思われる．
コードを見てみると認証しているかどうかを確認する部分が該当箇所の直後である256行目に存在するため，この中で処理を行うようにすることが適切であると考えられる．

lib/user/opSecurityUser.class.php

249     $this->initializeUserStatus();
250 
251     if ($this->getAuthAdapter()->getAuthForm()->getValue('is_remember_me'))
252     {
253       $this->setRememberLoginCookie();
254     }
255 
256     if ($this->isAuthenticated())
257     {
258       $this->setCurrentAuthMode($this->getAuthAdapter()->getAuthModeName());
259       $uri = $this->getAuthAdapter()->getAuthForm()->getValue('next_uri');
260 

実装案¶

diff --git a/lib/user/opSecurityUser.class.php b/lib/user/opSecurityUser.class.php
index 7d66a46..2dd1714 100644
--- a/lib/user/opSecurityUser.class.php
+++ b/lib/user/opSecurityUser.class.php
@@ -248,13 +248,13 @@ class opSecurityUser extends opAdaptableUser

     $this->initializeUserStatus();

-    if ($this->getAuthAdapter()->getAuthForm()->getValue('is_remember_me'))
-    {
-      $this->setRememberLoginCookie();
-    }
-
     if ($this->isAuthenticated())
     {
+      if ($this->getAuthAdapter()->getAuthForm()->getValue('is_remember_me'))
+      {
+        $this->setRememberLoginCookie();
+      }
+
       $this->setCurrentAuthMode($this->getAuthAdapter()->getAuthModeName());
       $uri = $this->getAuthAdapter()->getAuthForm()->getValue('next_uri');

更新履歴 5b212017f7a4c20dd4ea5d7b9a6240559d6b8ee3 で適用されました。

テストOKです。