Changeset 20254

Timestamp:

2011/02/20 16:39:58 (15 years ago)

Author:

homan

Message:

#974　/admin/system/LC_Page_Admin_System.php とテンプレ（index.tpl）のリファクタリング。
メンバー一覧で稼働・非稼働がわからなかったので、ついでに追加。

Location:

branches/version-2_5-dev/data

Files:

• Smarty/templates/admin/system/index.tpl (modified) (3 diffs)
• class/pages/admin/system/LC_Page_Admin_System.php (modified) (5 diffs)

branches/version-2_5-dev/data/Smarty/templates/admin/system/index.tpl

@@ -35 +35 @@
     <colgroup width="20%">
     <colgroup width="20%">
+    <colgroup width="10%">
     <colgroup width="5%">
     <colgroup width="5%">
@@ -45 +46 @@
       <th>名前</th>
       <th>所属</th>
+      <th>稼働</th>
       <th>編集</th>
       <th>削除</th>
@@ -54 +56 @@
       <td><!--{$list_data[data].name|h}--></td>
       <td><!--{$list_data[data].department|h}--></td>
+      <!--{assign var="work" value=$list_data[data].work}--><td><!--{$arrWORK[$work]|h}--></td>
       <td align="center"><a href="#" onClick="win01('./input.php?id=<!--{$list_data[data].member_id}-->&amp;pageno=<!--{$tpl_disppage}-->','member_edit','615','450'); return false;">編集</a></td>
       <td align="center"><!--{if $workmax > 1}--><a href="#" onClick="fnDeleteMember(<!--{$list_data[data].member_id}-->,<!--{$tpl_disppage}-->); return false;">削除</a><!--{else}-->-<!--{/if}--></td>

branches/version-2_5-dev/data/class/pages/admin/system/LC_Page_Admin_System.php

@@ -57 +57 @@
         $masterData = new SC_DB_MasterData_Ex();
         $this->arrAUTHORITY = $masterData->getMasterData('mtb_authority');
+        $this->arrWORK[0]   = "非稼働";
+        $this->arrWORK[1]   = "稼働";
     }
 
@@ -75 +77 @@
      */
     function action() {
-        $objSess  = new SC_Session();
 
         // 認証可否の判定
-        SC_Utils_Ex::sfIsSuccess($objSess);
+        SC_Utils_Ex::sfIsSuccess(new SC_Session());
 
         // ADMIN_ID以外の管理者件数を取得
@@ -87 +88 @@
             = $this->getMemberCount("work = 1 AND del_flg <> 1 AND member_id <> " . ADMIN_ID);
 
-        // ページ送りの処理
-        $pageno = isset($_GET['pageno']) ? $_GET['pageno'] : 1;
+        // ページ送りの処理 $_GET['pageno']が信頼しうる値かどうかチェックする。
+        $pageno = $this->lfCheckPageNo($_GET['pageno']);
+
         $objNavi = new SC_PageNavi($pageno, $linemax, MEMBER_PMAX, "fnMemberPage", NAVI_PMAX);
         $this->tpl_strnavi  = $objNavi->strnavi;
@@ -115 +117 @@
      */
      function getMemberCount($where) {
-        $objQuery = new SC_Query();
+        $objQuery =& SC_Query::getSingletonInstance();
         $table = 'dtb_member';
         return $objQuery->count($table, $where);
@@ -134 +136 @@
         $objSql->setLimitOffset(MEMBER_PMAX, $startno);
 
-        $objQuery = new SC_Query();
+        $objQuery =& SC_Query::getSingletonInstance();
         $arrMemberData = $objQuery->getAll($objSql->getSql());
 
         return $arrMemberData;
      }
+
+    /**
+     * ページ番号が信頼しうる値かチェックする.
+     *
+     * @access private
+     * @param integer  $pageno ページの番号（$_GETから入ってきた値）
+     * @return integer $clean_pageno チェック後のページの番号
+     */
+    function lfCheckPageNo($pageno) {
+
+        $clean_pageno = "";
+
+        // $pagenoが0以上の整数かチェック
+        if(SC_Utils_Ex::sfIsInt($pageno) && $pageno > 0) {
+            $clean_pageno = $pageno;
+        }
+
+        // 例外は全て1とする
+        else {
+            $clean_pageno = 1;
+        }
+
+        return $clean_pageno;
+    }
 }
 ?>