PHP 8.0 Compatibility - Githubissues

Project60 / org.project60.membership

CiviCRM extension to facilitate the "European" membership concept

GNU Affero General Public License v3.0

6 stars 14 forks source link

PHP 8.0 Compatibility #60

Closed abugey closed 11 months ago

abugey commented 1 year ago

[x] Analyse code regarding PHP 8 compatibility
[ ] Fix deprecations found (to be specified)

0spinboson commented 1 year ago

not compatible with php8, immediately errors out when I try to install it with php8 selected, installs fine after switching to 7.4.

bjendres commented 1 year ago

not compatible with php8, immediately errors out when I try to install it with php8 selected, installs fine after switching to 7.4.

Yes, thank you, that's what this ticket is about. @0spinboson would you be able to help with this, or do some testing?

abugey commented 1 year ago

Errors by inspect code feature of PHPStorm in PHP 8.0:

PaidByLogic.php: Zeile 723: Illegal array key type
SynchroniseLogic.php: Zeile 180: Illegal array key type

TobiasVoigt commented 1 year ago

PHP error log entry after upgrading to PHP8.1 (with v0.6.2):

PHP Fatal error: Array and string offset access syntax with curly braces is no longer supported in /html/wordpress/wp-content/uploads/civicrm/ext/org.project60.membership/membership.civix.php on line 246