Changeset 81706 in spip-zone


Ignore:
Timestamp:
Apr 1, 2014, 2:46:04 PM (5 years ago)
Author:
tcharlss@…
Message:

Correction d'un bug sur les authorisations (qui m'a fait donné du fil à retordre).
Ou comment un problème de nommage d'une fonction peut avoir des répercussions inattendues :
le formulaire des mots-clés s'obstinait à s'afficher sur les commandes, même quand l'utilisation des mots-clés étaient désactivé.
Tout ça résultait d'une coquille sur l'autorisation de "voir" une commande : la fonction était mal nommée ("autoriser_commande_dist" au lieu de "autoriser_commande_voir_dist")

Je note que l'autorisation à passer une commande suppose la présence du plugin contact, qui n'est pourtant pas requis par le plugin.
De toute façon le plugin devrait être agnostique sur ce point à mon avis, mais je n'y touche pas pour préserver la compatibilité.

Location:
_plugins_/commandes/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/commandes/trunk/commandes_autorisations.php

    r72799 r81706  
    1616 * Fonction d'appel pour le pipeline
    1717 * @pipeline autoriser */
    18 function commandes_autoriser($flux){ return $flux; }
     18function commandes_autoriser(){}
     19
    1920
    2021/**
     
    3940}
    4041
     42
    4143/**
    42  * Autorisation à voir(?) une commande
     44 * Autorisation à voir une commande
    4345 * - l'auteur de la commande
    4446 * - admin (mais pas restreint)
     
    5153 * @return bool          true s'il a le droit, false sinon
    5254**/
    53 function autoriser_commande_dist($faire, $quoi, $id, $qui, $opts) {
     55function autoriser_commande_voir_dist($faire, $quoi, $id, $qui, $opts) {
    5456        return
    5557                $qui['id_auteur'] == sql_getfetsel('id_auteur', 'spip_commandes', 'id_commande = '.sql_quote($id)) OR
     
    5759                         AND !$qui['restreint'] );
    5860}
     61
    5962
    6063/**
  • _plugins_/commandes/trunk/paquet.xml

    r81704 r81706  
    22        prefix="commandes"
    33        categorie="divers"
    4         version="0.10.11"
     4        version="0.10.12"
    55        etat="dev"
    66        compatibilite="[3.0.5;3.0.*]"
Note: See TracChangeset for help on using the changeset viewer.