Changeset 20467 for branches/version-2_5-dev/data/class/pages/shopping/LC_Page_Shopping_LoadPaymentModule.php

Timestamp:

2011/03/03 12:02:18 (13 years ago)

Author:

nanasess

bzr:base-revision:

ohkouchi@loop-az.jp-20110302111225-7nt684mmays9jmmw

bzr:committer:

Kentaro Ohkouchi <ohkouchi@loop-az.jp>

bzr:file-ids:

data/class/pages/shopping/LC_Page_Shopping_Confirm.php 15223@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fpages%2Fshopping%2FLC_Page_Shopping_Confirm.php
data/class/pages/shopping/LC_Page_Shopping_LoadPaymentModule.php lc_page_shopping_loa-20101025071457-rpmc2g2y1ik9lyow-1
data/class/pages/upgrade/helper/LC_Upgrade_Helper_Json.php 16839@1e3b908f-19a9-db11-a64c-001125224ba8:branches%2Ffeature-module-update%2Fdata%2Fclass%2Fpages%2Fupgrade%2Fhelper%2FLC_Upgrade_Helper_Json.php
test/class/page/shopping/LC_Page_Shopping_LoadPaymentModule_Test.php lc_page_shopping_loa-20101025071501-16934ygv2y12gz92-1

bzr:mapping-version:

v4

bzr:merge:

ohkouchi@loop-az.jp-20110303030004-43u2fhzy6y0i15nf

bzr:repository-uuid:

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

bzr:revision-id:

ohkouchi@loop-az.jp-20110303030214-5ztlfgyzhqa3cncl

bzr:revno:

3194

bzr:revprop:branch-nick:

branches/version-2_5-dev

bzr:root:

branches/version-2_5-dev

bzr:text-revisions:

data/class/pages/shopping/LC_Page_Shopping_Confirm.php ohkouchi@loop-az.jp-20110303030004-43u2fhzy6y0i15nf
data/class/pages/shopping/LC_Page_Shopping_LoadPaymentModule.php ohkouchi@loop-az.jp-20110303030004-43u2fhzy6y0i15nf
test/class/page/shopping/LC_Page_Shopping_LoadPaymentModule_Test.php ohkouchi@loop-az.jp-20110303030004-43u2fhzy6y0i15nf

bzr:timestamp:

2011-03-03 12:02:14.236999989 +0900

bzr:user-agent:

bzr2.2.1+bzr-svn1.0.4

svn:original-date:

2011-03-03T03:02:14.237000Z

Message:

#842(決済モジュールから load_payment_module へ戻った場合の挙動の改善)

• 受注フローの変更に伴い, LC_Page_Shopping_LoadPaymentModule.php を修正

File:

• branches/version-2_5-dev/data/class/pages/shopping/LC_Page_Shopping_LoadPaymentModule.php (modified) (5 diffs)

branches/version-2_5-dev/data/class/pages/shopping/LC_Page_Shopping_LoadPaymentModule.php

@@ -28 +28 @@
  * 決済モジュールの呼び出しを行うクラス.
  *
+ * 決済フローの妥当性検証は, トランザクションID等を使用して, 決済モジュール側で
+ * 行う必要がある.
+ *
  * @package Page
  * @author Kentaro Ohkouchi
@@ -52 +55 @@
      */
     function process() {
-        $objSiteSess = new SC_SiteSession_Ex();
-        $objCartSess = new SC_CartSession_Ex();
-        $objPurchase = new SC_Helper_Purchase_Ex();
 
-        if (!$objSiteSess->isPrePage()) {
-            SC_Utils_Ex::sfDispSiteError(PAGE_ERROR, $objSiteSess);
-        }
-
-        $uniqid = $objSiteSess->getUniqId();
-        $objPurchase->verifyChangeCart($uniqid, $objCartSess);
-
-        $payment_id = $this->getPaymentId();
-        if ($payment_id === false) {
+        $order_id = $this->getOrderId();
+        if ($order_id === false) {
             SC_Utils_Ex::sfDispSiteError(PAGE_ERROR, "", true);
             return;
         }
 
-        $module_path = $this->getModulePath($payment_id);
+        $module_path = $this->getModulePath($order_id);
         if ($module_path === false) {
             SC_Utils_Ex::sfDispSiteError(FREE_ERROR_MSG, "", true,
@@ -76 +69 @@
         }
         require_once($module_path);
-    }
-
-    /**
-     * モバイルページを初期化する.
-     *
-     * @return void
-     */
-    function mobileInit() {
-        $this->init();
-    }
-
-    /**
-     * Page のプロセス(モバイル).
-     *
-     * @return void
-     */
-    function mobileProcess() {
-        $this->process();
     }
 
@@ -106 +81 @@
 
     /**
-     * 支払い方法IDをキーにして, 決済モジュールのパスを取得する.
+     * 受注IDをキーにして, 決済モジュールのパスを取得する.
      *
      * 決済モジュールが取得できた場合は, require 可能な決済モジュールのパスを返す.
-     * 支払い方法IDが無効な場合, 取得したパスにファイルが存在しない場合は false
+     * 受注IDが無効な場合, 取得したパスにファイルが存在しない場合は false
      *
-     * @param integer $payment_id 支払い方法ID
+     * @param integer $order_id 受注ID
      * @return string|boolean 成功した場合は決済モジュールのパス;
      *                        失敗した場合 false
      */
-    function getModulePath($payment_id) {
+    function getModulePath($order_id) {
         $objQuery =& SC_Query::getSingletonInstance();
         $sql = <<< __EOS__
             SELECT module_path
-              FROM dtb_payment
-             WHERE payment_id = ?
+              FROM dtb_payment T1
+              JOIN dtb_order T2
+                ON T1.payment_id = T2.payment_id
+             WHERE order_id = ?
 __EOS__;
-        $module_path = $objQuery->getOne($sql, array($payment_id));
+        $module_path = $objQuery->getOne($sql, array($order_id));
         if (file_exists($module_path)) {
             return $module_path;
@@ -130 +107 @@
 
     /**
-     * 支払い方法ID を取得する.
+     * 受注ID を取得する.
      *
-     * 以下の順序で支払い方法IDを取得する.
+     * 以下の順序で受注IDを取得する.
      *
-     * 1. $_SESSION['payment_id']
-     * 2. $_POST['payment_id']
-     * 3. $_GET['payment_id']
+     * 1. $_SESSION['order_id']
+     * 2. $_POST['order_id']
+     * 3. $_GET['order_id']
      *
-     * 支払い方法IDが取得できない場合は false を返す.
+     * 受注IDが取得できない場合は false を返す.
      *
      * @access private
-     * @return integer|boolean 支払い方法の取得に成功した場合は支払い方法IDを返す;
+     * @return integer|boolean 受注IDの取得に成功した場合は受注IDを返す;
      *                         失敗した場合は, false を返す.
      */
-    function getPaymentId() {
-        if (isset($_SESSION['payment_id'])
-            && !SC_Utils_Ex::isBlank($_SESSION['payment_id'])) {
-            return $_SESSION['payment_id'];
+    function getOrderId() {
+        if (isset($_SESSION['order_id'])
+            && !SC_Utils_Ex::isBlank($_SESSION['order_id'])
+            && SC_Utils_Ex::sfIsInt($_SESSION['order_id'])) {
+            return $_SESSION['order_id'];
         }
 
-        if (isset($_POST['payment_id'])
-            && !SC_Utils_Ex::isBlank($_POST['payment_id'])) {
-            return $_POST['payment_id'];
+        if (isset($_POST['order_id'])
+            && !SC_Utils_Ex::isBlank($_POST['order_id'])
+            && SC_Utils_Ex::sfIsInt($_POST['order_id'])) {
+            return $_POST['order_id'];
         }
 
-        if (isset($_GET['payment_id'])
-            && !SC_Utils_Ex::isBlank($_GET['payment_id'])) {
-            return $_GET['payment_id'];
+        if (isset($_GET['order_id'])
+            && !SC_Utils_Ex::isBlank($_GET['order_id'])
+            && SC_Utils_Ex::sfIsInt($_GET['order_id'])) {
+            return $_GET['order_id'];
         }
-
         return false;
     }