This is the correct way:

<?php
if (!isset($foo['account_removed) || $foo['account_removed'])
?>
This commit is contained in:
Roland Häder 2022-07-19 18:06:36 +02:00
parent 4e5179a7b3
commit cf8698afe6
No known key found for this signature in database
GPG key ID: C82EDE5DDFA0BA77
3 changed files with 3 additions and 3 deletions

View file

@ -65,7 +65,7 @@ function photos_init(App $a) {
if (DI::args()->getArgc() > 1) { if (DI::args()->getArgc() > 1) {
$owner = User::getOwnerDataByNick(DI::args()->getArgv()[1]); $owner = User::getOwnerDataByNick(DI::args()->getArgv()[1]);
if (empty($owner) || $owner['account_removed']) { if (!isset($owner['account_removed']) || $owner['account_removed']) {
throw new HTTPException\NotFoundException(DI::l10n()->t('User not found.')); throw new HTTPException\NotFoundException(DI::l10n()->t('User not found.'));
} }

View file

@ -310,7 +310,7 @@ HELP;
* @return bool True, if the delete was successful * @return bool True, if the delete was successful
* @throws \Exception * @throws \Exception
*/ */
private function deleteUser() private function deleteUser(): bool
{ {
$user = $this->getUserByNick(1); $user = $this->getUserByNick(1);

View file

@ -221,7 +221,7 @@ class Profile
public static function load(App $a, string $nickname, bool $show_contacts = true) public static function load(App $a, string $nickname, bool $show_contacts = true)
{ {
$profile = User::getOwnerDataByNick($nickname); $profile = User::getOwnerDataByNick($nickname);
if (empty($profile) || !isset($profile['account_removed']) || $profile['account_removed']) { if (!isset($profile['account_removed']) || $profile['account_removed']) {
Logger::info('profile error: ' . DI::args()->getQueryString()); Logger::info('profile error: ' . DI::args()->getQueryString());
return []; return [];
} }