Changeset 21624 for branches/version-2_12-dev

Timestamp:

2012/03/11 15:22:55 (12 years ago)

Author:

Seasoft

Message:

#1694 (SC_SessionFactory#getInstance モバイルの default が不自然な動作)
#1695 (同一ホストに複数の EC-CUBE や他の PHP アプリケーションを導入した場合、セッションキーを共有する)
#1613 (typo修正・ソース整形・ソースコメントの改善)

Location:

branches/version-2_12-dev/data/class

Files:

• SC_Cookie.php (modified) (2 diffs)
• SC_SessionFactory.php (modified) (1 diff)
• sessionfactory/SC_SessionFactory_UseCookie.php (modified) (1 diff)

branches/version-2_12-dev/data/class/SC_Cookie.php

@@ -22 +22 @@
  */
 
-/* 日時表示用クラス */
+/**
+ * クッキー用クラス
+ *
+ */
 class SC_Cookie {
 
@@ -35 +38 @@
     // クッキー書き込み
     function setCookie($key, $val) {
-        setcookie($key, $val, $this->expire, '/', DOMAIN_NAME);
+        setcookie($key, $val, $this->expire, ROOT_URLPATH, DOMAIN_NAME);
     }
 
-    // クッキー取得
+    /**
+     * クッキー取得
+     *
+     * EC-CUBE をURLパスルート以外にインストールしている場合、上位ディレクトリの値も(劣後ではあるが)取得する点に留意。
+     */
     function getCookie($key) {
         return isset($_COOKIE[$key]) ? $_COOKIE[$key] : null;

branches/version-2_12-dev/data/class/SC_SessionFactory.php

@@ -69 +69 @@
             // クッキーを使用する
             case 'useCookie':
+            default:
                 // モバイルの場合はSC_SessionFactory_UseRequestを使用する
                 if (SC_Display_Ex::detectDevice() == DEVICE_TYPE_MOBILE) {
                     $session = new SC_SessionFactory_UseRequest;
                     $session->setState('mobile');
-                    break;
+                } else {
+                    $session = new SC_SessionFactory_UseCookie;
                 }
-            default:
-                $session = new SC_SessionFactory_UseCookie;
                 break;
         }

branches/version-2_12-dev/data/class/sessionfactory/SC_SessionFactory_UseCookie.php

@@ -41 +41 @@
      * セッションパラメーターの指定
      * ・ブラウザを閉じるまで有効
-     * ・すべてのパスで有効
-     *   FIXME 多分、同一ホスト名に複数の EC-CUBE をインストールした場合に望ましくない状態である。特段の事由がなければ、アプリケーションルートを指定すべきだし、あればコメントに残すべき。
+     * ・EC-CUBE ルート配下で有効
      * ・同じドメイン間で共有
+     * FIXME セッションキーのキーが PHP デフォルトのため、上位ディレクトリーで定義があると、その値で動作すると考えられる。
      **/
     function initSession() {
         ini_set('session.cache_limiter', 'none');
         if (session_id() === '') {
-            session_set_cookie_params(0, '/', DOMAIN_NAME);
+            session_set_cookie_params(0, ROOT_URLPATH, DOMAIN_NAME);
             if (!ini_get('session.auto_start')) {
                 // セッション開始