unfollow portal users

application/Espo/Modules/Crm/Hooks/CaseObj/Contacts.php

@@ -34,7 +34,6 @@ use Espo\Core\Hook\Hook\AfterSave;
 use Espo\Core\InjectableFactory;
 use Espo\Entities\User;
 use Espo\Modules\Crm\Entities\CaseObj;
-use Espo\Modules\Crm\Entities\Contact;
 use Espo\ORM\Entity;
 use Espo\ORM\EntityManager;
 use Espo\ORM\Name\Attribute;
@@ -48,6 +47,9 @@ class Contacts implements AfterSave
 {
     private ?StreamService $streamService = null;
 
+    private const ATTR_CONTACT_ID = 'contactId';
+    private const RELATION_CONTACTS = 'contacts';
+
     public function __construct(
         private EntityManager $entityManager,
         private InjectableFactory $injectableFactory,
@@ -59,23 +61,20 @@ class Contacts implements AfterSave
      */
     public function afterSave(Entity $entity, SaveOptions $options): void
     {
-        if (!$entity->isAttributeChanged('contactId')) {
+        if (!$entity->isAttributeChanged(self::ATTR_CONTACT_ID)) {
             return;
         }
 
-        /** @var ?string $contactId */
-        $contactId = $entity->get('contactId');
-        $contactIdList = $entity->get('contactsIds') ?? [];
-        /** @var ?string $fetchedContactId */
-        $fetchedContactId = $entity->getFetched('contactId');
+        $contact = $entity->getContact();
 
-        $relation = $this->entityManager
-            ->getRDBRepositoryByClass(CaseObj::class)
-            ->getRelation($entity, 'contacts');
+        /** @var ?string $fetchedContactId */
+        $fetchedContactId = $entity->getFetched(self::ATTR_CONTACT_ID);
+
+        $contactsRelation = $this->entityManager->getRelation($entity, self::RELATION_CONTACTS);
 
         if ($fetchedContactId) {
             $previousPortalUser = $this->entityManager
-                ->getRDBRepository(User::ENTITY_TYPE)
+                ->getRDBRepositoryByClass(User::class)
                 ->select([Attribute::ID])
                 ->where([
                     'contactId' => $fetchedContactId,
@@ -90,25 +89,17 @@ class Contacts implements AfterSave
             }
         }
 
-        if (!$contactId && $fetchedContactId) {
-            $relation->unrelateById($fetchedContactId);
+        if (!$contact && $fetchedContactId) {
+            $contactsRelation->unrelateById($fetchedContactId);
 
             return;
         }
 
-        if (!$contactId) {
+        if (!$contact) {
             return;
         }
 
-        $portalUser = $this->entityManager
-            ->getRDBRepository(User::ENTITY_TYPE)
-            ->select([Attribute::ID])
-            ->where([
-                'contactId' => $contactId,
-                'type' => User::TYPE_PORTAL,
-                'isActive' => true,
-            ])
-            ->findOne();
+        $portalUser = $this->getPortalUser($contact->getId());
 
         if ($portalUser) {
             // @todo Solve ACL check issue when a user is in multiple portals.
@@ -119,21 +110,15 @@ class Contacts implements AfterSave
             }
         }
 
-        if (in_array($contactId, $contactIdList)) {
+        if (in_array($contact->getId(), $entity->getContacts()->getIdList())) {
             return;
         }
 
-        $contact = $this->entityManager->getEntityById(Contact::ENTITY_TYPE, $contactId);
-
-        if (!$contact) {
+        if ($contactsRelation->isRelatedById($contact->getId())) {
             return;
         }
 
-        if ($relation->isRelated($contact)) {
-            return;
-        }
-
-        $relation->relateById($contactId);
+        $contactsRelation->relateById($contact->getId());
     }
 
     private function getStreamService(): StreamService
@@ -144,4 +129,17 @@ class Contacts implements AfterSave
 
         return $this->streamService;
     }
+
+    private function getPortalUser(?string $contactId): ?User
+    {
+        return $this->entityManager
+            ->getRDBRepositoryByClass(User::class)
+            ->select([Attribute::ID])
+            ->where([
+                'contactId' => $contactId,
+                'type' => User::TYPE_PORTAL,
+                'isActive' => true,
+            ])
+            ->findOne();
+    }
 }

application/Espo/Modules/Crm/Hooks/CaseObj/IsInternal.php

@@ -0,0 +1,68 @@
+<?php
+/************************************************************************
+ * This file is part of EspoCRM.
+ *
+ * EspoCRM – Open Source CRM application.
+ * Copyright (C) 2014-2024 Yurii Kuznietsov, Taras Machyshyn, Oleksii Avramenko
+ * Website: https://www.espocrm.com
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program. If not, see <https://www.gnu.org/licenses/>.
+ *
+ * The interactive user interfaces in modified source and object code versions
+ * of this program must display Appropriate Legal Notices, as required under
+ * Section 5 of the GNU Affero General Public License version 3.
+ *
+ * In accordance with Section 7(b) of the GNU Affero General Public License version 3,
+ * these Appropriate Legal Notices must retain the display of the "EspoCRM" word.
+ ************************************************************************/
+
+namespace Espo\Modules\Crm\Hooks\CaseObj;
+
+use Espo\Core\FieldProcessing\Stream\FollowersLoader;
+use Espo\Core\Hook\Hook\AfterSave;
+use Espo\Modules\Crm\Entities\CaseObj;
+use Espo\ORM\Entity;
+use Espo\ORM\Repository\Option\SaveOptions;
+use Espo\Services\Stream;
+
+
+/**
+ * @implements AfterSave<CaseObj>
+ */
+class IsInternal implements AfterSave
+{
+    public function __construct(
+        private Stream $streamService,
+        private FollowersLoader $followersLoader,
+    ) {}
+
+    /**
+     * @param CaseObj $entity
+     */
+    public function afterSave(Entity $entity, SaveOptions $options): void
+    {
+        $this->processUnfollowPortalUsers($entity);
+    }
+
+    private function processUnfollowPortalUsers(CaseObj $entity): void
+    {
+        if (!$entity->isInternal() || $entity->isNew()) {
+            return;
+        }
+
+        $this->streamService->unfollowPortalUsersFromEntity($entity);
+
+        $this->followersLoader->processFollowers($entity);
+    }
+}

application/Espo/Tools/Stream/Service.php

@@ -38,6 +38,9 @@ use Espo\Core\ORM\Type\FieldType;
 use Espo\Entities\StreamSubscription;
 use Espo\Modules\Crm\Entities\Account;
 use Espo\ORM\Name\Attribute;
+use Espo\ORM\Query\Part\Condition;
+use Espo\ORM\Query\Part\Expression;
+use Espo\ORM\Query\SelectBuilder;
 use Espo\Repositories\EmailAddress as EmailAddressRepository;
 
 use Espo\ORM\Query\Part\Expression as Expr;
@@ -358,6 +361,39 @@ class Service
         $this->entityManager->getQueryExecutor()->execute($delete);
     }
 
+    /**
+     * Unfollow all portal users from an entity.
+     *
+     * @since 9.0.0
+     */
+    public function unfollowPortalUsersFromEntity(Entity $entity): void
+    {
+        if (!$entity->hasId()) {
+            return;
+        }
+
+        $delete = $this->entityManager->getQueryBuilder()
+            ->delete()
+            ->from(StreamSubscription::ENTITY_TYPE)
+            ->where([
+                'entityId' => $entity->getId(),
+                'entityType' => $entity->getEntityType(),
+            ])
+            ->where(
+                Condition::in(
+                    Expression::column('userId'),
+                    SelectBuilder::create()
+                        ->from(User::ENTITY_TYPE)
+                        ->select('id')
+                        ->where(['type' => User::TYPE_PORTAL])
+                        ->build()
+                )
+            )
+            ->build();
+
+        $this->entityManager->getQueryExecutor()->execute($delete);
+    }
+
     private function loadAssignedUserName(Entity $entity): void
     {
         $user = $this->entityManager