Changeset 23369

Timestamp:

2014/04/17 11:03:18 (10 years ago)

Author:

Seasoft

Message:

#2528 (SC_Helper_Purchase#sendOrderMail 端末種別の認識で受注情報を参照していない)
#2448 (typo修正・ソース整形・ソースコメントの改善 for 2.13.2)

File:

• branches/version-2_13-dev/data/class/helper/SC_Helper_Purchase.php (modified) (1 diff)

branches/version-2_13-dev/data/class/helper/SC_Helper_Purchase.php

@@ -1058 +1058 @@
 
     /**
-     * 受注完了メールを送信する.
-     *
-     * HTTP_USER_AGENT の種別により, 携帯電話の場合は携帯用の文面,
-     * PC の場合は PC 用の文面でメールを送信する.
-     *
-     * @param  integer $orderId 受注ID
+     * 注文受付メールを送信する.
+     *
+     * 端末種別IDにより, 携帯電話の場合は携帯用の文面,
+     * それ以外の場合は PC 用の文面でメールを送信する.
+     *
+     * @param integer $order_id 受注ID
      * @param  object  $objPage LC_Page インスタンス
-     * @return void
-     */
-    public function sendOrderMail($orderId, &$objPage = NULL)
-    {
-        $mailHelper = new SC_Helper_Mail_Ex();
+     * @return boolean 送信に成功したか。現状では、正確には取得できない。
+     */
+    public static function sendOrderMail($order_id, &$objPage = NULL)
+    {
+        $objMail = new SC_Helper_Mail_Ex();
 
         // setPageは、プラグインの処理に必要(see #1798)
         if (is_object($objPage)) {
-            $mailHelper->setPage($objPage);
-        }
-
-        $template_id =
-            SC_Display_Ex::detectDevice() == DEVICE_TYPE_MOBILE ? 2 : 1;
-        $mailHelper->sfSendOrderMail($orderId, $template_id);
+            $objMail->setPage($objPage);
+        }
+
+        $arrOrder = SC_Helper_Purchase::getOrder($order_id);
+        if (empty($arrOrder)) {
+            return false; // 失敗
+        }
+        $template_id = $arrOrder['device_type_id'] == DEVICE_TYPE_MOBILE ? 2 : 1;
+        $objMail->sfSendOrderMail($order_id, $template_id);
+
+        return true; // 成功
     }