JcDenis / alias

Dotclear plugin - Create aliases of your blog's URLs
GNU General Public License v2.0
0 stars 0 forks source link

Exception php 7.4 #2

Closed gibus closed 1 year ago

gibus commented 1 year ago

Salut,

En php 7.4, il y a une exception qui fait que les pages d'admin sont en erreur HTTP 500:

Parse error: syntax error, unexpected '|', expecting variable (T_VARIABLE) in /var/www/alternc/g/gibuskro/dotclear/plugins/alias/src/Backend.php on line 74

Visiblement la syntaxe function (bool|FlatBackupItem $line, FlatImportV2 $bk) n'est pas supporté en php 7, je n'ai aucune idée de ce à quoi elle sert, mais enlever bool| et de même ligne 84, résout le problème.

gibus commented 1 year ago

OK, il s'agit d'union type cf. https://php.watch/versions/8.0/union-types et donc oui, ce n'est pas supporté avant php 8. Je sais qu'il est prévu que le core dotclear exigera PHP 8.1 à la fin de l'année, mais d'ici là, il ne faudrait pas que les plugins cassent la compatibilité php 7. Suivant les hébergements, ce n'est pas toujours évident de passe à php 8…

JcDenis commented 1 year ago

Effectivement, à force de mélanger les versions, je suis allé un peu vite ici. Je corrige pour le laisser en php7.4 encore un peu.

gibus commented 1 year ago

Super, merci!