[Svn-src-all:1398] [version-2_5-dev 20288] ref #1015 [管理画面]コンテンツ管理(CSV出力項目設定以外)

nakanishi admin @ mail.ec-cube.net
2011年 2月 21日 (月) 10:13:55 JST


Subversion committed to /home/svn/open 20288
http://svn.ec-cube.net/open_trac/changeset/20288
┌────────────────────────────┐
│更新者 :  nakanishi                                    │
│更新日時:  2011-02-21 10:13:55 +0900 (月, 21  2月 2011)│
└────────────────────────────┘

Log:
--------------------------------------------------------
ref #1015 [管理画面]コンテンツ管理(CSV出力項目設定以外)
再度リファクタリング。
変数名の修正。
デバッグ文を削除。
メソッドにコメントを付与。



Changed:                      [U:修正,A:追加,D:削除]
--------------------------------------------------------
U   branches/version-2_5-dev/data/Smarty/templates/admin/contents/index.tpl
U   branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents.php
U   branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents_Recommend.php
U   branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents_RecommendSearch.php

変更: branches/version-2_5-dev/data/Smarty/templates/admin/contents/index.tpl
===================================================================
--- branches/version-2_5-dev/data/Smarty/templates/admin/contents/index.tpl	2011-02-20 22:24:58 UTC (rev 20287)
+++ branches/version-2_5-dev/data/Smarty/templates/admin/contents/index.tpl	2011-02-21 01:13:55 UTC (rev 20288)
@@ -196,26 +196,26 @@
       <th class="delete">削除</th>
       <th>移動</th>
     </tr>
-    <!--{section name=data loop=$list_data}-->
-    <tr style="background:<!--{if $list_data[data].news_id eq $news_id}--><!--{$smarty.const.SELECT_RGB}--><!--{else}-->#ffffff<!--{/if}-->;" class="center">
-      <!--{assign var=db_rank value="`$list_data[data].rank`"}-->
-      <!--{assign var=rank value="`$line_max-$db_rank+1`"}-->
+    <!--{section name=data loop=$arrNews}-->
+    <tr style="background:<!--{if $arrNews[data].news_id eq $news_id}--><!--{$smarty.const.SELECT_RGB}--><!--{else}-->#ffffff<!--{/if}-->;" class="center">
+      <!--{assign var=db_rank value="`$arrNews[data].rank`"}-->
+      <!--{assign var=rank value="`$line_max - $db_rank+1`"}-->
       <td><!--{$rank|h}--></td>
-      <td><!--{$list_data[data].cast_news_date|date_format:"%Y/%m/%d"}--></td>
+      <td><!--{$arrNews[data].cast_news_date|date_format:"%Y/%m/%d"}--></td>
       <td class="left">
-        <!--{if $list_data[data].link_method eq 1 && $list_data[data].news_url != ""}--><a href="<!--{$list_data[data].news_url|h}-->" ><!--{$list_data[data].news_title|h|nl2br}--></a>
-        <!--{elseif $list_data[data].link_method eq 1 && $list_data[data].news_url == ""}--><!--{$list_data[data].news_title|h|nl2br}-->
-        <!--{elseif $list_data[data].link_method eq 2 && $list_data[data].news_url != ""}--><a href="<!--{$list_data[data].news_url|h}-->" target="_blank" ><!--{$list_data[data].news_title|h|nl2br}--></a>
-        <!--{else}--><!--{$list_data[data].news_title|h|nl2br}-->
+        <!--{if $arrNews[data].link_method eq 1 && $arrNews[data].news_url != ""}--><a href="<!--{$arrNews[data].news_url|h}-->" ><!--{$arrNews[data].news_title|h|nl2br}--></a>
+        <!--{elseif $arrNews[data].link_method eq 1 && $arrNews[data].news_url == ""}--><!--{$arrNews[data].news_title|h|nl2br}-->
+        <!--{elseif $arrNews[data].link_method eq 2 && $arrNews[data].news_url != ""}--><a href="<!--{$arrNews[data].news_url|h}-->" target="_blank" ><!--{$arrNews[data].news_title|h|nl2br}--></a>
+        <!--{else}--><!--{$arrNews[data].news_title|h|nl2br}-->
         <!--{/if}-->
       </td>
-      <td><a href="#" onclick="return func_edit('<!--{$list_data[data].news_id|h}-->');">編集</a></td>
-      <td><a href="#" onclick="return func_del('<!--{$list_data[data].news_id|h}-->');">削除</a></td>
+      <td><a href="#" onclick="return func_edit('<!--{$arrNews[data].news_id|h}-->');">編集</a></td>
+      <td><a href="#" onclick="return func_del('<!--{$arrNews[data].news_id|h}-->');">削除</a></td>
       <td>
-      <!--{if count($list_data) != 1}-->
-      <input type="text" name="pos-<!--{$list_data[data].news_id|h}-->" size="3" class="box3" />番目へ<a href="?" onclick="fnFormModeSubmit('move', 'moveRankSet','news_id', '<!--{$list_data[data].news_id|h}-->'); return false;">移動</a><br />
+      <!--{if count($arrNews) != 1}-->
+      <input type="text" name="pos-<!--{$arrNews[data].news_id|h}-->" size="3" class="box3" />番目へ<a href="?" onclick="fnFormModeSubmit('move', 'moveRankSet','news_id', '<!--{$arrNews[data].news_id|h}-->'); return false;">移動</a><br />
       <!--{/if}-->
-      <!--{if $list_data[data].rank ne $max_rank}--><a href="#" onclick="return func_rankMove('up', '<!--{$list_data[data].news_id|h}-->', '<!--{$max_rank|h}-->');">上へ</a><!--{/if}--> <!--{if $list_data[data].rank ne 1}--><a href="#" onclick="return func_rankMove('down', '<!--{$list_data[data].news_id|h}-->', '<!--{$max_rank|h}-->');">下へ</a><!--{/if}-->
+      <!--{if $arrNews[data].rank ne $max_rank}--><a href="#" onclick="return func_rankMove('up', '<!--{$arrNews[data].news_id|h}-->', '<!--{$max_rank|h}-->');">上へ</a><!--{/if}--> <!--{if $arrNews[data].rank ne 1}--><a href="#" onclick="return func_rankMove('down', '<!--{$arrNews[data].news_id|h}-->', '<!--{$max_rank|h}-->');">下へ</a><!--{/if}-->
       </td>
     </tr>
     <!--{sectionelse}-->

変更: branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents.php
===================================================================
--- branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents.php	2011-02-20 22:24:58 UTC (rev 20287)
+++ branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents.php	2011-02-21 01:13:55 UTC (rev 20288)
@@ -118,10 +118,9 @@
         case 'delete':
         //---- データ削除
             if (is_numeric($news_id)) {
-                $this->p($news_id);
                 $pre_rank = $this->getRankByNewsId($news_id);
                 $this->computeRankForDelete($news_id,$pre_rank);
-                $this->objDisplay->reload();             //自分にリダイレクト(再読込による誤動作防止)
+                SC_Response_Ex::reload();             //自分にリダイレクト(再読込による誤動作防止)
             }
             break;
         case 'move':
@@ -148,8 +147,8 @@
             break;
         }
 
-        $this->list_data = $this->getNews();
-        $this->line_max = count($this->list_data);
+        $this->arrNews = $this->getNews();
+        $this->line_max = count($this->arrNews);
         $this->max_rank = $this->getRankMax();
     }
 

変更: branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents_Recommend.php
===================================================================
--- branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents_Recommend.php	2011-02-20 22:24:58 UTC (rev 20287)
+++ branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents_Recommend.php	2011-02-21 01:13:55 UTC (rev 20288)
@@ -112,7 +112,7 @@
             break;
         }
         
-        $this->category_id = $this->getCategoryId($arrPost['category_id']);
+        $this->category_id = intval($arrPost['category_id']);
         $this->arrItems = $arrItems;
 
         // カテゴリ取得
@@ -237,18 +237,5 @@
         return $arrItems;
     }
 
-    /**
-     * カテゴリーIDを取得する。
-     * @param Integer $category_id
-     * @return Integer
-     */
-    function getCategoryId($category_id){
-        if ( SC_Utils_Ex::sfIsInt($category_id)){
-            return $category_id;
-        }else{
-            return 0;
-        }
-    }
-
 }
 ?>

変更: branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents_RecommendSearch.php
===================================================================
--- branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents_RecommendSearch.php	2011-02-20 22:24:58 UTC (rev 20287)
+++ branches/version-2_5-dev/data/class/pages/admin/contents/LC_Page_Admin_Contents_RecommendSearch.php	2011-02-21 01:13:55 UTC (rev 20288)
@@ -175,12 +175,22 @@
             return array($where, $arrval);
     }
 
+    /**
+     * 指定された条件の商品の数を取得する
+     * @param String $where 検索条件
+     * @param Array $arrval 検索パラメータ
+     * @return Integer $linemax 条件にマッチする商品の数
+     */
     function getLineCount($where,$arrval){
         $objQuery =& SC_Query::getSingletonInstance();
         $linemax = $objQuery->count("dtb_products", $where, $arrval);
         return $linemax;
     }
 
+    /**
+     * @param  $search_page_max
+     * @return string
+     */
     function setPageMax($search_page_max){
             // ページ送りの処理
             if(isset($search_page_max)  && is_numeric($search_page_max)) {
@@ -191,6 +201,13 @@
             return $page_max;
     }
 
+    /**
+     * 指定された条件の商品データを取得する。
+     * @param Object $objQuery ObjQueryオブジェクト
+     * @param Array $arrval 検索パラメータ
+     * @param String $where 検索条件
+     * @return Array 商品情報の配列
+     */
     function getProducts(&$objQuery,$arrval,$where){
         $objQuery =& SC_Query::getSingletonInstance();
         return $objQuery->select("*", SC_Product::alldtlSQL(), $where, $arrval);




Svn-src-all メーリングリストの案内