Changeset 95070 in spip-zone


Ignore:
Timestamp:
Feb 10, 2016, 3:43:04 PM (3 years ago)
Author:
rastapopoulos@…
Message:

Plein d'accolades pour la syntaxe dans ce fichier, de l'indentation, et deux vérif que l'id est bien un entier.

Location:
_plugins_/formidable/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/formidable/trunk/formidable_autorisations.php

    r87149 r95070  
    8484       
    8585        /* administrateur ? */
    86         if (isset($qui['statut']) and $qui['statut'] <= '0minirezo' and (count($qui['restreint']) == 0))
     86        if (isset($qui['statut']) and $qui['statut'] <= '0minirezo' and (count($qui['restreint']) == 0)) {
    8787                return true;
     88        }
    8889
    8990        /* Test des autorisations par auteur */
     
    9293        }
    9394       
    94         /* Test des autorisations pour un admin restreint */
    95         if (count($qui['restreint'])) {
    96             $autoriser_admin_restreint = isset($GLOBALS['autoriser_admin_restreint'])
    97                     ? $GLOBALS['autoriser_admin_restreint']
    98                         : lire_config('formidable/analyse/autoriser_admin_restreint') == 'on'
    99                             ? true
    100                             : false;
    101             return $autoriser_admin_restreint;
     95        /* Test des autorisations pour un admin restreint */
     96        if (count($qui['restreint'])) {
     97                $autoriser_admin_restreint = isset($GLOBALS['autoriser_admin_restreint'])
     98                                ? $GLOBALS['autoriser_admin_restreint']
     99                                        : lire_config('formidable/analyse/autoriser_admin_restreint') == 'on'
     100                                                ? true
     101                                                : false;
     102               
     103                return $autoriser_admin_restreint;
    102104        }
    103105}
     
    116118**/
    117119function autoriser_formulaires_menu_dist($faire, $type, $id, $qui, $opt){
    118     if (isset($qui['statut']) and $qui['statut'] <= '1comite') return true;
    119     else return false;
     120    if (isset($qui['statut']) and $qui['statut'] <= '1comite') {
     121                return true;
     122        }
     123    else {
     124                return false;
     125        }
    120126}
    121127
     
    138144**/
    139145function autoriser_formulaire_repondre_dist($faire, $type, $id, $qui, $opt){
    140 // On regarde si il y a déjà le formulaire dans les options
    141     if (isset($options['formulaire']))
    142         $formulaire = $options['formulaire'];
    143     // Sinon on va le chercher
    144     else{
    145         $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = '.$id);
    146     }
    147 
    148     $traitements = unserialize($formulaire['traitements']);
    149 
    150     // S'il n'y a pas d'enregistrement, c'est forcément bon
    151     if (!isset($traitements['enregistrement']) OR !($options = $traitements['enregistrement'])) {
    152         return true;
    153     // Sinon faut voir les options
    154     } else {
    155         // Si multiple = oui c'est bon
    156         if ($options['multiple'])
    157             return true;
    158         else{
    159             // Si c'est modifiable, c'est bon
    160             if ($options['modifiable'])
    161                 return true;
    162             else{
    163                 include_spip('inc/formidable');
    164                 // Si la personne n'a jamais répondu, c'est bon
    165                 if (!formidable_verifier_reponse_formulaire($id))
    166                     return true;
    167                 else
    168                     return false;
    169             }
    170         }
    171     }
     146        $id = intval($id);
     147       
     148        // On regarde si il y a déjà le formulaire dans les options
     149        if (isset($opt['formulaire'])) {
     150                $formulaire = $opt['formulaire'];
     151        }
     152        // Sinon on va le chercher
     153        else {
     154                $formulaire = sql_fetsel('*', 'spip_formulaires', 'id_formulaire = '.$id);
     155        }
     156
     157        $traitements = unserialize($formulaire['traitements']);
     158
     159        // S'il n'y a pas d'enregistrement, c'est forcément bon
     160        if (!isset($traitements['enregistrement']) OR !($options = $traitements['enregistrement'])) {
     161                return true;
     162        }
     163        // Sinon faut voir les options
     164        else {
     165                // Si multiple = oui c'est bon
     166                if ($options['multiple']) {
     167                        return true;
     168                }
     169                else {
     170                        // Si c'est modifiable, c'est bon
     171                        if ($options['modifiable']) {
     172                                return true;
     173                        }
     174                        else {
     175                                include_spip('inc/formidable');
     176                                // Si la personne n'a jamais répondu, c'est bon
     177                                if (!formidable_verifier_reponse_formulaire($id)) {
     178                                        return true;
     179                                }
     180                                else {
     181                                        return false;
     182                                }
     183                        }
     184                }
     185        }
    172186}
    173187
     
    239253**/
    240254function autoriser_formulairesreponse_modifier_dist($faire, $type, $id, $qui, $opt){
     255    $id = intval($id);
     256   
    241257    if ($id_formulaire = intval(sql_getfetsel(
    242                         'id_formulaire', 'spip_formulaires_reponses', "id_formulaires_reponse=$id"))) {
    243 
    244                 $retour = (autoriser_formulaire_editer_dist($faire, $type, $id_formulaire, $qui, $opt)
    245                                 and formidable_auteur_admin_reponse($qui));
    246         }
    247         return $retour;
     258                'id_formulaire',
     259                'spip_formulaires_reponses',
     260                "id_formulaires_reponse=$id"
     261        ))) {
     262                return
     263                        autoriser_formulaire_editer_dist($faire, $type, $id_formulaire, $qui, $opt)
     264                        and formidable_auteur_admin_reponse($qui);
     265        }
     266       
     267        return false;
    248268}
    249269
     
    262282function autoriser_formulairesreponse_supprimer_dist($faire, $type, $id, $qui, $opt) {
    263283        $retour = autoriser_formulairesreponse_modifier_dist($faire, $type, $id, $qui, $opt);
     284       
    264285        return $retour;
    265286}
    266 
    267 ?>
  • _plugins_/formidable/trunk/paquet.xml

    r93549 r95070  
    22        prefix="formidable"
    33        categorie="communication"
    4         version="2.9.5"
     4        version="2.9.6"
    55        etat="stable"
    66        compatibilite="[3.0.0;3.1.*]"
Note: See TracChangeset for help on using the changeset viewer.