What would be a cleaner way to write the following?
// Check if add user was attempted
if ($params['submit']) {
// Verify all fields are filled in
if (Utilities::checkAllFieldsNotEmpty($params)) {
// Make sure username is not taken
if (Admin::checkUsername($params['username'])) {
// Check if directory is created
if (Admin::createUsersDirectory($params['username'])) {
if (Admin::createNewUser($params['name'], $params['username'], $params['password'], $params['admin'])) {
Utilities::setMessage("Excellent!", "User was created successfully.", "admin_modal");
}
} else {
Utilities::setMessage("Whoa!", "Directory creation failed.", "admin_modal");
}
} else {
Utilities::setMessage("Whoa!", "Username in use.", "admin_modal");
}
} else {
Utilities::setMessage("Whoa!", "Please fill in all fields", "admin_modal");
}
// Return admin view with message
return $this->view->render($response, 'admin/admin.twig', [
'name' => $user['name'],
'message' => $_SESSION['message'],
'form' => [
'name' => $params['name'],
'username' => $params['username'],
'admin' => $params['admin']
]
]);
}