Declaration of app\admin\model\Group::delDataById($group_id, $delSon = false) should be compatible with app\admin\model\Common::delDataById($id = '', $delSon = false) #11

hzdok · 2019-05-20T07:04:56Z

https://github.com/72crm/72crm/blob/58d446279867ab3f5f171fc2173bbd5fa438cfbb/application/admin/model/Group.php#L106

PHP7.2.10 下delDataById方法重写，由于参数不同报错（php5.6没问题）
源代码位置：/application/admin/model/Group.php#L106
源代码：public function delDataById($group_id)
建议修改为：public function delDataById($group_id= '', $delSon = false)

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Declaration of app\admin\model\Group::delDataById($group_id, $delSon = false) should be compatible with app\admin\model\Common::delDataById($id = '', $delSon = false) #11

Declaration of app\admin\model\Group::delDataById($group_id, $delSon = false) should be compatible with app\admin\model\Common::delDataById($id = '', $delSon = false) #11

hzdok commented May 20, 2019

Declaration of app\admin\model\Group::delDataById($group_id, $delSon = false) should be compatible with app\admin\model\Common::delDataById($id = '', $delSon = false) #11

Declaration of app\admin\model\Group::delDataById($group_id, $delSon = false) should be compatible with app\admin\model\Common::delDataById($id = '', $delSon = false) #11

Comments

hzdok commented May 20, 2019