对于不能添加教练的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 https://github.com/ekszz/OurAcm/issues/2#issuecomment-44003535.

[ekszz]

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

[angela0]

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

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

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

— Reply to this email directly or view it on GitHub https://github.com/ekszz/OurAcm/issues/2#issuecomment-44015286.

[ekszz]

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