Changeset 87946 in spip-zone for _plugins_/abonnements_zones


Ignore:
Timestamp:
Mar 13, 2015, 11:30:51 PM (4 years ago)
Author:
rastapopoulos@…
Message:

Je ne comprends pas du tout le commit [73322] ni même le précédent [73280] : ce plugin ne s'intéresse absolument qu'au *statut* des abonnements. Or ces statuts ne sont changés que dans 'instituer', jamais dans 'modifier' (c'est dans le principe même de l'API des objets : statut = instituer). Là en plus, ça liait les zones même quand on ne faisait PAS d'activation (quand on changeait d'autres champs sans que le statut ne change). On se retrouvait avec des zones même avec un abonnement inactif.

Location:
_plugins_/abonnements_zones/trunk
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/abonnements_zones/trunk/abozones_pipelines.php

    r81793 r87946  
    2626        if (
    2727                $flux['args']['table'] == 'spip_abonnements'
    28                 and (($flux['args']['action'] == 'modifier') or ($flux['args']['action'] == 'instituer'))
     28                and $flux['args']['action'] == 'instituer'
    2929                and $id_abonnement = intval($flux['args']['id_objet'])
    3030                and $abonnement = sql_fetsel('id_abonnements_offre,id_auteur', 'spip_abonnements', 'id_abonnement = '.$id_abonnement)
     
    4343                }
    4444               
    45                 // Si c'est une activation on ajoute les zones trouvées à l'utilisateur de l'abonnement SANS autorisation
     45                // Si c'est une activation (nouveau statut actif, ancien différent)
     46                // on ajoute les zones trouvées à l'utilisateur de l'abonnement SANS autorisation
    4647                if (
    47                         (($flux['data']['statut'] == 'actif') or !isset($flux['data']['statut']))
    48                         and !isset($flux['data']['statut_ancien'])                                             
     48                        $flux['data']['statut'] == 'actif'
     49                        and $flux['data']['statut_ancien'] != 'actif'
    4950                ) {
    5051                        autoriser_exception('affecterzones', 'auteur', $id_auteur);
  • _plugins_/abonnements_zones/trunk/paquet.xml

    r87210 r87946  
    22        prefix="abozones"
    33        categorie="auteur"
    4         version="1.0.2"
     4        version="1.0.3"
    55        etat="stable"
    66        compatibilite="[3.0.5;3.0.*]"
Note: See TracChangeset for help on using the changeset viewer.