Encore une histoire de warnings:
J'obtiens une centaine de
"Warning: undefined index ## in skel_article-dist.php3 on line 377"
## allant de 1 à 100...
Soit, en clair, le squelette effectue 100 fois une requête avec une
erreur de syntaxe ($hierarchie[##] est utiisé dans la construction de la
requête).
Est-ce voulu? Est-ce un bug que j'ai introduit moi? Y a-t-il une
subtilisté que j'ai raté?
Bonjour Fil et à tous,
@ spipjbcombes@altern.org <spipjbcombes@altern.org> :
Voici un patch qui reduit (un ptit peu) le nombre des warnings.
J'ai tenté de garder la sémantique unset == false.
Je n'aime pas du tout cette manière de réduire les warnings, car ça réduit
aussi (et de beaucoup) la compréhension du code. Je préférerais (si ça peut
suffire) qu'on déclare toutes les variables au début (en profitant pour les
commenter si besoin est).
Cela me parait aussi une meilleur idée...
A ce propos, est ce qu'il y a un document sur la norme de codage ?
Cela permet souvent d'avoir une meilleur lisibilité et donc
d'accueillir plus facilement de nouveaux dévelopeurs.
Bruno
En réponse à Fil,
@ Bruno Demeilliez <webmaster@aquanaute.com> :
> A ce propos, est ce qu'il y a un document sur la norme de codage ?Non. Mais je crois qu'un certain Bruno Demeilliez met la dernière main à
une base de gestion de bugs (http://rezo.net/spip-dev/devel/BUGS/)
-- on ne
peut
pas dire que les choses n'avancent pas....
Pour la norme de codage, ce n'était pas une critique juste une question...
J'ai un peu de doc a ce sujet pour le dev en java, et je peux essayer de faire
une adaptation que je vous soumettrez pour discussion si l'équipe de dev juge
cela intéressant. Encore une fois, ma démarche se veut constructive, il n'est
pas question d'imposer quoi que ce soit, juste d'aider dans la mesure de mes
faibles capacités une équipe qui a fait du super boulot avec SPIP.
Pour le gestionnaire de bugs, il devrait être complètement configuré ce soir.
Il faut que je trouve le temps de finaliser deux ou trois points avant qu'il
soit complètement exploitable.
Bruno
Hi Fil!
@ spipjbcombes@altern.org <spipjbcombes@altern.org> :
> Voici un patch qui reduit (un ptit peu) le nombre des warnings.
> J'ai tenté de garder la sémantique unset == false.Je n'aime pas du tout cette manière de réduire les warnings, car ça réduit
aussi (et de beaucoup) la compréhension du code. Je préférerais (si ça peut
suffire) qu'on déclare toutes les variables au début (en profitant pour les
commenter si besoin est).
Ce n'est pas directement possible. Il faut pouvoir determiner ce que
l'on teste, i.e. si on teste l'existence ou la valeur de vérité de la
vaiable, et souvent, savoir d'où elle sort.
La première étape est (pour moi) d'en éliminer un certain nombre, pour
pouvoir faire fonctionner SPIP cvs chez moi. La deuxième étape est de
vérifier les isset(), mais je ne connais pas assez le code pour tout
suivre... Notament sur les tableaux et les objets...
> J'obtiens une centaine de
> "Warning: undefined index ## in skel_article-dist.php3 on line 377"
> ## allant de 1 à 100...
> Soit, en clair, le squelette effectue 100 fois une requête avec une
> erreur de syntaxe ($hierarchie[##] est utiisé dans la construction de la
> requête).
> Est-ce voulu? Est-ce un bug que j'ai introduit moi? Y a-t-il une
> subtilisté que j'ai raté?Aucune idée. Peux-tu mettre ton fichier CACHE/skel_article-dist.php3 quelque
part sur ton site (en .phps ou .txt histoire qu'on puisse le lire) ?
En ligne http://jbcombes.free.fr/skel_articles.txt
ligne 375 et suivantes.
Elles sont générées par inc-calcul-squel.php3 l. 1623 et suivantes
Voila.
From gaetan@virtual-net.fr Tue Apr 9 12:11:35 2002
Return-Path: <gaetan@virtual-net.fr>
Received: from kawa.virtual-net.fr (aboukir-101-2-1-vnet.adsl.nerim.net
[62.4.19.143])
by miel.brainstorm.fr (Postfix) with ESMTP id 9D5CB1C7FB
for <spip-dev@rezo.net>; Tue, 9 Apr 2002 12:11:34 +0200 (CEST)
Received: from nutella.virtual-net.fr (root@nutella.virtual-net.fr
[192.168.1.1])
by kawa.virtual-net.fr (8.9.3/8.9.3/Debian 8.9.3-21) with ESMTP id MAA08091
for <spip-dev@rezo.net>; Tue, 9 Apr 2002 12:11:37 +0200
Received: (from gaetan@localhost)
by nutella.virtual-net.fr (8.9.3/8.9.3/Debian 8.9.3-21) id MAA17734
for spip-dev@rezo.net; Tue, 9 Apr 2002 12:11:34 +0200