The variable $typeconfig is never passed, but is used in the function. The only reason it doesn't break is because we use !isset($typeconfig['allowinstructorcustom']) and isset() won't complain. This was discovered reviewing MDL-60416 which will address this issue in 3.5 (assuming it gets integrated), but this still needs fixing in prior supported branches.