Changeset 20216

Timestamp:

2011/02/20 01:23:40 (13 years ago)

Author:

miningbrownie

Message:

リファクタリング頑張った

File:

• branches/version-2_5-dev/data/class/pages/admin/order/LC_Page_Admin_Order_Mail.php (modified) (7 diffs)

branches/version-2_5-dev/data/class/pages/admin/order/LC_Page_Admin_Order_Mail.php

@@ -74 +74 @@
         // 検索パラメータの引き継ぎ
         $this->arrSearchHidden = SC_Utils_Ex::sfFilterKey($_POST,"^search_");
-        $this->tpl_order_id = $_POST['order_id'];
         // パラメータ管理クラス
         $objFormParam = new SC_FormParam();
         // パラメータ情報の初期化
         $this->lfInitParam($objFormParam);
+        // POST値の取得
+        $objFormParam->setParam($_POST);
+        $this->tpl_order_id = $objFormParam->getValue('order_id');
 
         switch($this->getMode()) {
@@ -84 +86 @@
                 break;
             case 'return':
-                // POST値の取得
-                $objFormParam->setParam($_POST);
                 break;
             case 'send':
-                $this->send($objFormParam);
-                exit;
+                // 入力値の変換 TODO ここ気持ち悪いんだ returnの時にやってなかったからもって上でやっていいものなのかどうか。
+                $objFormParam->convParam();
+                $sendStatus = $this->doSend($objFormParam,
+                $objFormParam->getValue('order_id'),
+                $objFormParam->getValue('template_id'),
+                $objFormParam->getValue('subject'),
+                $objFormParam->getValue('header'),
+                $objFormParam->getValue('footer'));
+                if($sendStatus){
+                    SC_Response_Ex::sendRedirect(ADMIN_ORDER_URLPATH);
+                    exit;
+                }
             case 'confirm':
-                $this->confirm($objFormParam);
+                // 入力値の変換 ここ気持ち悪いんだ
+                $objFormParam->convParam();
+                $status = $this->confirm($objFormParam,
+                $objFormParam->getValue('order_id'),
+                $objFormParam->getValue('template_id'),
+                $objFormParam->getValue('subject'),
+                $objFormParam->getValue('header'),
+                $objFormParam->getValue('footer'));
+                if($status){
+                    return ;
+                }
                 break;
             case 'change':
-                // POST値の取得
                 $this->changeData($objFormParam);
                 break;
         }
 
-        if(SC_Utils_Ex::sfIsInt($_POST['order_id'])) {
-            $this->arrMailHistory = $this->getMailHistory($_POST['order_id']);
+        if(SC_Utils_Ex::sfIsInt($objFormParam->getValue('order_id'))) {
+            $this->arrMailHistory = $this->getMailHistory($objFormParam->getValue('order_id'));
         }
 
         $this->arrForm = $objFormParam->getFormParamList();
     }
-    
+
     /**
      * 指定された注文番号のメール履歴を取得する。
@@ -119 +138 @@
 
     /**
-     * 
+     *
      * メールを送る。
      * @param SC_FormParam $objFormParam
      */
-    function send(&$objFormParam){
-        // POST値の取得
-        $objFormParam->setParam($_POST);
+    function doSend(&$objFormParam,$order_id, $template_id, $subject, $header, $footer){
         // 入力値の変換
         $objFormParam->convParam();
+
         $this->arrErr = $objFormParam->checkerror();
         // メールの送信
@@ -133 +151 @@
             // 注文受付メール
             $objMail = new SC_Helper_Mail_Ex();
-            $objMail->sfSendOrderMail($_POST['order_id'], $_POST['template_id'], $_POST['subject'], $_POST['header'], $_POST['footer']);
-        }
-        SC_Response_Ex::sendRedirect(ADMIN_ORDER_URLPATH);
-    }
-
-    function confirm(&$objFormParam){
-        // POST値の取得
-        $objFormParam->setParam($_POST);
-        // 入力値の変換
-        $objFormParam->convParam();
+            $objSendMail = $objMail->sfSendOrderMail($order_id, $template_id, $subject, $header, $footer);
+            // TODO $SC_SendMail から送信がちゃんと出来たか確認できたら素敵。
+            return true;
+        }
+        return false;
+    }
+
+    /**
+     * 確認画面を表示する為の準備
+     * @param SC_FormParam $objFormParam
+     * @param int $order_id
+     * @param int $template_id
+     * @param string $subject
+     * @param string $header
+     * @param string $footer
+     */
+    function confirm(&$objFormParam,$order_id, $template_id, $subject, $header, $footer){
         // 入力値の引き継ぎ
         $this->arrHidden = $objFormParam->getHashArray();
@@ -150 +175 @@
             // 注文受付メール(送信なし)
             $objMail = new SC_Helper_Mail_Ex();
-            $objSendMail = $objMail->sfSendOrderMail($_POST['order_id'], $_POST['template_id'], $_POST['subject'], $_POST['header'], $_POST['footer'], false);
-            // 確認ページの表示
-            $this->tpl_subject = $_POST['subject'];
+            $objSendMail = $objMail->sfSendOrderMail(
+            $order_id,
+            $template_id,
+            $subject,
+            $header,
+            $footer, false);
+            
+            $this->tpl_subject = $objFormParam->getValue('subject');
             $this->tpl_body = mb_convert_encoding( $objSendMail->body, CHAR_CODE, "auto" );
             $this->tpl_to = $objSendMail->tpl_to;
             $this->tpl_mainpage = 'order/mail_confirm.tpl';
-            return;
-        }
-    }
-
+            return true;
+        }
+        return false;
+    }
+
+    /**
+     * 
+     * テンプレートの文言をフォームに入れる。
+     * @param SC_FormParam $objFormParam
+     */
     function changeData(&$objFormParam){
-        $objFormParam->setValue('template_id', $_POST['template_id']);
-        if(SC_Utils_Ex::sfIsInt($_POST['template_id'])) {
+        $objFormParam->setValue('template_id', $objFormParam->getValue('template_id'));
+        if(SC_Utils_Ex::sfIsInt($objFormParam->getValue('template_id'))) {
             $objQuery =& SC_Query::getSingletonInstance();
             $where = "template_id = ?";
-            $arrRet = $objQuery->select("subject, header, footer", "dtb_mailtemplate", $where, array($_POST['template_id']));
-            $objFormParam->setParam($arrRet[0]);
+            $mailTemplates = $objQuery->select("subject, header, footer", "dtb_mailtemplate", $where, array($objFormParam->getValue('template_id')));
+            $objFormParam->setParam($mailTemplates[0]);
         }
     }
@@ -181 +217 @@
 
 
-    /* パラメータ情報の初期化 */
     /**
      * パラメータ情報の初期化
@@ -187 +222 @@
      */
     function lfInitParam(&$objFormParam) {
+        $objFormParam->addParam("オーダーID", "order_id", INT_LEN, "n", array("EXIST_CHECK", "MAX_LENGTH_CHECK", "NUM_CHECK"));
         $objFormParam->addParam("テンプレート", "template_id", INT_LEN, "n", array("EXIST_CHECK", "MAX_LENGTH_CHECK", "NUM_CHECK"));
         $objFormParam->addParam("メールタイトル", "subject", STEXT_LEN, "KVa",  array("EXIST_CHECK", "MAX_LENGTH_CHECK", "SPTAB_CHECK"));