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

Exception php 7.4 #2

Closed
gibus opened this issue Jun 14, 2023 · 3 comments
Closed

Exception php 7.4 #2

gibus opened this issue Jun 14, 2023 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@gibus
Copy link

gibus commented Jun 14, 2023

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
Copy link
Author

gibus commented Jun 14, 2023

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
Copy link
Owner

JcDenis commented Jun 16, 2023

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.

@JcDenis JcDenis self-assigned this Jun 16, 2023
@JcDenis JcDenis added the bug Something isn't working label Jun 16, 2023
@JcDenis JcDenis pinned this issue Jun 16, 2023
@JcDenis JcDenis unpinned this issue Jun 16, 2023
@JcDenis JcDenis closed this as completed Jun 16, 2023
@gibus
Copy link
Author

gibus commented Jun 16, 2023

Super, merci!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants