Exception php 7.4 #2

Closed
opened 2023-06-14 21:20:58 +00:00 by gibus · 3 comments
gibus commented 2023-06-14 21:20:58 +00:00 (Migrated from github.com)

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.

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 2023-06-14 21:39:00 +00:00 (Migrated from github.com)

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…

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 2023-06-16 18:49:04 +00:00 (Migrated from github.com)

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.

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 2023-06-16 21:57:44 +00:00 (Migrated from github.com)

Super, merci!

Super, merci!
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: JcDenis/alias#2
No description provided.