Project

General

Profile

Actions

Bug #97463

closed

Administration Log increased with Warnings form EmailFinisher.php -L337

Added by Michael Hilse about 2 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Logging
Target version:
Start date:
2022-04-24
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
11
PHP Version:
7.4
Tags:
Complexity:
no-brainer
Is Regression:
Sprint Focus:

Description

The Administration Log gets entries like:

Core: Error handler (FE): PHP Warning: Invalid argument supplied for foreach() in .../typo3/sysext/form/Classes/Domain/Finishers/EmailFinisher.php line 337

Problem in Line 337 of EmailFinisher.php:
L337: foreach ($recipients as $address => $name) {

If $recipients is not an array the warning occurs!
So we must get sure, that $recipients is cast to an array like the following code:

Solution-Proposal: (tested and worked) replace L337 with the following code

L337: foreach ( (array) $recipients as $address => $name) {

Good Luck

Actions #1

Updated by Gerrit Code Review about 2 years ago

  • Status changed from New to Under Review

Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74387

Actions #2

Updated by Gerrit Code Review about 2 years ago

Patch set 1 for branch 11.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/74396

Actions #3

Updated by Oliver Bartsch about 2 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #4

Updated by Oliver Hader over 1 year ago

  • Related to Bug #98178: Form EmailFinisher overrides for recipients with TypoScript not possible added
Actions #5

Updated by Oliver Hader over 1 year ago

  • Related to deleted (Bug #98178: Form EmailFinisher overrides for recipients with TypoScript not possible)
Actions #6

Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF