Changeset 79899 in spip-zone

Timestamp:

Jan 8, 2014, 12:00:51 AM (7 years ago)

Author:

gildas.cotomale@…

Message:

suite de r79851

Location:

_plugins_/Associaspip/trunk

Files:

• action/ajouter_fonctions2groupe.php (deleted)
• action/editer_asso_fonctions.php (modified) (3 diffs)
• formulaires/ajouter_asso_fonctions2groupe.html (modified) (1 diff)
• formulaires/ajouter_asso_fonctions2groupe.php (modified) (1 diff)
• formulaires/ajouter_asso_fonctions2membre.html (modified) (1 diff)
• formulaires/ajouter_asso_fonctions2membre.php (modified) (1 diff)
• formulaires/editer_asso_fonctions2groupe.php (modified) (2 diffs)
• formulaires/editer_asso_fonctions2membre.php (modified) (1 diff)
• prive/editer/editer_asso_groupe.html (modified) (1 diff)
• prive/editer/editer_asso_membre.html (modified) (1 diff)

_plugins_/Associaspip/trunk/action/editer_asso_fonctions.php

@@ -16 +16 @@
         $securiser_action = charger_fonction('securiser_action', 'inc');
         $arg = $securiser_action();
+        $erreur = '';
 
         // cette action peut etre appelee selon trois modes
         if (strpos($arg, '-')) { // mode d'appel 1 : directement depuis un squelette avec en argument <id_groupe>-<id_auteur>
                 list($id_groupe, $id_auteur) = explode('-', $arg);
-                iou_fonction($id_groupe, $id_auteur, _request('fonctions') );
+                $erreur = iou_fonction($id_groupe, $id_auteur, _request('fonctions') );
         } else { // mise a jour par lot...
                 $id_auteur = association_recuperer_entier('id_auteur'); // editer_asso_fonctions2membre
                 $id_groupe = association_recuperer_entier('id_groupe'); // editer_asso_fonctions2groupe
                 $fonctions = association_recuperer_liste('fonctions', TRUE);
-                if ( $id_groupe && $id_groupe==$arg ) { // mettre a jour les fonctions des membres dans le groupe
+                if ( $id_auteur==$arg ) { // mettre a jour les fonctions des membres dans le groupe
                         foreach ($fonctions as $id_auteur => $fonction)
-                                iou_fonction ($id_groupe, $id_auteur, $fonction);
-                } elseif ( $id_groupe && $id_groupe==$arg ) { // mettre a jour les fonctions du membre dans les groupes
+                                $erreur .= iou_fonction ($id_groupe, $id_auteur, $fonction);
+                        if ( $erreur )
+                                $erreur = _T('asso:erreur_sgbdr');
+                } elseif ( $id_groupe==$arg ) { // mettre a jour les fonctions du membre dans les groupes
                         foreach ($fonctions as $id_groupe => $fonction)
-                                iou_fonction ($id_groupe, $id_auteur, $fonction);
+                                $erreur .= iou_fonction ($id_groupe, $id_auteur, $fonction);
+                        if ( $erreur )
+                                $erreur = _T('asso:erreur_sgbdr');
                 } else // mauvais parametres d'appel
-                        return '';
+                        $erreur = _L("argument $arg incompris");
         }
 
-        return '';
+        return $erreur;
 }
 
@@ -46 +51 @@
  * @param $id_auteur int
  * @param $fonction string
- * @return void
+ * @return string
+ *   Vide en cas de modification avec succes, sinon message generique...
  */
-function iou_fonction ($id_groupe, $id_auteur, $fonction) {
+function iou_fonction($id_groupe, $id_auteur, $fonction) {
         if ( sql_countsel('spip_asso_fonctions', "id_groupe=$id_groupe AND id_auteur=$id_auteur") )
                 sql_updateq('spip_asso_fonctions', array(
@@ -59 +65 @@
                         'id_auteur' => $id_auteur,
                 ) );
+        if ( sql_countsel('spip_asso_fonctions', "id_groupe=$id_groupe AND id_auteur=$id_auteur and fonction=".sql_quote($fonction)) )
+                return _T('asso:erreur_sgbdr');
+        else
+                return '';
 }
 

_plugins_/Associaspip/trunk/formulaires/ajouter_asso_fonctions2groupe.html

@@ -11 +11 @@
                 <li class="liste_auteurs">
                         <div class="choix">
-                                <input type="checkbox" name="id_auteurs[]" value="#ID_AUTEUR" id="auteur#ID_AUTEUR" />
+                                <input type="checkbox" name="fonctions[#ID_AUTEUR]" value="" id="auteur#ID_AUTEUR" />
                                 <label for="auteur#ID_AUTEUR">[(#SEXE|association_formater_nom{#PRENOM, #NOM_FAMILLE})]</label>
                         </div>

_plugins_/Associaspip/trunk/formulaires/ajouter_asso_fonctions2groupe.php

@@ -14 +14 @@
 include_spip('inc/editer');
 
-function formulaires_ajouter_asso_fonctions2groupe_charger_dist($id_groupe='') {
-        $contexte['id_groupe'] = $id_groupe;
-        $contexte['_action'] = array('ajouter_asso_fonction2groupe', $id_groupe); // pour passer securiser action
+function formulaires_ajouter_asso_fonctions2groupe_charger_dist($id_groupe=0) {
+        $contexte['id_groupe'] = $id_groupe; // passer l'argument dans l'environnement
+        $contexte['_action'] = array('editer_asso_fonctions', $id_groupe); // pour passer securiser action
         return $contexte;
 }
 
-function formulaires_ajouter_asso_fonctions2groupe_traiter($id_groupe='') {
+function formulaires_ajouter_asso_fonctions2groupe_traiter($id_groupe=0) {
         $res = array();
         set_request('redirect'); // eviter la redirection forcee par l'action...
-        $action_ajouter_membres = charger_fonction('ajouter_fonctions2groupe','action');
-        $action_ajouter_membres($id_groupe);
+        $action_ajouter_membres = charger_fonction('editer_asso_fonctions', 'action');
+        $res['message_erreur'] = $action_ajouter_membres($id_groupe);
         $res['message_ok'] = '';
         $id_groupe = intval($id_groupe);
-        if ($id_groupe>0 && $id_groupe<100) {
-                $res['redirect'] = generer_url_ecrire('edit_groupe_autorisations', 'id='.$id_groupe);
-        } else {
-                $res['redirect'] = generer_url_ecrire('edit_groupe', 'id='.$id_groupe);
-        }
+        $res['redirect'] = generer_url_ecrire((($id_groupe>0 && $id_groupe<100)?'edit_groupe_autorisations':'edit_groupe'), 'id='.$id_groupe);
         return $res;
 }

_plugins_/Associaspip/trunk/formulaires/ajouter_asso_fonctions2membre.html

@@ -11 +11 @@
                 <li class="liste_auteurs">
                         <div class="choix">
-                                <input type="checkbox" name="id_auteurs[]" value="#ID_GROUPE" id="groupe#ID_GROUPE" />
+                                <input type="checkbox" name="fonctions[#ID_GROUPE]" value="" id="groupe#ID_GROUPE" />
                                 <label for="groupe#ID_GROUPE">[(#NOM)]</label>
                         </div>

_plugins_/Associaspip/trunk/formulaires/ajouter_asso_fonctions2membre.php

@@ -15 +15 @@
 include_spip('inc/editer');
 
-function formulaires_ajouter_asso_fonctions2membre_charger_dist($id_auteur='') {
-        $contexte['id_auteur'] = $id_auteur;
-        $contexte['_action'] = array('ajouter_asso_fonction2membre', $id_auteur); // pour passer securiser action
+function formulaires_ajouter_asso_fonctions2membre_charger_dist($id_auteur=0) {
+        $contexte['id_auteur'] = $id_auteur; // passer l'argument dans l'environnement
+        $contexte['_action'] = array('editer_asso_fonctions', $id_auteur); // pour passer securiser action
         return $contexte;
 }
 
-function formulaires_ajouter_asso_fonctions2membre_traiter($id_auteur='') {
+function formulaires_ajouter_asso_fonctions2membre_traiter($id_auteur=0) {
         $res = array();
         set_request('redirect'); // eviter la redirection forcee par l'action...
-        $action_ajouter_membres = charger_fonction('ajouter_membre_groupes','action');
-        $action_ajouter_membres($id_auteur);
+        $action_ajouter_membres = charger_fonction('editer_asso_fonctions', 'action');
+        $res['message_erreur'] = $action_ajouter_membres($id_auteur);
         $res['message_ok'] = '';
         $res['redirect'] = generer_url_ecrire('edit_adherent', 'id='.$id_auteur);

_plugins_/Associaspip/trunk/formulaires/editer_asso_fonctions2groupe.php

@@ -16 +16 @@
 
 function formulaires_editer_asso_fonctions2groupe_charger_dist($id_groupe=0) {
-        $contexte['id_groupe'] = $id_groupe;
+        $contexte['id_groupe'] = $id_groupe; // passer l'argument dans l'environnement
         $contexte['_action'] = array('editer_asso_fonctions', $id_groupe); // pour passer securiser action
 
@@ -33 +33 @@
         $res['message_ok'] = '';
         $id_groupe = intval($id_groupe);
-        if ($id_groupe>0 && $id_groupe<100) {
-                $res['redirect'] = generer_url_ecrire('edit_groupe_autorisations', 'id='.$id_groupe);
-        } else {
-                $res['redirect'] = generer_url_ecrire('edit_groupe', 'id='.$id_groupe);
-        }
+        $res['redirect'] = generer_url_ecrire((($id_groupe>0 && $id_groupe<100)?'edit_groupe_autorisations':'edit_groupe'), 'id='.$id_groupe);
         return $res;
 }

_plugins_/Associaspip/trunk/formulaires/editer_asso_fonctions2membre.php

@@ -16 +16 @@
 
 function formulaires_editer_asso_fonctions2membre_charger_dist($id_auteur=0) {
-        $contexte['id_auteur'] = $id_auteur;
+        $contexte['id_auteur'] = $id_auteur; // passer l'argument dans l'environnement
         $contexte['_action'] = array('editer_asso_fonctions', $id_auteur); // pour passer securiser action
 

_plugins_/Associaspip/trunk/prive/editer/editer_asso_groupe.html

@@ -1 @@
-#FORMULAIRE_EDITER_ASSO_GROUPE{#ENV{id}}
+[<div class="ajax">(#FORMULAIRE_EDITER_ASSO_GROUPE{#ENV{id}})</div>]
 [<div class="ajax">(#FORMULAIRE_EDITER_ASSO_FONCTIONS2GROUPE{#ENV{id}})</div>]
 [<div class="ajax">(#FORMULAIRE_AJOUTER_ASSO_FONCTIONS2GROUPE{#ENV{id}})</div>]

_plugins_/Associaspip/trunk/prive/editer/editer_asso_membre.html

@@ -1 @@
-#FORMULAIRE_EDITER_ASSO_MEMBRE{#ENV{id_auteur}}
+[<div class="ajax">(#FORMULAIRE_EDITER_ASSO_MEMBRE{#ENV{id_auteur}})</div>]
 [<div class="ajax">(#FORMULAIRE_EDITER_ASSO_FONCTIONS2MEMBRE{#ENV{id_auteur}})</div>]
 [<div class="ajax">(#FORMULAIRE_AJOUTER_ASSO_FONCTIONS2MEMBRE{#ENV{id_auteur}})</div>]