Skip to content

Commit 44843d9

Browse files
authored
Merge pull request #17 from georgringer/patch-3
[BUGFIX] Fix notices in ReceiverMailReceiverPropertiesService
2 parents 07df997 + 5cbe571 commit 44843d9

File tree

1 file changed

+7
-7
lines changed

1 file changed

+7
-7
lines changed

Classes/Domain/Service/Mail/ReceiverMailReceiverPropertiesService.php

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -118,11 +118,11 @@ public function getReceiverName(): string
118118
protected function setReceiverEmails(): void
119119
{
120120
$emailArray = $this->getEmailsFromFlexForm();
121-
$emailArray = $this->getEmailsFromFeGroup($emailArray, (int)$this->settings['receiver']['fe_group']);
122-
$emailArray = $this->getEmailsFromBeGroup($emailArray, (int)$this->settings['receiver']['be_group']);
121+
$emailArray = $this->getEmailsFromFeGroup($emailArray, (int)($this->settings['receiver']['fe_group'] ?? 0));
122+
$emailArray = $this->getEmailsFromBeGroup($emailArray, (int)($this->settings['receiver']['be_group'] ?? 0));
123123
$emailArray = $this->getEmailsFromPredefinedEmail(
124124
$emailArray,
125-
(string)$this->settings['receiver']['predefinedemail']
125+
(string)($this->settings['receiver']['predefinedemail'] ?? '')
126126
);
127127
$emailArray = $this->overWriteEmailsWithTypoScript($emailArray);
128128
$emailArray = $this->getEmailFromDevelopmentContext($emailArray);
@@ -140,7 +140,7 @@ protected function setReceiverEmails(): void
140140
*/
141141
protected function getEmailsFromFlexForm(): array
142142
{
143-
if ((int)$this->settings['receiver']['type'] === self::RECEIVERS_DEFAULT) {
143+
if ((int)($this->settings['receiver']['type'] ?? 0) === self::RECEIVERS_DEFAULT) {
144144
$mailRepository = ObjectUtility::getObjectManager()->get(MailRepository::class);
145145
$emailString = TemplateUtility::fluidParseString(
146146
$this->settings['receiver']['email'],
@@ -161,7 +161,7 @@ protected function getEmailsFromFlexForm(): array
161161
*/
162162
protected function getEmailsFromFeGroup(array $emailArray, int $uid): array
163163
{
164-
if ((int)$this->settings['receiver']['type'] === self::RECEIVERS_FRONTENDGROUP && !empty($uid)) {
164+
if ((int)($this->settings['receiver']['type'] ?? 0) === self::RECEIVERS_FRONTENDGROUP && !empty($uid)) {
165165
$userRepository = ObjectUtility::getObjectManager()->get(UserRepository::class);
166166
$users = $userRepository->findByUsergroup($uid);
167167
$emailArray = [];
@@ -186,7 +186,7 @@ protected function getEmailsFromFeGroup(array $emailArray, int $uid): array
186186
*/
187187
protected function getEmailsFromBeGroup(array $emailArray, int $uid): array
188188
{
189-
if ((int)$this->settings['receiver']['type'] === self::RECEIVERS_BACKENDGROUP && !empty($uid)) {
189+
if ((int)($this->settings['receiver']['type'] ?? 0) === self::RECEIVERS_BACKENDGROUP && !empty($uid)) {
190190
/** @var BackendUserRepository $beUserRepository */
191191
$beUserRepository = ObjectUtility::getObjectManager()->get(BackendUserRepository::class);
192192
$query = $beUserRepository->createQuery();
@@ -217,7 +217,7 @@ protected function getEmailsFromBeGroup(array $emailArray, int $uid): array
217217
*/
218218
protected function getEmailsFromPredefinedEmail(array $emailArray, string $predefinedString): array
219219
{
220-
if ((int)$this->settings['receiver']['type'] === self::RECEIVERS_PREDEFINED && !empty($predefinedString)) {
220+
if ((int)($this->settings['receiver']['type'] ?? 0) === self::RECEIVERS_PREDEFINED && !empty($predefinedString)) {
221221
$receiverString = TypoScriptUtility::overwriteValueFromTypoScript(
222222
'',
223223
$this->configuration['receiver.']['predefinedReceiver.'][$predefinedString . '.'],

0 commit comments

Comments
 (0)