Changeset 22833

Timestamp:

2013/05/21 12:01:24 (11 years ago)

Author:

kimoto

Message:

#2043 (typo修正・ソース整形・ソースコメントの改善 for 2.13.0)
商品表示条件が点在しているので1箇所にまとめてプラグインを作りやすいよう
にする

Location:

branches/version-2_13-dev/data/class

Files:

• SC_Product.php (modified) (1 diff)
• helper/SC_Helper_DB.php (modified) (10 diffs)
• pages/LC_Page_Sitemap.php (modified) (2 diffs)
• pages/products/LC_Page_Products_List.php (modified) (3 diffs)
• pages/products/LC_Page_Products_Review.php (modified) (2 diffs)
• pages/rss/LC_Page_Rss_Products.php (modified) (2 diffs)

branches/version-2_13-dev/data/class/SC_Product.php

@@ -773 +773 @@
         return $sql;
     }
+
+    function getProductDispConditions($tablename = null) {
+        $tablename = ($tablename) ? $tablename . '.' : null;
+        return $tablename . 'del_flg = 0 AND ' . $tablename . 'status = 1 ';
+    }
+
 }

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

@@ -29 +29 @@
  * @version $Id:SC_Helper_DB.php 15532 2007-08-31 14:39:46Z nanasess $
  */
-class SC_Helper_DB 
+class SC_Helper_DB
 {
 
@@ -91 +91 @@
         return false;
     }
-    
+
     function sfColumnAdd($tableName, $colName, $colType) {
         $objQuery =& SC_Query_Ex::getSingletonInstance($dsn);
@@ -185 +185 @@
      * 基本情報のキャッシュデータファイルを生成する
      * データはsfGetBasisDataより取得。
-     * 
+     *
      * このメソッドが直接呼ばれるのは、
      *「基本情報管理＞SHOPマスター」の更新完了後。
@@ -726 +726 @@
 
         //共通のfrom/where文の構築
-        $sql_where = 'alldtl.del_flg = 0 AND alldtl.status = 1';
+        $sql_where = SC_Product_Ex::getProductDispConditions('alldtl');
         // 在庫無し商品の非表示
         if (NOSTOCK_HIDDEN) {
@@ -1161 +1161 @@
 
         $max = $objQuery->max('rank', $tableName, $where);
-        
+
         // 更新するランク値を取得
         $newRank = $this->getNewRank($pos, $max);
-        // 他のItemのランクを調整する 
+        // 他のItemのランクを調整する
         $ret = $this->moveOtherItemRank($newRank, $oldRank, $objQuery, $tableName, $where);
         if (!$ret) {
@@ -1184 +1184 @@
         $objQuery->commit();
     }
-    
+
     /**
      * 指定された位置の値をDB用のRANK値に変換する
      * 指定位置が1番目に移動なら、newRankは最大値
      * 指定位置が1番下へ移動なら、newRankは1
-     * 
+     *
      * @param int $position 指定された位置
      * @param int $maxRank 現在のランク最大値
      * @return int $newRank DBに登録するRANK値
-     */ 
+     */
     function getNewRank($position, $maxRank) {
-        
+
         if ($position > $maxRank) {
             $newRank = 1;
@@ -1208 +1208 @@
     /**
      * 指定した順位の商品から移動させる商品までのrankを１つずらす
-     * 
+     *
      * @param int $newRank
      * @param int $oldRank
      * @param object $objQuery
      * @param string $where
-     * @return boolean 
+     * @return boolean
      */
     function moveOtherItemRank($newRank, $oldRank, &$objQuery, $tableName, $addWhere) {
-        
+
         $sqlval = array();
         $arrRawSql = array();
@@ -1235 +1235 @@
             return false;
         }
-        
+
         return $objQuery->update($tableName, $sqlval, $where, $arrWhereVal, $arrRawSql);
-        
-    }
-    
+
+    }
+
 
     /**
@@ -1324 +1324 @@
     /**
      * カテゴリ変更時の移動処理を行う.
-     * 
+     *
      * ※この関数って、どこからも呼ばれていないのでは？？
      *
@@ -1622 +1622 @@
     /**
      * レコード件数を計算.
-     * 
+     *
      * @param string $table
      * @param string $where

branches/version-2_13-dev/data/class/pages/LC_Page_Sitemap.php

@@ -51 +51 @@
  * :TODO: filemtime 関数を使えば、静的なページの更新時間も取得できそう
  */
-class LC_Page_Sitemap extends LC_Page_Ex 
+class LC_Page_Sitemap extends LC_Page_Ex
 {
 
@@ -240 +240 @@
     {
         $objQuery = SC_Query_Ex::getSingletonInstance();
-        $sql = 'SELECT product_id, update_date FROM dtb_products WHERE del_flg = 0 AND status = 1';
+        $sql = 'SELECT product_id, update_date FROM dtb_products WHERE ' . SC_Product_Ex::getProductDispConditions();
         $result = $objQuery->getAll($sql);
 

branches/version-2_13-dev/data/class/pages/products/LC_Page_Products_List.php

@@ -32 +32 @@
  * @version $Id$
  */
-class LC_Page_Products_List extends LC_Page_Ex 
+class LC_Page_Products_List extends LC_Page_Ex
 {
 
@@ -362 +362 @@
         // ▼対象商品IDの抽出
         // 商品検索条件の作成（未削除、表示）
-        $searchCondition['where'] = 'alldtl.del_flg = 0 AND alldtl.status = 1 ';
+        $searchCondition['where'] = SC_Product_Ex::getProductDispConditions('alldtl');
 
         if (strlen($searchCondition['where_category']) >= 1) {
@@ -483 +483 @@
     /**
      *
-     * @param type $objProduct 
+     * @param type $objProduct
      * @return void
      */

branches/version-2_13-dev/data/class/pages/products/LC_Page_Products_Review.php

@@ -32 +32 @@
  * @version $Id:LC_Page_Products_Review.php 15532 2007-08-31 14:39:46Z nanasess $
  */
-class LC_Page_Products_Review extends LC_Page_Ex 
+class LC_Page_Products_Review extends LC_Page_Ex
 {
 
@@ -203 +203 @@
         $objQuery =& SC_Query_Ex::getSingletonInstance();
 
-        return $objQuery->get('name', 'dtb_products', 'product_id = ? AND del_flg = 0 AND status = 1', array($product_id));
+        return $objQuery->get('name', 'dtb_products', 'product_id = ? AND ' . SC_Product_Ex::getProductDispConditions(), array($product_id));
     }
 

branches/version-2_13-dev/data/class/pages/rss/LC_Page_Rss_Products.php

@@ -32 +32 @@
  * @version $Id$
  */
-class LC_Page_Rss_Products extends LC_Page_Ex 
+class LC_Page_Rss_Products extends LC_Page_Ex
 {
 
@@ -296 +296 @@
     {
         // --- 商品一覧の取得
-        $objQuery->setWhere('del_flg = 0 AND status = 1');
+        $objProduct = new SC_Product_Ex();
+        $objQuery->setWhere($objProduct->getProductDispConditions());
         $objQuery->setOrder('product_id');
-        $objProduct = new SC_Product_Ex();
         $arrProductLsit = $objProduct->lists($objQuery);
         // 各商品のカテゴリIDとランクの取得