[spip-dev] Forum de messagerie incomplet

Dev
1

Bonjour,

Il semble que certaines réponses aux messages ne soient pas prises en compte.

En recherchant un peu, j'ai vu que la pagination était en cause dans inc/discuter.php :

(ligne 46) function inc_discuter_dist($id, $script, $objet, $statut='prive', $debut=1, $pas=10, $id_parent = 0)

si $debut n'est pas précisé alors :

$q = sql_select('*', 'spip_forum', $where, '', "date_heure DESC", "$debut,$pas");

ne renvoie pas le premier message trouvé : "$debut,$pas" == "1,10"

Pour corriger (momentanément !?) l'erreur j'ai mis "$debut=0" :

(ligne 46) function inc_discuter_dist($id, $script, $objet, $statut='prive', $debut=0, $pas=10, $id_parent = 0)

là ça marche... mais je connais pas assez bien le code pour proposer un correctif correct dans tous les cas.

Pat

2

Bonsoir,

Je réitère ici. La messagerie est bien mise à mal en ce moment...
Une mise à jour SVN de ce soir casse complètement les forums attachés aux messages internes :
Erreur(s) dans le squelette
Erreur SQL : "titre FROM spip_messages WHERE id_message="
You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 3

En cherchant un peu, j'ai trouvé que IE plantait alors que FF ne plantait pas ! La raison est l'ancre. Le lien 'Poster un message' donne: ecrire/?exec=poster_forum_prive&statut=perso&id=14&script=message#formulaire
et IE6 va interpréter script="message#formulaire" !!!
Du coup action_poster_forum_prive_dist() répond "pas compris 14/0/perso/message#formulaire/" au lieu de : "14/0/perso/message/id_message".

Je laisse les dev de SPIP trouver le bon patch...
Moi je me suis borné à remplacer dans inc/discuter.php et inc/forum.php : "#formulaire" par "&toto=toto#formulaire"

Merci d'avance !

Euh, le pb évoqué ci-dessus n'est pas non plus réglé. SPIP est tellement complexe que c'est pas évident de déboguer un truc qu'on n'a pas du tout en tête...

Pat

Pat a écrit :

3

Oui, c'est un bug usuel de IE et ajax que j'avais reperé dans d'autre cas.
Cédric

4

Avez-vous jeté un oeil à ce bug ?

Pat a écrit :

5

http://trac.rezo.net/trac/spip/changeset/11784

Committo,Ergo:Sum