[spip-dev] r18938 - spip/ecrire/inc

Dev
1

?

#2509 ne porte pas sur le phpdoc
mais sur strlen

JL

2

le strlen sur le premier argument URL qui ne peut être qu'une chaine de caractère alors que ton exemple fournit un array()
D'où le correctif qui est de la documentation :slight_smile:

Cédric

3

ok !

4

Je n'ai pas compris ;-(

Ligne 346 on a ce code :

elseif (strlen($v)) {

Si $v est une chaine, pas de problème. Mais si $v est un tableau, en php5.2 strlen renvoie 5 (nb de caracteres dans la chaine "Array") avec un E_NOTICE, et en php5.3, strlen renvoie NULL (avec un E_WARNING). Enfin d'après http://php.net/manual/fr/function.strlen.php

5

En effet, le code semble buggué avec un $v au format array.
Mais il faut écrire un test complet avant de toucher à cette fonction sinon on va introduire des bugs a coup sur.

Cédric

6

Peux tu vérifier que c’est bon avec http://core.spip.org/projects/spip/repository/revisions/18959
Cédric