Changeset 20859

Timestamp:

2011/04/18 19:24:28 (10 years ago)

Author:

nanasess

bzr:base-revision:

svn-v4:1e3b908f-19a9-db11-a64c-001125224ba8:branches/version-2_11-dev:20858

bzr:committer:

Kentaro Ohkouchi <ohkouchi@loop-az.jp>

bzr:file-ids:

data/Smarty/templates/admin/design/header.tpl 15732@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2FSmarty%2Ftemplates%2Fdefault%2Fadmin%2Fdesign%2Fheader.tpl
data/class/pages/admin/design/LC_Page_Admin_Design_Bloc.php 15687@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fpages%2Fadmin%2Fdesign%2FLC_Page_Admin_Design_Bloc.php
data/class/pages/admin/design/LC_Page_Admin_Design_Header.php 15688@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fpages%2Fadmin%2Fdesign%2FLC_Page_Admin_Design_Header.php

bzr:mapping-version:

v4

bzr:merge:

ohkouchi@loop-az.jp-20110418102242-i3r248nn3kzq7yyr

bzr:repository-uuid:

1e3b908f-19a9-db11-a64c-001125224ba8

bzr:revision-id:

ohkouchi@loop-az.jp-20110418102425-xm10rcev2j2xr3db

bzr:revno:

3570

bzr:revprop:branch-nick:

branches/version-2_11-dev

bzr:root:

branches/version-2_11-dev

bzr:text-revisions:

data/Smarty/templates/admin/design/header.tpl ohkouchi@loop-az.jp-20110418102242-i3r248nn3kzq7yyr
data/class/pages/admin/design/LC_Page_Admin_Design_Bloc.php ohkouchi@loop-az.jp-20110418102242-i3r248nn3kzq7yyr
data/class/pages/admin/design/LC_Page_Admin_Design_Header.php ohkouchi@loop-az.jp-20110418102242-i3r248nn3kzq7yyr

bzr:timestamp:

2011-04-18 19:24:25.545000076 +0900

bzr:user-agent:

bzr2.2.1+bzr-svn1.0.4

svn:original-date:

2011-04-18T10:24:25.545000Z

Message:

#972 (リファクタリング開発：[管理画面]デザイン管理)

• ヘッダ/フッタ設定

Location:

branches/version-2_11-dev/data

Files:

• Smarty/templates/admin/design/header.tpl (modified) (4 diffs)
• class/pages/admin/design/LC_Page_Admin_Design_Bloc.php (modified) (1 diff)
• class/pages/admin/design/LC_Page_Admin_Design_Header.php (modified) (3 diffs)

branches/version-2_11-dev/data/Smarty/templates/admin/design/header.tpl

@@ -22 +22 @@
  */
 *}-->
-<!--{*
+<div id="design" class="contents-main">
 
-XXX: ヘッダーとフッターでwrapの設定が違うのは疑問。それぞれの良し悪しはともかく、統一を。
+    <!--{if $arrErr.err != ""}-->
+        <div class="message">
+            <span class="attention"><!--{$arrErr.err}--></span>
+        </div>
+    <!--{/if}-->
 
-*}-->
-<div id="design" class="contents-main">
     <!--{* ▼ヘッダー編集ここから *}-->
     <h2>ヘッダー編集</h2>
@@ -35 +37 @@
     <input type="hidden" name="division" value="header" />
     <input type="hidden" name="header_row" value="<!--{$header_row}-->" />
-    <input type="hidden" name="browser_type" value="" />
     <input type="hidden" name="device_type_id" value="<!--{$device_type_id|h}-->" />
 
-        <textarea id="header-area" class="top" name="header" rows="<!--{$header_row}-->" wrap="off" style="width: 100%;"><!--{$header_data|h|smarty:nodefaults}--></textarea>
+        <textarea id="header-area" class="top" name="header" rows="<!--{$header_row}-->" style="width: 100%;"><!--{$header_data|h|smarty:nodefaults}--></textarea>
         <div class="btn">
             <a id="header-area-resize-btn" class="btn-normal" href="javascript:;" onclick="ChangeSize('#header-area-resize-btn', '#header-area', 50, 13); $('input[name=header_row]').val($('#header-area').attr('rows'));return false;"><span>拡大</span></a>
@@ -59 +60 @@
     <input type="hidden" name="division" value="footer" />
     <input type="hidden" name="footer_row" value=<!--{$footer_row}--> />
-    <input type="hidden" name="browser_type" value="" />
     <input type="hidden" name="device_type_id" value="<!--{$device_type_id|h}-->" />
 
@@ -76 +76 @@
     <!--{* ▲フッター編集ここまで *}-->
 </div>
-<script type="text/javascript">
-    /* ブラウザの種類をセットする */
-    function lfnSetBrowser(form, item){
-        browser_type = 0;
-        if(navigator.userAgent.indexOf("MSIE") >= 0){
-            browser_type = 1;
-        }
-        else if(navigator.userAgent.indexOf("Gecko/") >= 0){
-            browser_type = 2;
-        }
-
-        document[form][item].value=browser_type;
-    }
-
-</script>

branches/version-2_11-dev/data/class/pages/admin/design/LC_Page_Admin_Design_Bloc.php

@@ -73 +73 @@
     function action() {
         $objFormParam = new SC_FormParam_Ex();
-        $this->lfInitParam($objFormParam, $_REQUEST);
+        $this->lfInitParam($objFormParam);
         $objFormParam->setParam($_REQUEST);
         $objFormParam->convParam();

branches/version-2_11-dev/data/class/pages/admin/design/LC_Page_Admin_Design_Header.php

@@ -24 +24 @@
 // {{{ requires
 require_once CLASS_EX_REALDIR . 'page_extends/admin/LC_Page_Admin_Ex.php';
+require_once CLASS_EX_REALDIR . 'helper_extends/SC_Helper_FileManager_Ex.php';
 
 /**
@@ -68 +69 @@
      * Page のアクション.
      *
-     * FIXME テンプレートの取得方法を要修正
-     *
      * @return void
      */
     function action() {
-        // 端末種別IDを取得
-        if (isset($_REQUEST['device_type_id'])
-            && is_numeric($_REQUEST['device_type_id'])) {
-            $device_type_id = $_REQUEST['device_type_id'];
+        $objFormParam = new SC_FormParam_Ex();
+        $this->lfInitParam($objFormParam);
+        $objFormParam->setParam($_REQUEST);
+        $objFormParam->convParam();
+        $this->arrErr = $objFormParam->checkError();
+        $is_error = (!SC_Utils_Ex::isBlank($this->arrErr));
+
+        $this->device_type_id = $objFormParam->getValue('device_type_id', DEVICE_TYPE_PC);
+
+        switch ($this->getMode()) {
+        // 登録
+        case 'regist':
+            if ($this->doRegister($objFormParam)) {
+                $this->tpl_onload = "alert('登録が完了しました。');";
+            }
+            break;
+
+        default:
+            break;
+        }
+
+        if (!$is_error) {
+            // テキストエリアに表示
+            $header_path = $this->getTemplatePath($this->device_type_id, 'header');
+            $footer_path = $this->getTemplatePath($this->device_type_id, 'footer');
+            if ($header_path === false || $footer_path === false) {
+                $this->arrErr['err'] = '※ ファイルの取得に失敗しました<br />';
+            } else {
+                $this->header_data = file_get_contents($header_path);
+                $this->footer_data = file_get_contents($footer_path);
+            }
         } else {
-            $device_type_id = DEVICE_TYPE_PC;
+            // 画面にエラー表示しないため, ログ出力
+            GC_Utils_Ex::gfPrintLog('Error: ' . print_r($this->arrErr, true));
         }
-        $this->device_type_id = $device_type_id;
 
         //サブタイトルの追加
-        $this->tpl_subtitle .= ' - ' . $this->arrDeviceType[$device_type_id];
-
-        // テンプレートのパス
-        $template_path = $this->lfGetTemplatePath($device_type_id);
-
-        // データ更新処理
-        if (isset($_POST['division']) && $_POST['division'] != '') {
-            $division = $_POST['division'];
-            $content = $_POST[$division]; // TODO no checked?
-
-            switch ($this->getMode()) {
-            case 'regist':
-                // 正規のテンプレートに書き込む
-                $template = $template_path . '/' . $division . '.tpl';
-                $this->lfUpdateTemplate($template, $content);
-                $this->tpl_onload="alert('登録が完了しました。');";
-                break;
-            default:
-                // なにもしない
-                break;
-            }
-        }
-
-        // テキストエリアに表示
-        $this->header_data = file_get_contents($template_path . '/header.tpl');
-        $this->footer_data = file_get_contents($template_path . '/footer.tpl');
-
-        // ブラウザタイプ
-        $this->browser_type = isset($_POST['browser_type']) ? $_POST['browser_type'] : "";
-    }
-
-    function lfUpdateTemplate($template, $content) {
-        $fp = fopen($template,'w');
-        fwrite($fp, $content);
-        fclose($fp);
-    }
-
-    function lfGetTemplatePath($device_type_id) {
-        $objLayout = new SC_Helper_PageLayout_Ex();
-        return $objLayout->getTemplatePath($device_type_id);
+        $this->tpl_subtitle .= ' - ' . $this->arrDeviceType[$this->device_type_id];
     }
 
@@ -133 +120 @@
         parent::destroy();
     }
+
+    /**
+     * パラメータ情報の初期化
+     *
+     * @param object $objFormParam SC_FormParamインスタンス
+     * @return void
+     */
+    function lfInitParam(&$objFormParam) {
+        $objFormParam->addParam("端末種別ID", "device_type_id", INT_LEN, 'n', array("NUM_CHECK", "MAX_LENGTH_CHECK"));
+        $objFormParam->addParam("division", "division", STEXT_LEN, 'a', array("MAX_LENGTH_CHECK"));
+        $objFormParam->addParam("ヘッダデータ", "header");
+        $objFormParam->addParam("フッタデータ", "footer");
+    }
+
+    /**
+     * エラーチェックを行う.
+     *
+     * @param SC_FormParam $objFormParam SC_FormParam インスタンス
+     * @return array エラーメッセージの配列
+     */
+    function lfCheckError(&$objFormParam, &$arrErr, &$objLayout) {
+        $arrParams = $objFormParam->getHashArray();
+        $objErr = new SC_CheckError_Ex($arrParams);
+        $objErr->arrErr =& $arrErr;
+        $objErr->doFunc(array("division", "division", STEXT_LEN), array("EXIST_CHECK"));
+        return $objErr->arrErr;
+    }
+
+    /**
+     * 登録を実行する.
+     *
+     * ファイルの作成に失敗した場合は, エラーメッセージを出力する.
+     *
+     * @param SC_FormParam $objFormParam SC_FormParam インスタンス
+     * @return integer|boolean 登録が成功した場合 true; 失敗した場合 false
+     */
+    function doRegister(&$objFormParam) {
+        $division = $objFormParam->getValue('division');
+        $contents = $objFormParam->getValue($division);
+        $tpl_path = $this->getTemplatePath($objFormParam->getValue('device_type_id'), $division);
+        if ($tpl_path === false
+            || !SC_Helper_FileManager_Ex::sfWriteFile($tpl_path, $contents)) {
+            $this->arrErr['err'] = '※ ファイルの書き込みに失敗しました<br />';
+            return false;
+        }
+        return true;
+    }
+
+    /**
+     * テンプレートパスを取得する.
+     *
+     * @param integer $device_type_id 端末種別ID
+     * @param string $division "header" or "footer"
+     * @return string|boolean 成功した場合, テンプレートのパス; 失敗した場合 false
+     */
+    function getTemplatePath($device_type_id, $division) {
+        $tpl_path = SC_Helper_PageLayout_Ex::getTemplatePath($device_type_id) . '/' . $division . '.tpl';
+        if (file_exists($tpl_path)) {
+            return $tpl_path;
+        } else {
+            return false;
+        }
+    }
 }
+?>