diff --git a/application/Espo/Classes/Acl/Attachment/AccessChecker.php b/application/Espo/Classes/Acl/Attachment/AccessChecker.php index cdcc0cac8b..76f6c8f1e0 100644 --- a/application/Espo/Classes/Acl/Attachment/AccessChecker.php +++ b/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, - EntityManager $entityManager + private AclManager $aclManager, + 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; } diff --git a/application/Espo/Core/Notification/DefaultAssignmentNotificator.php b/application/Espo/Core/Notification/DefaultAssignmentNotificator.php index 865db01cdf..bd5ddff48c 100644 --- a/application/Espo/Core/Notification/DefaultAssignmentNotificator.php +++ b/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 */ - $fetchedAssignedUserIdList = $entity->getFetched('assignedUsersIds'); - - if (!is_array($fetchedAssignedUserIdList)) { - $fetchedAssignedUserIdList = []; - } + /** @var string[] $fetchedAssignedUserIdList */ + $fetchedAssignedUserIdList = $entity->getFetched('assignedUsersIds') ?? []; foreach ($userIdList as $userId) { if (in_array($userId, $fetchedAssignedUserIdList)) { diff --git a/application/Espo/Tools/EmailNotification/HookProcessor.php b/application/Espo/Tools/EmailNotification/HookProcessor.php index d2297fe2df..011bb816b0 100644 --- a/application/Espo/Tools/EmailNotification/HookProcessor.php +++ b/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 ( diff --git a/package-lock.json b/package-lock.json index 68f40e72b9..cd590c039f 100644 --- a/package-lock.json +++ b/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": { diff --git a/package.json b/package.json index e93abead7a..ef36960d58 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "espocrm", - "version": "7.5.0", + "version": "7.5.1", "description": "Open-source CRM.", "repository": { "type": "git", diff --git a/upgrades/7.5/scripts/AfterUpgrade.php b/upgrades/7.5/scripts/AfterUpgrade.php index d86d477dbd..49e0cea4a2 100644 --- a/upgrades/7.5/scripts/AfterUpgrade.php +++ b/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);