Changeset 22739 for branches/version-2_13-dev/data/class/helper/SC_Helper_CSV.php

Timestamp:

2013/03/22 11:07:49 (13 years ago)

Author:

Seasoft

Message:

#1710 (SC_Helper_CSV#sfDownloadCsv 第4引数($order)が無効となるケースがある)

• 個別処理の統合

File:

• branches/version-2_13-dev/data/class/helper/SC_Helper_CSV.php (modified) (2 diffs)

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

@@ -75 +75 @@
     function sfDownloadCsv($csv_id, $where = '', $arrVal = array(), $order = '', $is_download = false)
     {
+        $objQuery =& SC_Query_Ex::getSingletonInstance();
+
         // CSV出力タイトル行の作成
         $arrOutput = SC_Utils_Ex::sfSwapArray($this->sfGetCsvOutput($csv_id, 'status = ' . CSV_COLUMN_STATUS_FLG_ENABLE));
@@ -80 +82 @@
         $arrOutputCols = $arrOutput['col'];
 
-        $objQuery =& SC_Query_Ex::getSingletonInstance();
-        $objQuery->setOrder($order);
         $cols = SC_Utils_Ex::sfGetCommaList($arrOutputCols, true);
 
-        // TODO: 固有処理 なんかエレガントな処理にしたい
         // 商品の場合
         if ($csv_id == 1) {
-            $objProduct = new SC_Product_Ex();
-            // このWhereを足さないと無効な規格も出力される。現行仕様と合わせる為追加。
+            // この WHERE 句を足さないと無効な規格も出力される。現行仕様と合わせる為追加。
             $inner_where = 'dtb_products_class.del_flg = 0';
-            $from = $objProduct->prdclsSQL($inner_where);
-            $sql = $objQuery->getSql($cols, $from, $where);
-        }
-        // 商品以外の場合
-        else {
-            // 会員の場合
-            if ($csv_id == 2) {
-                $from = 'dtb_customer';
-            }
-            // 注文の場合
-            else if ($csv_id == 3) {
-                $from = 'dtb_order';
-            }
-            // レビューの場合
-            else if ($csv_id == 4) {
-                $from = 'dtb_review AS A INNER JOIN dtb_products AS B on A.product_id = B.product_id';
-            }
-            // カテゴリの場合
-            else if ($csv_id == 5) {
-                $from = 'dtb_category';
-            }
-
-            $sql = "SELECT $cols FROM $from $where";
-            // 「getSqlを使っているcsv_id=1」以外で「order指定」がある場合は末尾に付与
-            // 全csv_idでgetSqlを使用して生成するよう統一する場合は当メソッドの呼び元も要修正
-            if ($csv_id != 1 && strlen($order) >= 1) {
-                $sql .= " ORDER BY $order";
-            }
-        }
-        // 固有処理ここまで
+            $from = SC_Product_Ex::prdclsSQL($inner_where);
+        }
+        // 会員の場合
+        else if ($csv_id == 2) {
+            $from = 'dtb_customer';
+        }
+        // 注文の場合
+        else if ($csv_id == 3) {
+            $from = 'dtb_order';
+        }
+        // レビューの場合
+        else if ($csv_id == 4) {
+            $from = 'dtb_review AS A INNER JOIN dtb_products AS B on A.product_id = B.product_id';
+        }
+        // カテゴリの場合
+        else if ($csv_id == 5) {
+            $from = 'dtb_category';
+        }
+
+        $objQuery->setOrder($order);
+        $sql = $objQuery->getSql($cols, $from, $where);
 
         return $this->sfDownloadCsvFromSql($sql, $arrVal, $this->arrSubnavi[$csv_id], $arrOutput['disp_name'], $is_download);