systopia / de.systopia.mailingtools

CiviCRM Extension to support mailings
GNU Affero General Public License v3.0
1 stars 2 forks source link

PHP 8.0 Compatibility #9

Closed abugey closed 1 year ago

abugey commented 2 years ago
abugey commented 1 year ago

no deprecations found

jofranz commented 1 year ago

Deprecated function: Array and string offset access syntax with curly braces is deprecated in require_once() (line 3 of sites/default/files/civicrm/ext/de.systopia.mailingtools/mailingtools.php).

jofranz commented 1 year ago

FILE: CRM/Mailingtools/Form/Settings.php

---------------------------------------------------------------------------------------------------------------------
FOUND 2 ERRORS AFFECTING 2 LINES
---------------------------------------------------------------------------------------------------------------------
 251 | ERROR | The default value of the $flags parameter for html_entity_decode() was changed from ENT_COMPAT to
     |       | ENT_QUOTES | ENT_SUBSTITUTE | ENT_HTML401 in PHP 8.1. For cross-version compatibility, the $flags
     |       | parameter should be explicitly set.
 253 | ERROR | The default value of the $flags parameter for html_entity_decode() was changed from ENT_COMPAT to
     |       | ENT_QUOTES | ENT_SUBSTITUTE | ENT_HTML401 in PHP 8.1. For cross-version compatibility, the $flags
     |       | parameter should be explicitly set.
---------------------------------------------------------------------------------------------------------------------
pbatroff commented 1 year ago

This is about PHP 8.0 compatibility and should be moved to a separate ticket, though it looks like it can be fixed esaily

jofranz commented 1 year ago

Ref. 19735