Changeset 48921 in spip-zone for _plugins_/spip-listes/spip-listes_1_9_3/formulaires/gestion_abonnement.php

Timestamp:

Jun 19, 2011, 10:55:40 AM (10 years ago)

Author:

paladin@…

Message:

Révision du formulaire de gestion abonnement.

File:

• _plugins_/spip-listes/spip-listes_1_9_3/formulaires/gestion_abonnement.php (modified) (4 diffs)

_plugins_/spip-listes/spip-listes_1_9_3/formulaires/gestion_abonnement.php

@@ -102 +102 @@
                 // pour cet auteur
                 $mes_abos = spiplistes_abonnements_listes_auteur ($id_auteur, true);
-                
+                                                          
                 // demander de stopper une inscription ?
                 if ($stop > 0)
@@ -111 +111 @@
                         {
                                 spiplistes_abonnements_auteur_desabonner ($id_auteur, $id_liste);
-                                $contexte = array(
-                                        'message_ok' => _T('spiplistes:vous_etes_desinscrit')
-                                );
+                                $message_ok = _T('spiplistes:abonnement_modifie').'.'
+                                        . '<br />' . PHP_EOL
+                                        . _T('spiplistes:vous_n_etes_plus_abonne_aux_listes')
+                                        ;
                         }
                 }
@@ -125 +126 @@
                                 : array()
                                 ;
+                        
+                        /**
+                         * supprime d'abord tous les abonnements
+                         */
+                        spiplistes_abonnements_auteur_desabonner ($id_auteur, 'toutes');
+                        $message_ok = _T('spiplistes:abonnement_modifie').'.';
+                        
+                        /**
+                         * Abonne aux listes sélectionnées
+                         */
                         if (count ($listes_souhaitees))
                         {
+                                                                          
+                                // les cles sont les id_listes
                                 $listes_souhaitees = array_flip ($listes_souhaitees);
                                 
-                                // abonner aux listes demandées
-                                foreach (array_keys($listes_souhaitees) as $id_liste)
-                                {
-                                        if (!isset($mes_abos[$id_liste]))
-                                        {
-                                                spiplistes_abonnements_ajouter ($id_auteur, $id_liste);
-                                                $mes_abos[$id_liste] = '';
-                                        }
-                                }
-                                
-                                // désabonner les listes non souhaitées
-                                foreach (array_keys($mes_abos) as $id_liste)
-                                {
-                                        if (!isset ($listes_souhaitees[$id_liste]))
-                                        {
-                                                spiplistes_abonnements_auteur_desabonner ($id_auteur, $id_liste);
-                                        }
-                                }
-                                
-                                // abonner aux listes demandées,
-                                // si pas déjà abonné
-                                foreach (array_keys($listes_souhaitees) as $id_liste)
-                                {
-                                        if (!isset ($mes_abos[$id_liste]))
-                                        {
-                                                if(spiplistes_abonnements_ajouter($id_auteur, $id_liste) !== false)
-                                                {
-                                                        // @todo a ameliorer, style une ligne de confirmation par liste ?
-                                                        $message_ok = _T('spiplistes:abonnement_modifie');
-                                                }
-                                        }
-                                }
-                        }
-                        else if (count ($mes_abos))
-                        {
-                                // tout est déselectionné ?
-                                // supprimer tout les abonnements !
-                                spiplistes_abonnements_auteur_desabonner ($id_auteur, 'toutes');
-                                $mes_abos = array();
+                                spiplistes_abonnements_ajouter ($id_auteur
+                                                                                                , array_keys($listes_souhaitees)
+                                                                                                );
+                                $nb = count ($listes_souhaitees);
+                                if ($nb >= 1)
+                                {
+                                        $message_ok .= '<br />' . PHP_EOL;
+                                        $message_ok .=
+                                                ($nb > 1)
+                                                ? _T('spiplistes:vous_etes_abonne_aux_listes_selectionnees')
+                                                : _T('spiplistes:vous_etes_abonne_a_la_liste_selectionnee')
+                                                ;
+                                }
+                        }
+                        else if (count($mes_abos))
+                        {
+                                $message_ok .= '<br />' . PHP_EOL
+                                        . _T('spiplistes:vous_n_etes_plus_abonne_aux_listes')
+                                        ;
                         }
                         
@@ -179 +172 @@
                                         }
                                         
-                                        $message_ok = _T('spiplistes:desabonnement_valid').' :&nbsp;'.$email;  
+                                        $message_ok = _T('spiplistes:abonnement_modifie').'.'
+                                                . '<br />' . PHP_EOL
+                                                . _T('spiplistes:vous_n_etes_plus_abonne_aux_listes')
+                                                ; 
                                 }
                                 else {