[SPIP Zone] [Spip-zone-commit] r25189 - in /_plugins_/_dev_/spipBB/dev: plugin.xml spipbb_fonctions.php

Archives Zone
1

Salut Bruno

brunobergot@gmail.com a écrit :

Author: brunobergot@gmail.com
Date: Sat Dec 13 19:30:34 2008
New Revision: 25189

Log:
Eviter d'avoir des erreurs Fatal error: Cannot redeclare inc_notifications_dist() quand on utilise un autre formulaire de forum que celui de spipbb.
Je pense que ce commit ne vas pas dans le même sens que les précédents de Chrys mais il a le mérite de corrriger un bug.

Ce serait cool que j'ai un aperçu du bug en question STP... car je n'en ai nulle trace ...

2

Salut Chryjs,

Pour reproduire le bug il suffit de :

- se créer un formulaire de forum perso en copiant/suffixant celui de
la dist (formulaires/foum.php + formulaires/forum.html +
balise/forum.php)
- utiliser ce formulaire dans un squelette d'article par exemple

Une fois que c'est fait, si on utilise spipbb avant la révision 25189
(http://trac.rezo.net/trac/spip-zone/changeset/25189/_plugins_/_dev_/spipBB/dev)
on obtient l'erreur suivante à la validation du formulaire de prévisu
de notre formulaire perso :

Fatal error: Cannot redeclare inc_notifications_dist() (previously
declared in /home/bb/Sites/spipmut/ecrire/inc/notifications.php:18) in
/home/bb/Sites/spipmut/ecrire/inc/notifications.php on line 31

J'ai découvert ce bug sur un site qui utilise spipbb et sur lequel
nous utilisons un formulaire de forum perso qui surcharge celui de
spip...

++
b_b

Le 15 décembre 2008 10:23, chryjs <chryjs@free.fr> a écrit :

Salut Bruno

brunobergot@gmail.com a écrit :

Author: brunobergot@gmail.com
Date: Sat Dec 13 19:30:34 2008
New Revision: 25189

Log:
Eviter d'avoir des erreurs Fatal error: Cannot redeclare
inc_notifications_dist() quand on utilise un autre formulaire de forum que
celui de spipbb.
Je pense que ce commit ne vas pas dans le même sens que les précédents de
Chrys mais il a le mérite de corrriger un bug.

Ce serait cool que j'ai un aperçu du bug en question STP... car je n'en ai
nulle trace ...

3

bruno bergot a écrit :

Salut Chryjs,

Pour reproduire le bug il suffit de :

[...]

J'ai découvert ce bug sur un site qui utilise spipbb et sur lequel
nous utilisons un formulaire de forum perso qui surcharge celui de
spip...

++
b_b

Ce ne fut pas simple comme manip mais j'ai recréé la situation. Le bug était dans l'inclusion "forcé" de la librairie au lieu de vérifier si elle était nécessaire... mea culpa. Du coup, j'en ai profité pour quelques autres corrections et améliorations :wink:

Tiens moi au courant si jamais il réapparait sous une autre forme :stuck_out_tongue: