La fonction importer_csv envoie un deprecated d'affectation false vers array #5573

Dans la fonction importer_csv() on initialise le retour à false, puis on le cast comme un array quand on traite les lignes du csv.

Le code actuel :

	$return = false;
    ...
    $return[] =: $row;
    ...
    return $return;

Je propose la modification suivante :

	$return = [];
    ...
    $return[] =: $row;
    ...
    return $return ? $return : false;

Dans la fonction importer_csv() on initialise le retour à false, puis on le cast comme un array quand on traite les lignes du csv. Le code actuel : ```php $return = false; ... $return[] =: $row; ... return $return; ``` Je propose la modification suivante : ```php $return = []; ... $return[] =: $row; ... return $return ? $return : false; ```

Heu tu parles de https://git.spip.net/spip/spip/src/branch/master/ecrire/inc/importer_csv.php#L146 ?

Euh moi j'étais en 4.3dev, tu veux dire que c'est corrigé en 5.0 dev ?

bah… oui a priori :)

Cool mais pour la branche 4.2 il faut backporter ?

En fait je ne pense pas que ça soit corrigé, ce que tu signales n’est pas un problème sur le return… mais bon, c’est quand même plus pratique si tu indiquais clairement l’erreur et la ligne concernée indiquée dans la version master.

En fait je ne pense pas que ça soit corrigé, ce que tu signales n’est pas un problème sur le `return`… mais bon, c’est quand même plus pratique si tu indiquais clairement l’erreur et la ligne concernée indiquée dans la version master.

J'ai du mal à comprendre le reproche là.
J'ai repris le code exact.
C'est simple on initialise $return à false puis on lui met un tableau et ça gueule.

https://git.spip.net/spip/spip/src/branch/4.2/ecrire/inc/importer_csv.php#L151

Et donc c'est bien corrigé en 5.0 mais pas en 4.2

J'ai du mal à comprendre le reproche là. J'ai repris le code exact. C'est simple on initialise $return à false puis on lui met un tableau et ça gueule. https://git.spip.net/spip/spip/src/branch/4.2/ecrire/inc/importer_csv.php#L151 Et donc c'est bien corrigé en 5.0 mais pas en 4.2

Je viens de tomber aussi sur

<b>Deprecated</b>:  Automatic conversion of false to array is deprecated in 
<b>spip-bonux/inc/importer_csv.php</b> on line <b>152</b><br />

Je viens de tomber aussi sur ```html Deprecated: Automatic conversion of false to array is deprecated in spip-bonux/inc/importer_csv.php on line 152 ```

b100d7ab60

Du coup, cependant, quel est le besoin de cette surcharge dans SPIP Bonux ?

Et donc oui @Eric par ailleurs, tu as raison, il faut reporter dans SPIP 4.2 aussi effectivement.

https://git.spip.net/spip-contrib-extensions/spip-bonux/commit/b100d7ab601f9e0da20112cbcfad2311c89f212a Du coup, cependant, quel est le besoin de cette surcharge dans SPIP Bonux ? Et donc oui @Eric par ailleurs, tu as raison, il faut reporter dans SPIP 4.2 aussi effectivement.

C’est fait.

Labels Milestones

La fonction importer_csv envoie un deprecated d'affectation false vers array #5573