Check return values with empty()
This commit is contained in:
parent
ac4f62f556
commit
023fb38a75
1 changed files with 3 additions and 3 deletions
|
@ -178,7 +178,7 @@ HELP;
|
||||||
$nick = $this->getNick($arg_index);
|
$nick = $this->getNick($arg_index);
|
||||||
|
|
||||||
$user = UserModel::getByNickname($nick, ['uid']);
|
$user = UserModel::getByNickname($nick, ['uid']);
|
||||||
if (!$user) {
|
if (empty($user)) {
|
||||||
throw new RuntimeException($this->l10n->t('User not found'));
|
throw new RuntimeException($this->l10n->t('User not found'));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -206,7 +206,7 @@ HELP;
|
||||||
try {
|
try {
|
||||||
$result = UserModel::updatePassword($user['uid'], $password);
|
$result = UserModel::updatePassword($user['uid'], $password);
|
||||||
|
|
||||||
if (!$result) {
|
if (empty($result)) {
|
||||||
throw new \Exception($this->l10n->t('Password update failed. Please try again.'));
|
throw new \Exception($this->l10n->t('Password update failed. Please try again.'));
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -425,7 +425,7 @@ HELP;
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
if ($user) {
|
if (!empty($user)) {
|
||||||
$table->addRow($user);
|
$table->addRow($user);
|
||||||
}
|
}
|
||||||
$this->out($table->getTable());
|
$this->out($table->getTable());
|
||||||
|
|
Loading…
Reference in a new issue