Changeset 17959

Timestamp:

2009/04/09 21:41:05 (14 years ago)

Author:

Seasoft

Message:

商品一覧
・複数のカテゴリに登録された商品をカテゴリを指定せずに検索すると、重複して表示される不具合を修正。

flealog 様のご指摘。 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=3739&forum=1

・副作用により、パフォーマンスが改善された模様。

File:

• branches/comu-ver2/data/class/pages/products/LC_Page_Products_List.php (modified) (5 diffs)

branches/comu-ver2/data/class/pages/products/LC_Page_Products_List.php

@@ -402 +402 @@
             }
         }
-
+        
+        // ▼対象商品IDの抽出
+        // 商品検索条件の作成（未削除、表示）
+        $where = "del_flg = 0 AND status = 1 ";
+        
+        // 在庫無し商品の非表示
+        if (NOSTOCK_HIDDEN === true) {
+            $where .= ' AND (stock_max >= 1 OR stock_unlimited_max = 1)';
+        }
+        
+        if (strlen($where_category) >= 1) {
+            $where.= " $where_category";
+            $arrval = array_merge($arrval, $arrval_category);
+        }
+
+        // 商品名をwhere文に
+        $name = ereg_replace(",", "", $name);// XXX
+        // 全角スペースを半角スペースに変換
+        $name = str_replace('　', ' ', $name);
+        // スペースでキーワードを分割
+        $names = preg_split("/ +/", $name);
+        // 分割したキーワードを一つずつwhere文に追加
+        foreach ($names as $val) {
+            if ( strlen($val) > 0 ){
+                $where .= " AND ( name ILIKE ? OR comment3 ILIKE ?) ";
+                $ret = SC_Utils_Ex::sfManualEscape($val);
+                $arrval[] = "%$ret%";
+                $arrval[] = "%$ret%";
+            }
+        }
+
+        // メーカーらのWHERE文字列取得
+        if ( $maker_id ) {
+            $where .= " AND maker_id = ? ";
+            $arrval[] = $maker_id;
+        }
+        
+        $arrProduct_id = $objQuery->getCol('vw_products_allclass AS allcls', 'DISTINCT product_id', $where, $arrval);
+        // ▲対象商品IDの抽出
+        
+        // 行数の取得
+        $linemax = count($arrProduct_id);
+
+        $this->tpl_linemax = $linemax;   // 何件が該当しました。表示用
+
+        // ページ送りの取得
+        $this->objNavi = new SC_PageNavi($this->tpl_pageno, $linemax, $disp_num, "fnNaviPage", NAVI_PMAX);
+
+        $strnavi = $this->objNavi->strnavi;
+        $strnavi = str_replace('onclick="fnNaviPage', 'onclick="form1.mode.value=\''.'\'; fnNaviPage', $strnavi);
+        // 表示文字列
+        $this->tpl_strnavi = empty($strnavi) ? " " : $strnavi;
+        $startno = $this->objNavi->start_row;                 // 開始行
+        
+        // ▼商品詳細取得
         $col = <<< __EOS__
              product_id
@@ -431 +485 @@
             ,del_flg
 __EOS__;
-        $from = "vw_products_allclass AS allcls";
+        
+        $from = "vw_products_allclass_detail AS alldtl";
+        
+        // WHERE 句
+        $where = '0=0';
+        if (is_array($arrProduct_id) && !empty($arrProduct_id)) {
+            $where .= ' AND product_id IN (' . implode(',', $arrProduct_id) . ')';
+        }
         
         //表示順序
@@ -455 +516 @@
                             JOIN dtb_category T3
                                 USING (category_id)
-                        WHERE T2.product_id = allcls.product_id
+                        WHERE T2.product_id = alldtl.product_id
                             $where_category
                         ORDER BY T3.rank DESC, T2.rank DESC
@@ -467 +528 @@
                             JOIN dtb_category T3
                                 USING (category_id)
-                        WHERE T2.product_id = allcls.product_id
+                        WHERE T2.product_id = alldtl.product_id
                             $where_category
                         ORDER BY T3.rank DESC, T2.rank DESC
@@ -477 +538 @@
                 break;
         }
-
-        // 商品検索条件の作成（未削除、表示）
-        $where = "del_flg = 0 AND status = 1 ";
-        
-        // 在庫無し商品の非表示
-        if (NOSTOCK_HIDDEN === true) {
-            $where .= ' AND (allcls.stock_max >= 1 OR allcls.stock_unlimited_max = 1)';
-        }
-        
-        if (strlen($where_category) >= 1) {
-            $where.= " $where_category";
-            $arrval = array_merge($arrval, $arrval_category);
-        }
-
-        // 商品名をwhere文に
-        $name = ereg_replace(",", "", $name);// XXX
-        // 全角スペースを半角スペースに変換
-        $name = str_replace('　', ' ', $name);
-        // スペースでキーワードを分割
-        $names = preg_split("/ +/", $name);
-        // 分割したキーワードを一つずつwhere文に追加
-        foreach ($names as $val) {
-            if ( strlen($val) > 0 ){
-                $where .= " AND ( name ILIKE ? OR comment3 ILIKE ?) ";
-                $ret = SC_Utils_Ex::sfManualEscape($val);
-                $arrval[] = "%$ret%";
-                $arrval[] = "%$ret%";
-            }
-        }
-
-        // メーカーらのWHERE文字列取得
-        if ( $maker_id ) {
-            $where .= " AND maker_id = ? ";
-            $arrval[] = $maker_id;
-        }
-        
-        // 行数の取得
-        $linemax = count($objQuery->getAll("SELECT DISTINCT product_id "
-                                         . "FROM vw_products_allclass AS allcls "
-                                         . (!empty($where) ? " WHERE " . $where
-                                                           : ""), $arrval));
-
-        $this->tpl_linemax = $linemax;   // 何件が該当しました。表示用
-
-        // ページ送りの取得
-        $this->objNavi = new SC_PageNavi($this->tpl_pageno, $linemax, $disp_num, "fnNaviPage", NAVI_PMAX);
-
-        $strnavi = $this->objNavi->strnavi;
-        $strnavi = str_replace('onclick="fnNaviPage', 'onclick="form1.mode.value=\''.'\'; fnNaviPage', $strnavi);
-        // 表示文字列
-        $this->tpl_strnavi = empty($strnavi) ? "&nbsp;" : $strnavi;
-        $startno = $this->objNavi->start_row;                 // 開始行
-
+        
         // 取得範囲の指定(開始行番号、行数のセット)
         $objQuery->setlimitoffset($disp_num, $startno);
         // 表示順序
         $objQuery->setorder($order);
-
+        
         // 検索結果の取得
-        $this->arrProducts = $objQuery->select($col, $from, $where, array_merge($arrval, $arrval_order));
-
+        $this->arrProducts = $objQuery->select($col, $from, $where, $arrval_order);
+        // ▲商品詳細取得
+        
         // 規格名一覧
         $arrClassName = $objDb->sfGetIDValueList("dtb_class", "class_id", "name");