Ignore:
Timestamp:
2011/02/22 15:57:30 (11 years ago)
Author:
coelacanth
Message:

#964 リファクタリング 宣言を除き, 引数や返り値が無く, すべて内部のメンバ変数で処理するような関数は極力作成しない

File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/version-2_5-dev/data/class/pages/admin/basis/LC_Page_Admin_Basis_Holiday.php

    r20323 r20326  
    8484            $this->lfInitParam($mode, $objFormParam); 
    8585            $objFormParam->setParam($_POST); 
     86            $objFormParam->convParam(); 
    8687 
    8788            $this->arrErr = $this->lfCheckError($mode, $objFormParam); 
     
    9091                return; 
    9192            } 
     93 
     94            $post = $objFormParam->getHashArray(); 
    9295        } 
    9396 
     
    97100        case 'edit': 
    98101            // POST値の引き継ぎ 
    99             $this->arrForm = $_POST; 
     102            $this->arrForm = $this->arrForm = $_POST; 
    100103 
    101104            if(count($this->arrErr) <= 0) { 
    102                 if($_POST['holiday_id'] == "") { 
     105                if($post['holiday_id'] == "") { 
    103106                    $this->lfInsertClass($this->arrForm, $_SESSION['member_id']);    // 新規作成 
    104107                } else { 
    105                     $this->lfUpdateClass($this->arrForm, $_POST['holiday_id']); // 既存編集 
     108                    $this->lfUpdateClass($this->arrForm, $post['holiday_id']);  // 既存編集 
    106109                } 
    107110                // 再表示 
     
    109112            } else { 
    110113                // POSTデータを引き継ぐ 
    111                 $this->tpl_holiday_id = $_POST['holiday_id']; 
     114                $this->tpl_holiday_id = $post['holiday_id']; 
    112115            } 
    113116            break; 
    114117        // 削除 
    115118        case 'delete': 
    116             $objDb->sfDeleteRankRecord("dtb_holiday", "holiday_id", $_POST['holiday_id'], "", true); 
     119            $objDb->sfDeleteRankRecord("dtb_holiday", "holiday_id", $post['holiday_id'], "", true); 
    117120            // 再表示 
    118121            $this->objDisplay->reload(); 
     
    121124        case 'pre_edit': 
    122125            // 編集項目を取得する。 
    123             $arrHolidayData = $this->lfGetHolidayDataByHolidayID($_POST['holiday_id']); 
     126            $arrHolidayData = $this->lfGetHolidayDataByHolidayID($post['holiday_id']); 
    124127 
    125128            // 入力項目にカテゴリ名を入力する。 
     
    128131            $this->arrForm['day'] = $arrHolidayData[0]['day']; 
    129132            // POSTデータを引き継ぐ 
    130             $this->tpl_holiday_id = $_POST['holiday_id']; 
     133            $this->tpl_holiday_id = $post['holiday_id']; 
    131134        break; 
    132135        case 'down': 
    133             $objDb->sfRankDown("dtb_holiday", "holiday_id", $_POST['holiday_id']); 
     136            $objDb->sfRankDown("dtb_holiday", "holiday_id", $post['holiday_id']); 
    134137            // 再表示 
    135138            $this->objDisplay->reload(); 
    136139            break; 
    137140        case 'up': 
    138             $objDb->sfRankUp("dtb_holiday", "holiday_id", $_POST['holiday_id']); 
     141            $objDb->sfRankUp("dtb_holiday", "holiday_id", $post['holiday_id']); 
    139142            // 再表示 
    140143            $this->objDisplay->reload(); 
     
    189192 
    190193    /* DBへの更新 */ 
    191     function lfUpdateClass($arrData, $holiday_id) { 
     194    function lfUpdateClass($arrData) { 
    192195        $objQuery =& SC_Query::getSingletonInstance(); 
    193196        // UPDATEする値を作成する。 
     
    198201        $where = "holiday_id = ?"; 
    199202        // UPDATEの実行 
    200         $ret = $objQuery->update("dtb_holiday", $sqlval, $where, array($holiday_id)); 
     203        $ret = $objQuery->update("dtb_holiday", $sqlval, $where, array($arrData['holiday_id'])); 
    201204        return $ret; 
    202205    } 
Note: See TracChangeset for help on using the changeset viewer.