Changeset 12187 for spip/ecrire/balise/formulaire_ecrire_auteur.php

Timestamp:

07/24/08 21:05:35 (6 months ago)

Author:

esj@…

Message:

Les LEFT JOIN c'est plus clair que le tableau fait pour le compilateur.

Files:

• spip/ecrire/balise/formulaire_ecrire_auteur.php (modified) (2 diffs)

spip/ecrire/balise/formulaire_ecrire_auteur.php

@@ -29 +29 @@
 
         // Pas d'id_auteur ni d'id_article ? Erreur de squelette
-        if (!$args[0] AND !$args[1])
+        $id = intval($args[1]);
+        if (!$args[0] AND !$id)
                 return erreur_squelette(
                         _T('zbug_champ_hors_motif',
@@ -35 +36 @@
                                         'motif' => 'AUTEURS/ARTICLES')), '');
 
-        // Si on est dans un contexte article, sortir tous les mails des auteurs
-        // de l'article
-        if (!$args[0] AND $args[1]) {
-                unset ($args[2]);
-                $s = sql_select('email',
-                                          array('auteurs' => 'spip_auteurs',
-                                                'L' => 'spip_auteurs_articles'),
-                                          array('auteurs.id_auteur=L.id_auteur',
-                                                'L.id_article='.intval($args[1])));
-                while ($row = sql_fetch($s)) {
-                        if ($row['email'] AND email_valide($row['email']))
-                                $args[2].= ','.$row['email'];
+        // Si on est dans un contexte article, 
+        // sortir tous les mails des auteurs de l'article
+        if (!$args[0] AND $id) {
+                $r = '';
+                $s = sql_allfetsel('email',
+                                   'spip_auteurs AS A LEFT JOIN spip_auteurs_articles AS L ON A.id_auteur=L.id_auteur',
+                                   "A.email != '' AND L.id_article=$id");
+                foreach($s as $row) {
+                        if (email_valide($row['email']))
+                                $r .= ','.$row['email'];
                 }
-                $args[2] = substr($args[2], 1);
+                $args[2] = substr($r, 1);
         }