Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUGFIX] PHP 8.1 deprecation exceptions #449

Merged
merged 2 commits into from
Feb 21, 2024
Merged

Conversation

baschny
Copy link
Contributor

@baschny baschny commented Dec 5, 2023

Fix PHP 8.1 related exceptions:

1. When preparing the newsletter on "Step 5":

(1/1) #1476107295 TYPO3\CMS\Core\Error\Exception
PHP Runtime Deprecation Notice: Function strftime() is deprecated in /app/html/typo3conf/ext/direct_mail/Classes/Module/DmailController.php line 1428

See: https://php.watch/versions/8.1/strftime-gmstrftime-deprecated

For now, we just supress the warning, this has to be rewritten when you want to be compatible with PHP 9.x later on.

2. When running the queue processor:

Die Ausführung von Task "Direct Mail: Mailing Queue (direct_mail)" ist fehlgeschlagen mit folgender Meldung: PHP Runtime Deprecation Notice: htmlspecialchars(): Passing null to parameter #1 ($string) of type string is deprecated in /app/html/typo3conf/ext/direct_mail/Classes/Dmailer.php line 357

3. When editing a category in the backend:

PHP 8.1 warning:
Bei dem Versuch, die Elemente für Feld "Must subscribe to one of the categories:" (PHP Warning: Undefined array key "transOrigPointerTable" in /app/html/typo3conf/ext/direct_mail/Classes/Repository/TempRepository.php line 318) zu verarbeiten, ist ein Fehler aufgetreten.

The TCA configuration option transOrigPointerTable does not exist since TYPO3 v8 anymore. See: https://docs.typo3.org/c/typo3/cms-core/main/en-us/Changelog/8.5/Breaking-78191-RemoveSupportForTransForeignTableInTCA.html

@baschny baschny changed the title Fix PHP 8.1 deprecation exceptions [BUGFIX] PHP 8.1 deprecation exceptions Dec 5, 2023
@kartolo kartolo merged commit 6223ba5 into kartolo:master Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants