Changeset 20332 for branches/version-2_5-dev/data

Timestamp:

2011/02/23 02:20:33 (13 years ago)

Author:

homan

Message:

#974 [管理画面]システム設定 メンバー一覧用ランク制御

File:

• branches/version-2_5-dev/data/class/pages/admin/system/LC_Page_Admin_System_Rank.php (modified) (5 diffs)

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

@@ -61 +61 @@
      */
     function action() {
-        $objQuery = new SC_Query();
 
         // ログインチェック
         SC_Utils::sfIsSuccess(new SC_Session());
 
-        // ランキングの変更
-        if($_GET['move'] == 'up') {
-            // 正当な数値であった場合
-            if(SC_Utils::sfIsInt($_GET['id'])){
-                $this->lfRunkUp($objQuery, $_GET['id']);
-            // エラー処理
-            } else {
-                GC_Utils::gfPrintLog("error id=".$_GET['id']);
-            }
-        } else if($_GET['move'] == 'down') {
-            if(SC_Utils::sfIsInt($_GET['id'])){
-                $this->lfRunkDown($objQuery, $_GET['id']);
-            // エラー処理
-            } else {
-                GC_Utils::gfPrintLog("error id=".$_GET['id']);
+        // チェック後のデータを格納
+        $arrClean = array();
+
+        // $_GET['move'] が想定値かどうかチェック
+        switch($_GET['move']) {
+            case 'up':
+            case 'down':
+                $arrClean['move'] = $_GET['move'];
+                break;
+            default:
+                $arrClean['move'] = "";
+                break;
+        }
+
+
+        // 正当な数値であればOK
+        if (SC_Utils::sfIsInt($_GET['id'])) {
+            $arrClean['id'] = $_GET['id'];
+
+            switch($arrClean['move']) {
+                case 'up':
+                    $this->lfRunkUp($arrClean['id']);
+                    break;
+
+                case 'down':
+                    $this->lfRunkDown($arrClean['id']);
+                    break;
+
+                default:
+                    break;
             }
         }
         
+        // エラー処理
+        else {
+            GC_Utils::gfPrintLog("error id=".$_GET['id']);
+        }
+
         // ページの表示
         SC_Response_Ex::sendRedirect(ADMIN_SYSTEM_URLPATH);
@@ -98 +117 @@
 
     // ランキングを上げる。
-    function lfRunkUp($objQuery, $id) {
+    function lfRunkUp($id) {
+        $objQuery =& SC_Query::getSingletonInstance();
+
         // 自身のランクを取得する。
-        $rank = $objQuery->getOne("SELECT rank FROM dtb_member WHERE member_id = ".$id);
+        $rank = $objQuery->getOne("SELECT rank FROM dtb_member WHERE member_id = ?", array($id));
+
         // ランクの最大値を取得する。
         $maxno = $objQuery->getOne("SELECT max(rank) FROM dtb_member");
@@ -108 +130 @@
             $sqlse = "SELECT member_id FROM dtb_member WHERE rank = ?";
             $up_id = $objQuery->getOne($sqlse, $rank + 1);
+
+            // Updateする値を作成する.
+            $sqlVal1 = array();
+            $sqlVal2 = array();
+            $sqlVal1['rank'] = $rank + 1;
+            $sqlVal2['rank'] = $rank;
+            $where = "member_id = ?";
+
             // ランク入れ替えの実行
             $objQuery->begin();
-            $sqlup = "UPDATE dtb_member SET rank = ? WHERE member_id = ?";
-            $objQuery->query($sqlup, array($rank + 1, $id));
-            $objQuery->query($sqlup, array($rank, $up_id));
+            $objQuery->update("dtb_member", $sqlVal1, $where, array($id));
+            $objQuery->update("dtb_member", $sqlVal2, $where, array($up_id));
             $objQuery->commit();
         }
@@ -118 +147 @@
 
     // ランキングを下げる。
-    function lfRunkDown($objQuery, $id) {
+    function lfRunkDown($id) {
+        $objQuery =& SC_Query::getSingletonInstance();
+
         // 自身のランクを取得する。
-        $rank = $objQuery->getOne("SELECT rank FROM dtb_member WHERE member_id = ".$id);
+        $rank = $objQuery->getOne("SELECT rank FROM dtb_member WHERE member_id = ?", array($id));
         // ランクの最小値を取得する。
         $minno = $objQuery->getOne("SELECT min(rank) FROM dtb_member");
@@ -128 +159 @@
             $sqlse = "SELECT member_id FROM dtb_member WHERE rank = ?";
             $down_id = $objQuery->getOne($sqlse, $rank - 1);
+
+            // Updateする値を作成する.
+            $sqlVal1 = array();
+            $sqlVal2 = array();
+            $sqlVal1['rank'] = $rank - 1;
+            $sqlVal2['rank'] = $rank;
+            $where = "member_id = ?";
+
             // ランク入れ替えの実行
             $objQuery->begin();
-            $sqlup = "UPDATE dtb_member SET rank = ? WHERE member_id = ?";
-            $objQuery->query($sqlup, array($rank - 1, $id));
-            $objQuery->query($sqlup, array($rank, $down_id));
-            $objQuery->query("COMMIT");
+            $objQuery->update("dtb_member", $sqlVal1, $where, array($id));
+            $objQuery->update("dtb_member", $sqlVal2, $where, array($down_id));
+            $objQuery->commit();
         }
     }