Passed
Push — master ( c0a3a7...3b84a4 )
by Jeroen
58:51
created

actions/uservalidationbyemail/delete.php (1 issue)

1
<?php
2
/**
3
 * Delete a user or users by guid
4
 */
5
6
$user_guids = get_input('user_guids');
7
$error = false;
8
9
if (!$user_guids) {
10
	return elgg_error_response(elgg_echo('uservalidationbyemail:errors:unknown_users'));
11
}
12
13
$access = access_show_hidden_entities(true);
14
15
foreach ($user_guids as $guid) {
16
	$user = get_entity($guid);
17
	if (!$user instanceof ElggUser) {
18
		$error = true;
19
		continue;
20
	}
21
22
	// don't delete validated users
23
	if ($user->isValidated() !== false || !$user->delete()) {
24
		$error = true;
25
		continue;
26
	}
27
}
28
29
access_show_hidden_entities($access);
30
31
if (count($user_guids) == 1) {
0 ignored issues
show
It seems like $user_guids can also be of type string; however, parameter $var of count() does only seem to accept Countable|array, maybe add an additional type check? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-type  annotation

31
if (count(/** @scrutinizer ignore-type */ $user_guids) == 1) {
Loading history...
32
	$message_txt = elgg_echo('uservalidationbyemail:messages:deleted_user');
33
	$error_txt = elgg_echo('uservalidationbyemail:errors:could_not_delete_user');
34
} else {
35
	$message_txt = elgg_echo('uservalidationbyemail:messages:deleted_users');
36
	$error_txt = elgg_echo('uservalidationbyemail:errors:could_not_delete_users');
37
}
38
39
if ($error) {
40
	return elgg_error_response($error_txt);
41
}
42
43
return elgg_ok_response('', $message_txt);
44