Not only local groups should be added to group inbox notificiation table

This commit is contained in:
Mikael Nordfeldth 2016-01-09 15:14:38 +01:00
parent 5ef10a14ef
commit 67801a5566
1 changed files with 4 additions and 5 deletions

View File

@ -1596,11 +1596,10 @@ class Notice extends Managed_DataObject
function saveAttention(Profile $target, $reason=null)
{
if ($target->isGroup()) {
// FIXME: Make sure we check that users are in the groups they send to!
if ($target->getGroup()->isLocal()) {
// legacy notification method, will still be in use for quite a while I think
$this->addToGroupInbox($target->getGroup());
}
// FIXME: Make sure we check (for both local and remote) users are in the groups they send to!
// legacy notification method, will still be in use for quite a while I think
$this->addToGroupInbox($target->getGroup());
} else {
if ($target->hasBlocked($this->getProfile())) {
common_log(LOG_INFO, "Not saving reply to profile {$target->id} ($uri) from sender {$sender->id} because of a block.");