Deprecated php 8.1.1 sur SPIP-dev #36

Closed
opened 6 months ago by Jack31 · 4 comments
Jack31 commented 6 months ago
Collaborator

J'ai ce deprecated en php 8.1.1 sur SPIP-dev

Deprecated: Automatic conversion of false to array is deprecated in /home/jack31/www_rumeur/plugins/auto/gis/v4.52.0/gis_pipelines.php on line 32

J'ai ce deprecated en php 8.1.1 sur SPIP-dev > Deprecated: Automatic conversion of false to array is deprecated in /home/jack31/www_rumeur/plugins/auto/gis/v4.52.0/gis_pipelines.php on line 32
Collaborator

Tu l'obtiens sur quelle page, dans quel contexte ?

Ha, je devine, genre à l'install quand il n'y a pas encore de conf ?

Tu l'obtiens sur quelle page, dans quel contexte ? Ha, je devine, genre à l'install quand il n'y a pas encore de conf ?
Collaborator

Tu peux teser avec ce patch stp et me dire si ça fixe le bug ?

diff --git a/gis_pipelines.php b/gis_pipelines.php
index 146cba8..4781c85 100755
--- a/gis_pipelines.php
+++ b/gis_pipelines.php
@@ -27,7 +27,8 @@ function gis_insert_head_css($flux) {
 function gis_insert_head($flux) {
 
 	// initialisation des valeurs de config
-	$config = @unserialize($GLOBALS['meta']['gis']);
+	include_spip('inc/config');
+	$config = lire_config('gis', []);
 	if (!isset($config['layers']) || !is_array($config['layers'])) {
 		$config['layers'] = ['openstreetmap_mapnik'];
 	}
Tu peux teser avec ce patch stp et me dire si ça fixe le bug ? ```diff diff --git a/gis_pipelines.php b/gis_pipelines.php index 146cba8..4781c85 100755 --- a/gis_pipelines.php +++ b/gis_pipelines.php @@ -27,7 +27,8 @@ function gis_insert_head_css($flux) { function gis_insert_head($flux) { // initialisation des valeurs de config - $config = @unserialize($GLOBALS['meta']['gis']); + include_spip('inc/config'); + $config = lire_config('gis', []); if (!isset($config['layers']) || !is_array($config['layers'])) { $config['layers'] = ['openstreetmap_mapnik']; } ```
b_b added the
amélioration
label 6 months ago
Poster
Collaborator

Alors quelques essais ce matin :

  • La ligne include_spip('inc/config'); génère une erreur sur la page d'accueil
Warning: Undefined variable $include_spip in /home/jack31/www_rumeur/plugins/auto/gis/v4.52.0/gis_pipelines.php on line 30
Erreur d’exécution ../prive/squelettes/head/dist.html
  • en ne laissant que $config = lire_config('gis', []); les warnings disparaissent

Du coup je suis retourné configurer de GIS, config qui avait sautée où n'avait pas été traitée :

  • en remettant gis_pipelines.php dans son état initial après config, plus de warning...
  • si je remets include_spip('inc/config'); je retrouve l'erreur d'éxécution

Donc c'est bien en ayant installé GIS sans l'avoir configuré qui génère l'erreur, j'ai dû faire des essais puis désinstaller GIS puisque je n'ai plus non plus aucun point géolocalisés...

Alors quelques essais ce matin : * La ligne include_spip('inc/config'); génère une erreur sur la page d'accueil ``` Warning: Undefined variable $include_spip in /home/jack31/www_rumeur/plugins/auto/gis/v4.52.0/gis_pipelines.php on line 30 Erreur d’exécution ../prive/squelettes/head/dist.html ``` * en ne laissant que $config = lire_config('gis', []); les warnings disparaissent Du coup je suis retourné configurer de GIS, config qui avait sautée où n'avait pas été traitée : * en remettant gis_pipelines.php dans son état initial après config, plus de warning... * si je remets include_spip('inc/config'); je retrouve l'erreur d'éxécution Donc c'est bien en ayant installé GIS sans l'avoir configuré qui génère l'erreur, j'ai dû faire des essais puis désinstaller GIS puisque je n'ai plus non plus aucun point géolocalisés...
Collaborator

Je viens de tester sur SPIP 4.2.0-dev GIT [master: 60644e21] en PHP 8.1.4 et je ne reproduis pas ton erreur avec include_spip, et le patch supprime bien le warning que tu signales dans ton premier message.

Je viens de tester sur SPIP 4.2.0-dev GIT [master: 60644e21] en PHP 8.1.4 et je ne reproduis pas ton erreur avec include_spip, et le patch supprime bien le warning que tu signales dans ton premier message.
b_b closed this issue 6 months ago
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.