对于不能添加教练的bug

[angela0]

在增加和修改队员的函数中
$tmp = intval(I('post.group'));
if($tmp == 0 || $tmp == 1 || $tmp == 9) $data['group'] = $tmp;
else $tmp = 0;
这个代码这样写不能添加教练, 既然分组是通过下拉选项选择的,肯定只有0 1 2 9 这4个数,那$tmp不是只能有这几种可能吗,为什么不能直接写成 $data['group'] = $tmp; ?

[ekszz]

确认BUG，感谢反馈。
不过用户的输入是不确定的，并不能保证输入值为0 1 2 9，可以任意的，因此后台还是需要进行过滤。
这边漏掉了教练组，待修复。

[angela0]

我对这段代码里面的
else $tmp = 0;
表示疑惑，这个是做什么用的

On 05/23/2014 08:36 PM, Ekszz wrote:

确认BUG，感谢反馈。
不过用户的输入是不确定的，并不能保证输入值为0 1 2 9，可以任意的，因此
后台还是需要进行过滤。
这边漏掉了教练组，待修复。

—
Reply to this email directly or view it on GitHub
#2 (comment).

[ekszz]

过滤用户输入的脏数据的。

[angela0]

但是，$tmp这个变量以后没有再用，没有这个操作应该这是可以的吧

On 05/23/2014 10:25 PM, Ekszz wrote:

过滤用户输入的脏数据的。

—
Reply to this email directly or view it on GitHub
#2 (comment).

[ekszz]

这个应该是BUG，应该为$data['group'] = 0，已提交修复代码。