[spip-dev] tentative de correction du bug "integre_image()" et "envoi des nouveautes"

Bon, je crois que j'ai trouvé : modif dans ecrire/inc.php3, et ajout d'un
fichier inc_image.php3 ; dites-moi si ça résoud le bug qui fait que le site
se bloque tous les deux jours au moment de l'envoi des nouveautés ?

Si j'ai bon, le bug provenait du fait qu'on appelait propre() [donc
integre_image()] à un endroit du fichier inc.php3 où la fonction n'est pas
encore définie (elle n'était définie que tout en bas du fichier inc.php3).
Peut-être le bug apparaissait-il alors sur certaines versions de php (php3),
d'autres (php4?) analysant toutes les fonctions avant d'exécuter ??

-- Fil

Salut,

Bon, heu, beaucoup de modifs. A peu près tous les fichiers de l'espace privé
sont concernés, et peut-être l'espace public aussi.

- Généralisé l'utilisation de include_local() dans l'espace privé. Du coup,
tous les includes (à part inc.php3 et inc_version.php3) doivent faire gaffe
à appeler les variables globales par $GLOBALS. C'est très important. Ne
pas utiliser global, qui balancera des warnings. De façon générale, faire
le moins possible de code global et tout mettre dans des fonctions.

- Enrichi la classe Link, qui introduit des variables temporaires et
des variables de session automatiques. Il faut donc maintenant généraliser
l'emploi de cette classe. Merci de ne plus ajouter de href codé en dur, ni
de form avec hidden inputs codées en dur.
(A titre d'exemple, les variables temporaires sont utilisées dans afficher_tranches()).

- Bazardé l'atroce enlettres.php3. Vais faire de même avec chiffrespar2().
Il faut se restreindre à des fonctions utiles, avec des noms clairs (idem
pour les fichiers).

NB : au début de articles_edit, le code (32ko) a l'air douteux.
Notamment, qu'est-ce qui se passe avec un texte de plus de 64ko ?

on a l'impression, depuis que vous avez intégré cette fonctionnalité
(importante, certes, mais mineure) que spip ne sert qu'à faire des sites
syndiqués...

Non, à mon avis, SPIP doit permettre de créer un annuaire de façon
réellement commode. Sinon, la fonctionnalité est sous-exploitée....

Peut-être le bug apparaissait-il alors sur certaines versions de php (php3),
d'autres (php4?) analysant toutes les fonctions avant d'exécuter ??

Oui.

a+

Antoine.

@ Antoine Pitrou (pitrou@free.fr) :

Bon, heu, beaucoup de modifs. A peu près tous les fichiers de l'espace privé
sont concernés, et peut-être l'espace public aussi.

Et tu as une nouvelle fois écrasé mes toutes dernières modifs (celles
concernant la vérifcation sur les mails) !!

- Enrichi la classe Link, qui introduit des variables temporaires et
des variables de session automatiques. Il faut donc maintenant généraliser
l'emploi de cette classe. Merci de ne plus ajouter de href codé en dur, ni
de form avec hidden inputs codées en dur.

Excellent. Est-ce que ta classe sait faire une fusion entre une URL et un
lien (absolu, /relatif ou relatif), par exemple on lui donne
    "http://www.uzine.net/spip/toto.html","/article2\.html"
    et on récupère [uZine 3] L’internet des pédophiles
    etc... ?.? Ce serait très pratique pour le mail des "nouveautés", entre
    autres

- Bazardé l'atroce enlettres.php3. Vais faire de même avec chiffrespar2().
Il faut se restreindre à des fonctions utiles, avec des noms clairs (idem
pour les fichiers).

D'autant que chiffrespar2($n) c'est bêtement sprintf("%02d",$n)

NB : au début de articles_edit, le code (32ko) a l'air douteux.
Notamment, qu'est-ce qui se passe avec un texte de plus de 64ko ?

Tu coupes(), tu récupères un bout < 32ko, et un autre que tu recoupes au
besoin, etc...

Non, à mon avis, SPIP doit permettre de créer un annuaire de façon
réellement commode. Sinon, la fonctionnalité est sous-exploitée....

Oui, il faut une page centrale où on intègre des sites, pas des boutons
partout dans l'interface !

> Peut-être le bug apparaissait-il alors sur certaines versions de php (php3),
> d'autres (php4?) analysant toutes les fonctions avant d'exécuter ??

Oui.

N'est-ce pas merveilleux ? (Tu lm'as écrasée aussi celle-ci ?)

-- Fil

> Bon, heu, beaucoup de modifs. A peu près tous les fichiers de l'espace privé
> sont concernés, et peut-être l'espace public aussi.

OK, j'ai rétabli mes dernières corrections au milieu des tiennes.
Bizarrement tu avais fait sauter toute la partie "articles" du mail
nouveautés, il ne restait plus que l'annonce des nouvelles brèves.

-- Fil

Fil wrote:

Et tu as une nouvelle fois écrasé mes toutes dernières modifs (celles
concernant la vérifcation sur les mails) !!

Désolé, ton mail est arrivé après ;(

Excellent. Est-ce que ta classe sait faire une fusion entre une URL et un
lien (absolu, /relatif ou relatif), par exemple on lui donne
    "http://www.uzine.net/spip/toto.html&quot;,&quot;/article2\.html&quot;
    et on récupère [uZine 3] L’internet des pédophiles
    etc... ?.? Ce serait très pratique pour le mail des "nouveautés", entre
    autres

Non, du tout... (zut)

N'est-ce pas merveilleux ? (Tu lm'as écrasée aussi celle-ci ?)

Comme j'ai modifié inc.php3, je l'ai refaite tout bêtement en
déplaçant integre_image() vers le haut. J'ai testé chez l'autre net
(php3), ça marche.

Bon, j'ai propagé l'utilisation d'include_local() dans l'espace public
aussi (modifs donc).

a+

Antoine.