merge

application/Espo/Classes/Acl/Attachment/AccessChecker.php

@@ -48,17 +48,12 @@ class AccessChecker implements AccessEntityCREDChecker
 {
     use DefaultAccessCheckerDependency;
 
-    private AclManager $aclManager;
-    private EntityManager $entityManager;
-
     public function __construct(
         DefaultAccessChecker $defaultAccessChecker,
-        AclManager $aclManager,
+        private AclManager $aclManager,
-        EntityManager $entityManager
+        private EntityManager $entityManager
     ) {
         $this->defaultAccessChecker = $defaultAccessChecker;
-        $this->aclManager = $aclManager;
-        $this->entityManager = $entityManager;
     }
 
     public function checkEntityRead(User $user, Entity $entity, ScopeData $data): bool
@@ -150,6 +145,10 @@ class AccessChecker implements AccessEntityCREDChecker
             return null;
         }
 
+        if ($note->getTargetType() === Note::TARGET_ALL) {
+            return true;
+        }
+
         if (!$note->getParentId() || !$note->getParentType()) {
             return null;
         }

application/Espo/Core/Notification/DefaultAssignmentNotificator.php

@@ -56,12 +56,8 @@ class DefaultAssignmentNotificator implements AssignmentNotificator
         if ($entity->hasLinkMultipleField('assignedUsers')) {
             /** @var string[] $userIdList */
             $userIdList = $entity->getLinkMultipleIdList('assignedUsers');
-            /** @var ?string[] $fetchedAssignedUserIdList */
+            /** @var string[] $fetchedAssignedUserIdList */
-            $fetchedAssignedUserIdList = $entity->getFetched('assignedUsersIds');
+            $fetchedAssignedUserIdList = $entity->getFetched('assignedUsersIds') ?? [];
-
-            if (!is_array($fetchedAssignedUserIdList)) {
-                $fetchedAssignedUserIdList = [];
-            }
 
             foreach ($userIdList as $userId) {
                 if (in_array($userId, $fetchedAssignedUserIdList)) {

application/Espo/Tools/EmailNotification/HookProcessor.php

@@ -89,7 +89,7 @@ class HookProcessor
     private function processMultiple(CoreEntity $entity): void
     {
         $userIdList = $entity->getLinkMultipleIdList('assignedUsers');
-        $fetchedAssignedUserIdList = $entity->getFetched('assignedUsersIds');
+        $fetchedAssignedUserIdList = $entity->getFetched('assignedUsersIds') ?? [];
 
         foreach ($userIdList as $userId) {
             if (

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "espocrm",
-  "version": "7.5.0",
+  "version": "7.5.1",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "espocrm",
-      "version": "7.5.0",
+      "version": "7.5.1",
       "hasInstallScript": true,
       "license": "GPL-3.0",
       "dependencies": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "espocrm",
-  "version": "7.5.0",
+  "version": "7.5.1",
   "description": "Open-source CRM.",
   "repository": {
     "type": "git",

upgrades/7.5/scripts/AfterUpgrade.php

@@ -61,7 +61,7 @@ class AfterUpgrade
                 continue;
             }
 
-            $data1 = $metadata->getCustom('selectDefs', $entityType);
+            $data1 = $metadata->getCustom('selectDefs', $entityType) ?? (object) [];
             $data1->primaryFilterClassNameMap = $primaryFilterClassNameMap;
 
             $metadata->saveCustom('selectDefs', $entityType, $data1);