Changeset 17605 for branches/comu-ver2/data/class/pages/admin/design/LC_Page_Admin_Design.php

Timestamp:

2008/09/10 20:50:26 (16 years ago)

Author:

Seasoft

Message:

・/index.php にリンクやリダイレクトしている箇所の「index.php」を定数化。

• 定数「DIR_INDEX_FILE」… DirectoryIndex? の実ファイル名。現状の EC-CUBE は、「index.php」。/html/define.php で定義するが、定義が無い場合 SC_Initial::defineDirectoryIndex() で「index.php」と定義。本来は、プログラム中で実ファイルとしての index.php を指すときとは、この定数を使うのが良い気がするが、今回はそこまでは書き換えていない。
• 定数「USE_FILENAME_DIR_INDEX」… DIR_INDEX_FILE にアクセスするときにファイル名を使用するか。/html/define.php で定義。
• 定数「DIR_INDEX_URL」… /index.php にリンクやリダイレクトしている箇所の「index.php」部分を示す。この定数は SC_Initial::defineDirectoryIndex() で自動生成する。

・$_SERVERPHP_SELF? や自身をあらわすファイル名を利用している箇所を文字列「?」に書き換え。これにより、/ が自己URLにリンクするときに /index.php となることを防ぐ。RFC3986 を参考にする。モバイル端末がこのRFCに準拠しているのか確信が無いので、モバイルではこの変更は見送った。
・従来 / にリンクしていた箇所に、定数「DIR_INDEX_URL」を付加。漏れがあると予測される。

File:

• branches/comu-ver2/data/class/pages/admin/design/LC_Page_Admin_Design.php (modified) (3 diffs)

branches/comu-ver2/data/class/pages/admin/design/LC_Page_Admin_Design.php

@@ -192 +192 @@
             if ($_POST['mode'] == 'preview') {
                 if ($page_id === "") {
-                    $this->sendRedirect($this->getLocation("./index.php"));
+                    $this->sendRedirect($this->getLocation(DIR_INDEX_URL));
                 }
                 $this->lfSetPreData($arrPageData, $objLayout);
@@ -198 +198 @@
                 $_SESSION['preview'] = "ON";
 
-                $this->sendRedirect($this->getLocation(URL_DIR . "preview/index.php", array("filename" => $arrPageData[0]["filename"])));
+                $this->sendRedirect($this->getLocation(URL_DIR . "preview/" . DIR_INDEX_URL, array("filename" => $arrPageData[0]["filename"])));
 
             }else{
-                $this->sendRedirect($this->getLocation("./index.php",
+                $this->sendRedirect($this->getLocation(DIR_INDEX_URL,
                                             array("page_id" => $page_id,
                                                   "msg" => "on")));
@@ -211 +211 @@
         if ($_POST['mode'] == 'delete' and  !$objLayout->lfCheckBaseData($page_id)) {
             $objLayout->lfDelPageData($page_id);
-            $this->sendRedirect($this->getLocation("./index.php"));
+            $this->sendRedirect($this->getLocation(DIR_INDEX_URL));
         }