[spip-dev] Beta 13 et messages

Nouveau message donne :

Warning: Supplied argument is not a valid MySQL result resource in
/home/sites/site183/web/client/snpi/athena/ecrire/message_edit.php3 on line
36
Vous n'avez pas acc=E8s =E0 ce message

Corrigé : ecrire/inc.php3 dans la beta 13. Désolé !

@ stephan (steph1@mac.com) :

Salut,

Effectivement, un bug a été introduit avec la fonction bouton_imachin(),
je viens de le corriger. (fichier modifié : inc.php3)

a+

Antoine.

stephan wrote:

Coucou,

Merci de reprendre la dernière version du inc.php3 ;))

a+

@ Antoine Pitrou (pitrou@free.fr) :

Merci de reprendre la dernière version du inc.php3 ;))

J'ai remodifié inc.php3 pour rétablir la correction de bug que tu avais
effacée. J'espère ne pas avoir fait de bêtise (cette fois-ci). (beta13,
ecrire/)

-- Fil

Hein, mais je ne l'avais pas effacée ;))
Enfin bon....

Fil wrote:

J'ai remis le mien, qui a l'air de le corriger mieux ;))

a+

Antoine Pitrou wrote:

@ Antoine Pitrou (pitrou@free.fr) :

J'ai remis le mien, qui a l'air de le corriger mieux ;))

Un regret : dans cette fonction bouton_imessage(), on va rechercher par un
select les données de l'auteur. Est-ce que ça ne ralentit pas à mort
l'affichage de la liste de tous les auteurs (1000 requêtes !) alors qu'on
pourrait faire plus optimal en récupérant la donnée au moment du select
auteurs ? Il faudrait peut-être changer le prototype de cette fonction en

bouton_imessage($id_auteur,$imessage=''){
    if ($imessage == '')
        // faire le select

}

-- Fil

Hello,

Fil wrote:

bouton_imessage($id_auteur,$imessage=''){
    if ($imessage == '')
        // faire le select

}

Non, il vaut mieux transmettre la row :

function afficher_auteur($row) {
  $id_auteur = $row[0];
  $nom = $row['nom'];
  $imessage = ($row['imessage'] == 'oui') && (lire_meta('activer_imessage') == 'oui');
  if ($imessage) {
    // Afficher logo avec lien vers nouveau message
  }
  // Afficher nom de l'auteur avec lien vers fiche auteur (auteur.php3?...)
}

a+

Antoine.