Project

General

Profile

Anomalie #4194

Rendre spip_loader compatible php 7.2

Added by Franck D about 1 year ago. Updated about 2 months ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
10/17/2018
Due date:
% Done:

0%

Resolution:
fixed
Navigateur:

Description

Hello :-)
En faisant une série de test https://www.mail-archive.com/spip-zone@rezo.net/msg46352.html
Cela m'a fait découvrir que spip_loader avait dans 2 cas chez ovh des problèmes (donc un grave).

Pour info, pour le voir, il faut obligatoirement faire une nouvelle installation ! Car sans quoi spip_loader fonctionne si un spip est déjà en place !
Dans le premier cas, spip_loader ne fonctionne pas, dans le deuxième, l'installation fonctionne quand même !

Environnement : stable
Version PHP 7.2
Moteur:phpcgi
Mode: développement

Warning: Use of undefined constant _DIR_TMP - assumed '_DIR_TMP' (this will throw an Error in a future version of PHP) in /.../pclzip.php on line 28
Après avoir fait un clique sur "Commencer l'installation"

Warning: Use of undefined constant _DIR_TMP - assumed '_DIR_TMP' (this will throw an Error in a future version of PHP) in /.../pclzip.php on line 28
Warning: Cannot modify header information - headers already sent by (output started at /.../pclzip.php:28) in /.../spip_loader.php on line 1228

Environnement : stable
Version PHP 7.2
Moteur:php
Mode: développement

Warning: Use of undefined constant _DIR_TMP - assumed '_DIR_TMP' (this will throw an Error in a future version of PHP) in /.../pclzip.php on line 28
Après avoir fait un clique sur "Commencer l'installation"

Warning: Use of undefined constant _DIR_TMP - assumed '_DIR_TMP' (this will throw an Error in a future version of PHP) in /.../pclzip.php on line 28

Franck

History

#1 Updated by Franck D about 1 year ago

Pour info, les tests ont été fait en spip_loader 3.0.5 et concernant spip j'avais fait les tests pour les installations neuve en spip 3.2

#2 Updated by Vincent ROBERT 4 months ago

Je confirme le bug que j'ai signalé en double ici #4360

#3 Updated by Franck D about 2 months ago

Hello :)
Juste pour dire que j'ai fait un test avec laragon en php 7.3 et 7.4, cela semble bon sauf le warning sur pclzip dit plus haut

Il y a une autre warning qui apparait:
warning: preg_match(): Compilation failed: invalid range in character class at offset 4 in c:\...\spip_loader.php on line 1329

mais dont la correction est simple via le super log que marcimat avait fait sur un warning identique dans svp https://zone.spip.net/trac/spip-zone/changeset/112275/spip-zone

Il faudrait mettre:
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_outils_/spip_loader/trunk/spip_loader.php#L1329
if (isset($_REQUEST['dest']) and preg_match('/^[\w_.-]+$/', $_REQUEST['dest'])) {

J'ai essayer, et cela fait bien disparaitre le warning, reste le cas de pclzip.php ...
https://zone.spip.net/trac/spip-zone/browser/spip-zone/_outils_/spip_loader/trunk/pclzip.php#L28
Il y a une proposition de commit dans ce ticket:
https://core.spip.net/issues/4384
Reste à voir si cela est sufisant

#4 Updated by marcimat 🌈 about 2 months ago

Ah bah j’ai du corriger celui là en passant, avec https://zone.spip.net/trac/spip-zone/changeset/117928 pour #4384

#5 Updated by Ben . about 2 months ago

  • Status changed from Nouveau to Fermé

#6 Updated by b b about 2 months ago

  • Resolution set to fixed

Also available in: Atom PDF