Changeset 23021

Timestamp:

2013/08/02 12:59:41 (11 years ago)

Author:

Seasoft

Message:

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

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

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

File:

• branches/version-2_13-dev/data/class/SC_FormParam.php (modified) (3 diffs)

branches/version-2_13-dev/data/class/SC_FormParam.php

@@ -550 +550 @@
     /**
      * パラメーターの削除
+     *
      * addParamの逆の関数
-     * カスタマイズおよびプラグインで使用されるのを想定
      */
     function removeParam($keyname)
@@ -558 +558 @@
 
         if ($index !== FALSE) {
-            // $this->paramに歯抜けが存在する場合は、NULLで埋めておく。
-            // 最後に配列を詰める際に、全ての項目が埋まっている必要がある。
-            foreach ($this->keyname as $key => $value) {
-                if (!isset($this->param[$key])) {
-                    $this->param[$key] = NULL;
-                }
-            }
-            // $this->paramがソートされていない時があるのでソート。
-            ksort($this->param);
-
             // 削除
             unset($this->disp_name[$index]);
@@ -574 +564 @@
             unset($this->convert[$index]);
             unset($this->arrCheck[$index]);
-            unset($this->default[$index]);
+            unset($this->arrDefault[$keyname]);
             unset($this->input_db[$index]);
-            unset($this->param[$index]);
 
             // 歯抜けになった配列を詰める
-            $this->disp_name = array_merge($this->disp_name);
-            $this->keyname = array_merge($this->keyname);
-            $this->length = array_merge($this->length);
-            $this->convert = array_merge($this->convert);
-            $this->arrCheck = array_merge($this->arrCheck);
-            $this->default = array_merge($this->default);
-            $this->input_db = array_merge($this->input_db);
-            $this->param = array_merge($this->param);
+            $this->disp_name    = array_merge($this->disp_name);
+            $this->keyname      = array_merge($this->keyname);
+            $this->length       = array_merge($this->length);
+            $this->convert      = array_merge($this->convert);
+            $this->arrCheck     = array_merge($this->arrCheck);
+            $this->input_db     = array_merge($this->input_db);
         }
     }