Changeset 47993 in spip-zone


Ignore:
Timestamp:
May 21, 2011, 9:05:48 PM (10 years ago)
Author:
yffic@…
Message:

Nommer différemment le paramètre URL et l'élément de tableau de retour de la fonction formulaires_abomailman_creation_liste_charger_dist, sinon le paramêtre URL est supprimé par le système CVT. Ceci corrige le bug suivant : prenez une liste existante, passez en modification, supprimez l'email de la liste pour provoquer l'erreur dans verifier, et validez : le formulaire affiché est vide.

Location:
_plugins_/abomailmans
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • _plugins_/abomailmans/exec/abomailmans_edit.php

    r39200 r47993  
    1616function exec_abomailmans_edit(){
    1717
    18         $id_abomailman = intval(_request('id_abomailman'));
     18        $id_abomailman = intval(_request('id_abo'));
    1919        $retour = _request('retour');
    2020

  • _plugins_/abomailmans/formulaires/abomailman_creation_liste.php

    r39241 r47993  
    77// chargement des valeurs par defaut des champs du formulaire
    88function formulaires_abomailman_creation_liste_charger_dist($id_abomailman = ""){
    9                 //initialise les variables d'environnement pas défaut
    10                 $valeurs = array();
    11                 $valeurs['editable'] = true;
     9        //initialise les variables d'environnement pas défaut
     10        $valeurs = array();
     11        $valeurs['editable'] = true;
    1212
    13                 // On verifie que la liste existe
    14                 if ($id_abomailman){
    15                         $valeurs = sql_fetsel('*','spip_abomailmans',"id_abomailman =$id_abomailman");
    16                         $valeurs['langue'] = $valeurs['lang'];
    17                         if(!$valeurs['id_abomailman']){
    18                                 $valeurs['editable'] = false;
    19                                 $valeurs['message_erreur'] = _T('abomailmans:liste_non_existante');
    20                         }
    21                 }       
    22                
    23                 if(!$valeurs['langue']){
    24                         $valeurs['langue'] = lang_select();
     13        // On verifie que la liste existe
     14        if ($id_abomailman){
     15                $valeurs = sql_fetsel('*','spip_abomailmans',"id_abomailman=$id_abomailman");
     16                $valeurs['langue'] = $valeurs['lang'];
     17                if(!$valeurs['id_abomailman']){
     18                        $valeurs['editable'] = false;
     19                        $valeurs['message_erreur'] = _T('abomailmans:liste_non_existante');
    2520                }
    26                 unset($valeurs['lang']);
    27                
    28                 $recuptemplate = explode('&',$valeurs['modele_defaut']);
    29                 $valeurs['template'] = $recuptemplate[0];
    30                 $valeurs['envoi_liste_parametres']=recup_param($valeurs['modele_defaut']);
     21        }       
     22       
     23        if(!$valeurs['langue']){
     24                $valeurs['langue'] = lang_select();
     25        }
     26        unset($valeurs['lang']);
     27
     28        $recuptemplate = explode('&',$valeurs['modele_defaut']);
     29        $valeurs['template'] = $recuptemplate[0];
     30        $valeurs['envoi_liste_parametres']=recup_param($valeurs['modele_defaut']);
    3131         
    3232        return $valeurs;
    … …  
    3939
    4040        // Faire une fonction de verif sur le mail et le titre pour validite
    41 
    4241        $titre = _request('titre');
    4342        $descriptif = _request('descriptif');
    … …  
    8079
    8180function formulaires_abomailman_creation_liste_traiter_dist($id_abomailman = ""){
    82     $message = '';
     81   $message = array();
    8382        $message['editable'] = true;
    8483        $valeurs['envoi_liste_parametres'] = _request('envoi_liste_parametres');
    … …  
    9897        $datas['lang'] = _request('langue');
    9998
    100     // on récupère les données de la liste
    101     if(intval($id_abomailman)){
    102         if($datas['desactive'] == '2'){
    103                 sql_delete("spip_abomailmans","id_abomailman = $id_abomailman");
    104                 $message['message_ok'] = _T('abomailmans:liste_supprimee',array("id"=>$id_abomailman,"titre"=> $datas['titre']));
    105                 $message['editable'] = false;
    106         }else{
    107                 sql_updateq("spip_abomailmans",$datas,"id_abomailman = $id_abomailman");
    108                 $message['message_ok'] = _T('abomailmans:liste_updatee',array("id"=>$id_abomailman,"titre"=> $datas['titre']));
    109         }
    110         $message['redirect'] = parametre_url(self(),'id_abomailman',$id_abomailman);
    111     }
    112         else{
     99        // on récupère les données de la liste
     100        if(intval($id_abomailman)){
     101                if($datas['desactive'] == '2'){
     102                        sql_delete("spip_abomailmans","id_abomailman = $id_abomailman");
     103                        $message['message_ok'] = _T('abomailmans:liste_supprimee',array("id"=>$id_abomailman,"titre"=> $datas['titre']));
     104                        $message['editable'] = false;
     105                }else{
     106                        sql_updateq("spip_abomailmans",$datas,"id_abomailman = $id_abomailman");
     107                        $message['message_ok'] = _T('abomailmans:liste_updatee',array("id"=>$id_abomailman,"titre"=> $datas['titre']));
     108                }
     109        }else{
    113110                $id_abomailman = sql_insertq("spip_abomailmans",$datas);
    114111                $message['message_ok'] = _T('abomailmans:liste_creee',array("id"=>$id_abomailman,"titre"=> $datas['titre']));
    115112                $message['editable'] = false;
    116                 $message['redirect'] = parametre_url(self(),'id_abomailman',$id_abomailman);
    117113        }
    118114
    119     return $message;
     115        $message['redirect'] = parametre_url(self(),'id_abo',$id_abomailman);
     116
     117   return $message;
    120118}
    121119

  • _plugins_/abomailmans/plugin.xml

    r45766 r47993  
    1111        MaZiaR [NetAktiv->http://www.netaktiv.com]
    1212        </auteur>
    13         <version>0.6.2</version>
     13        <version>0.6.3</version>
    1414        <version_base>0.33</version_base>
    1515        <prefix>abomailmans</prefix>

  • _plugins_/abomailmans/prive/abomailman_afficher_abomailmans.html

    r31284 r47993  
    1212                        </td>
    1313                        <td class="arial11">
    14                                 [(#TITRE|icone_horizontale{[(#URL_ECRIRE{abomailmans_edit}|parametre_url{id_abomailman,#ID_ABOMAILMAN}|parametre_url{retour,#SELF})],#CHEMIN{img_pack/mailman.gif},"edit.gif",""})]
     14                                [(#TITRE|icone_horizontale{[(#URL_ECRIRE{abomailmans_edit}|parametre_url{id_abo,#ID_ABOMAILMAN}|parametre_url{retour,#SELF})],#CHEMIN{img_pack/mailman.gif},"edit.gif",""})]
    1515                        </td>
    1616                        <td>
Note: See TracChangeset for help on using the changeset viewer.