Changes:
- ops, fixed variable name (thanks to @annando) - also used empty() for an empty array (and false/null)
This commit is contained in:
parent
c35e11e01f
commit
4b5ae32ec6
1 changed files with 3 additions and 3 deletions
|
@ -4020,12 +4020,12 @@ class Diaspora
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!$recipients) {
|
if (empty($recipients)) {
|
||||||
Logger::debug('No recipients provided, fetching for user', ['uid' => $uid]);
|
Logger::debug('No recipients provided, fetching for user', ['uid' => $uid]);
|
||||||
$recipients = DBA::selectToArray('contact', [], ['network' => Protocol::DIASPORA, 'uid' => $uid, 'rel' => [Contact::FOLLOWER, Contact::FRIEND]]);
|
$recipients = DBA::selectToArray('contact', [], ['network' => Protocol::DIASPORA, 'uid' => $uid, 'rel' => [Contact::FOLLOWER, Contact::FRIEND]]);
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!$recipients) {
|
if (empty($recipients)) {
|
||||||
Logger::warning('Cannot fetch recipients', ['uid' => $uid]);
|
Logger::warning('Cannot fetch recipients', ['uid' => $uid]);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
@ -4033,7 +4033,7 @@ class Diaspora
|
||||||
$message = self::createProfileData($uid);
|
$message = self::createProfileData($uid);
|
||||||
|
|
||||||
// @todo Split this into single worker jobs
|
// @todo Split this into single worker jobs
|
||||||
foreach ($recipientients as $recipient) {
|
foreach ($recipients as $recipient) {
|
||||||
Logger::info('Send updated profile data for user ' . $uid . ' to contact ' . $recipient['id']);
|
Logger::info('Send updated profile data for user ' . $uid . ' to contact ' . $recipient['id']);
|
||||||
self::buildAndTransmit($owner, $recipient, 'profile', $message);
|
self::buildAndTransmit($owner, $recipient, 'profile', $message);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue