A veces sucede que se llama a AdminCategory::deletecategory($parentId) sin un parámetro, pero el prototipo no tiene un valor predeterminado y, por lo tanto, se lanza una excepción. Dado que obtiene datos de una solicitud de publicación y siempre existe la posibilidad de que una categoría no tenga un padre, puede refactorizar su método para que se vea así:
function deletecategory($parentId = null)
{
$ids = $_POST['id'];
$this->model->deletecategory($ids);
if (null !== $parentId) {
header('location:'.URL.'admincategory/showchildren/'.$parentId);
}
// PUT MORE OF YOUR LOGIC HERE, I DO NOT KNOW WHAT SHOULD HAPPEN
}
Si está utilizando sugerencias de escritura, sería más apropiado hacer que el método se vea como
function deletecategory(string $parentId = ''): void //void is for php7.1
{
$ids = $_POST['id'];
$this->model->deletecategory($ids);
if ('' !== $parentId) {
header('location:'.URL.'admincategory/showchildren/'.$parentId);
}
// AGAIN LOGIC HERE
}
Si REALMENTE espera que se DEBE pasar parentId, en su lugar envuelva la persona que llama al método con try catch
if (method_exists($object, $this->method)) {
try {
call_user_func_array([$object, $this->method], $this->params);
} catch (\Exception $ex) {
// HANDLE EXCEPTION HERE
}
}