Changeset 23021


Ignore:
Timestamp:
2013/08/02 12:59:41 (7 years ago)
Author:
Seasoft
Message:

#2321 (SC_FormParam#removeParam デフォルト値の対象変数と配列要素を誤って処理している)
#2044 (無駄な処理を改善する for 2.13.0)

  • クラス変数 param は存在しない。

#2043 (typo修正・ソース整形・ソースコメントの改善 for 2.13.0)

File:
1 edited

Legend:

Unmodified
Added
Removed
  • branches/version-2_13-dev/data/class/SC_FormParam.php

    r22911 r23021  
    550550    /** 
    551551     * パラメーターの削除 
     552     * 
    552553     * addParamの逆の関数 
    553      * カスタマイズおよびプラグインで使用されるのを想定 
    554554     */ 
    555555    function removeParam($keyname) 
     
    558558 
    559559        if ($index !== FALSE) { 
    560             // $this->paramに歯抜けが存在する場合は、NULLで埋めておく。 
    561             // 最後に配列を詰める際に、全ての項目が埋まっている必要がある。 
    562             foreach ($this->keyname as $key => $value) { 
    563                 if (!isset($this->param[$key])) { 
    564                     $this->param[$key] = NULL; 
    565                 } 
    566             } 
    567             // $this->paramがソートされていない時があるのでソート。 
    568             ksort($this->param); 
    569  
    570560            // 削除 
    571561            unset($this->disp_name[$index]); 
     
    574564            unset($this->convert[$index]); 
    575565            unset($this->arrCheck[$index]); 
    576             unset($this->default[$index]); 
     566            unset($this->arrDefault[$keyname]); 
    577567            unset($this->input_db[$index]); 
    578             unset($this->param[$index]); 
    579568 
    580569            // 歯抜けになった配列を詰める 
    581             $this->disp_name = array_merge($this->disp_name); 
    582             $this->keyname = array_merge($this->keyname); 
    583             $this->length = array_merge($this->length); 
    584             $this->convert = array_merge($this->convert); 
    585             $this->arrCheck = array_merge($this->arrCheck); 
    586             $this->default = array_merge($this->default); 
    587             $this->input_db = array_merge($this->input_db); 
    588             $this->param = array_merge($this->param); 
     570            $this->disp_name    = array_merge($this->disp_name); 
     571            $this->keyname      = array_merge($this->keyname); 
     572            $this->length       = array_merge($this->length); 
     573            $this->convert      = array_merge($this->convert); 
     574            $this->arrCheck     = array_merge($this->arrCheck); 
     575            $this->input_db     = array_merge($this->input_db); 
    589576        } 
    590577    } 
Note: See TracChangeset for help on using the changeset viewer.