Unverified Commit cea8d790 authored by Côme Chilliet's avatar Côme Chilliet
Browse files

:ambulance: fix(departements) Fix problems in departmentManagement

This was broken since migration from simpleManagement.
Spotted thanks to PHPStan

issue #6114
Showing with 5 additions and 5 deletions
+5 -5
......@@ -58,17 +58,17 @@ class departmentManagement extends management
}
// Overridden remove request method - Avoid removal of the ldap base.
protected function removeEntryRequested ($action, array $target, array $all)
public function removeRequested (array $action)
{
global $config;
$target = array_remove_entries([$config->current['BASE']], $target);
return parent::removeEntryRequested($action, $target, $all);
$action['targets'] = array_remove_entries([$config->current['BASE']], $action['targets']);
return parent::removeRequested($action);
}
// Finally remove departments and update department browsers
function removeEntryConfirmed ($action, array $target, array $all)
public function removeConfirmed (array $action)
{
parent::removeEntryConfirmed($action, $target, $all);
parent::removeConfirmed($action);
$this->refreshDeps();
}
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment