疑是THINKPHP的一个表单自动填充BUG

最新推荐文章于 2021-03-10 18:42:07 发布

原创最新推荐文章于 2021-03-10 18:42:07 发布 · 167 阅读

0 ·

CC 4.0 BY-SA版权

文章标签：

#PHP

thinkphp 专栏收录该内容

8 篇文章

订阅专栏

本文详细介绍了THINKPHP框架中的自动填充特性，包括如何通过自定义模型类实现字段值的自动填充、数据过滤等功能，并针对源码进行了深入解析。此外还探讨了一种特殊情况下的解决方案。

THINKPHP有一个很不错的特性，就是能够设置自动填充来处理默认值、数据过滤以及其他系
统写入字段。

该特性通过自定义模型继承Model类进行实现。

例如，在自定义模型类中，设置$_auto属性


class ArticleModel extends Model
{
//定义$_auto属性，进行字段值的自动填充，数据过滤等。
protected $_auto = array(   
array(

   'title', //(填充的字段名称

   'autofill', //进行数据填充的该模型类的方法

   3,  //所有情况（1为新增数据的时候处理，2为更新数据的时候处理，3所有情况都进行处理 ）

   'callback',//调用该模型类的某一个方法进行数据过滤填充

   '自定义的参数';//若为callback，或者function，则此处为函数或方法所需的自定义参数（若未填写该表单字段，通常可设定为默认值）
));
}

该属性通过调用Model的autoOperation方法，进行实现，现在看看源代码：

打开[color=green][b]Think/core/Model.class.php[/b][/color]，找到大概857行：


private function autoOperation(&$data,$type) {
        // 自动填充
        if(!empty($this->_auto)) {
            foreach ($this->_auto as $auto){
                // 填充因子定义格式
                // array('field','填充内容','填充条件','附加规则',[额外参数])
                if(empty($auto[2])) $auto[2] = self::MODEL_INSERT; // 默认为新增的时候自动填充
                if( $type == $auto[2] || $auto[2] == self::MODEL_BOTH) {
                    switch($auto[3]) {
                        case 'function':    //  使用函数进行填充 字段的值作为参数
                        case 'callback': // 使用回调方法
                                 //####问题就在此处#####
                        	$args = isset($auto[4])?$auto[4]:array();

                            if(isset($data[$auto[0]])) {
                                array_unshift($args,$data[$auto[0]]);
                            }
                            if('function'==$auto[3]) {
                                $data[$auto[0]]  = call_user_func_array($auto[1], $args);
                            }else{
                                $data[$auto[0]]  =  call_user_func_array(array(&$this,$auto[1]), $args);
                            }
                            break;
                        case 'field':    // 用其它字段的值进行填充
                            $data[$auto[0]] = $data[$auto[1]];
                            break;
                        case 'string':
                        default: // 默认作为字符串填充
                            $data[$auto[0]] = $auto[1];
                    }
                    if(false === $data[$auto[0]] )   unset($data[$auto[0]]);
                }
            }
        }
        return $data;
    }

看完该方法后，就会有个问题：如果设置了$auto[4]，并且该值不为数组，则array_unshift函数作用后的$args值依然是$auto[4]（因为array_unshift函数的第一个参数值若不为数组，则直接返回参数值）。

这样，按照之前的$_auto属性定义，调用用户自定义的函数，将只能获取$auto[4]的值，不能获取提交的该表单值。

两种解决方案，第一种将$auto[4]设为数组，例如：


protected $_auto = array(  
array(

   'title', 

   'autofill', 

   3,  //所有情况

   'callback',

   array('自定义的参数');//可以把此处设为数组，但只是感觉不太爽

));

第二种解决方案，就是修改源代码了，将


$args = isset($auto[4]) ? $auto[4] : array();
//改为
$args = isset($auto[4]) ? ( is_array($auto[4]) ? $auto[4] : array($auto[4])) :array();

[b][color=blue]总结[/color][/b]：
文章标题为什么说“疑是”呢，是因为通过第一种解决方案，这不能算是一个bug（当需要传递给处理方法或函数多个参数，$auto[4]必须用数组），但只传递一个参数时，往往会忘记或者说不习惯将$auto[4]设为数组类型。

通过第二种方案，能够更好在$auto[4]为任何赋值类型时进行处理。 :?