[SPIP Zone] r100118 - in _plugins_/agenda/trunk

rastapopoulos · Octobre 24, 2016, 11:38

Le 24/10/2016 à 13:05, julien@cahri.com a écrit :

Log:
ajout d’une constante pour autoriser la création/modification d’évènements orphelins par les admins : _AGENDA_AUTORISER_ORPHELINS

Euuuuh, ça aurait peut-être été bien d'en parler non ?

C'est un truc voulu depuis longtemps ok, mais là tel quel, ça introduit du code qui est amené à devenir pérenne (puisque ajoutés dans le plugin central directement), donc est-ce que c'est comme ça qu'on veut faire ?

Un avis des mainteneurs du plugin avant d'inscrire dans le marbre telle ou telle manière de faire, c'est peut-être pas mal ? (Mais peut-être ai-je raté une discussion ?)

--
RastaPopoulos

Julien_Tessier1 · Octobre 24, 2016, 1:01

Bonjour,

J’avais déjà contribué au plugin dans le passé donc je ne suis pas nouveau sur le plugin ^^ J’ai aussi contacté Cédric Morin en parallèle pour l’avertir.

De plus, je pensais avoir respecté un des principes de la zone :

Ya des trucs spécifiques à SPIP :
- « on code d’abord, on discute après » (pour donner du coeur à l’ouvrage et éviter les discussions stériles)
- « gogogo » ! (version applicative du précédent précepte)

Du coup, on peut discuter maintenant Tu verras le commit : je me base sur des nouvelles autorisations qui permettent de changer le comportement éventuel de qui peut créer/modifier/visualiser/instituer des articles orphelins selon ce que la communauté pense être le mieux.

Pour info, la modif ne fait rien tant que la constante n’est pas définie et c’est bien le but : ne pas changer le comportement du plugin tant que les changements ne sont pas approuvés. Les utilisateurs n’en sauront rien tant que le mode opératoire exact ne sera pas en place.

Cordialement.

rastapopoulos · Octobre 24, 2016, 5:18

Le 24/10/2016 à 15:01, Julien Tessier a écrit :

J'avais déjà contribué au plugin dans le passé donc je ne suis pas
nouveau sur le plugin ^^ J'ai aussi contacté Cédric Morin en parallèle
pour l'avertir.

Oui oui d'où ma phrase "mais peut-être ai-je raté une discussion ?"
(qui aurait été bien ici, enfin je trouve)

Pour info, la modif ne fait rien tant que la constante n'est pas définie
et c'est bien le but : ne pas changer le comportement du plugin tant que
les changements ne sont pas approuvés. Les utilisateurs n'en sauront
rien tant que le mode opératoire exact ne sera pas en place.

Oui j'ai déjà lu ce qui est ajouté, mais justement, même si ça ne modifie rien à l'existant, ça ajoute quand même du code, une certaine gestion dérogatoire avec des autorisations en plus, et donc je posais la question… ne sachant pas si ça avait été discuté ou pas, et si ça correspond à l'orientation qu'on veut donner (ou la pérennité du truc).

Notamment ça ajoute du coup un concept (même si ce n'est pas visible, cela ajoute un concept) de "événement orphelin".

Sinon, en code, il me semble qu'il manque de faire passer le "$id" à toutes les nouvelles autorisations, pour avoir accès à quel événement on teste, car là tous les appels sont génériques autoriser('trucorphelin', 'evenement').

--
RastaPopoulos

YannX_Spip · Octobre 24, 2016, 6:50

Pour info, la modif ne fait rien tant que la constante n’est pas définie et c’est bien le but : ne pas changer le comportement du plugin tant que les changements ne sont pas approuvés. Les utilisateurs n’en sauront rien tant que le mode opératoire exact ne sera pas en place.

Cordialement.

Bravo !
Cela m’évite de continuer à chercher/inventer la modification
Et merci…

Maintenant, va falloir éclaircir la documentation
(de quel article, entre SPIP 2 et SPIP 3 ?

Laurent3 · Octobre 25, 2016, 6:26

Merci pour cette initiative qui répond à un besoin récurent !

je me demande même pourquoi cette fonctionnalité est réservée aux admins : il m'arrive de temps en temps de créer des interface pour proposer à des auteurs sur inscription de créer des événements sur un agenda commun. Je n'ai jusqu'à ce jour, pas trouvé d'autre manière de fonctionner que de créer un article "fourre tout" pour y déposer l'ensemble des événements entrés par les contributeurs.

Merci !

Le 24/10/2016 à 20:50, YannX SPIP a écrit :

Pour info, la modif ne fait rien tant que la constante n'est pas définie et c'est bien le but : ne pas changer le comportement du plugin tant que les changements ne sont pas approuvés. Les utilisateurs n'en sauront rien tant que le mode opératoire exact ne sera pas en place.

Cordialement.

2016-10-24 15:38 GMT+04:00 RastaPopoulos <rastapopoulos@spip.org <mailto:rastapopoulos@spip.org>>:

    Le 24/10/2016 à 13:05, julien@cahri.com <mailto:julien@cahri.com>
    a écrit :

        Log:
        ajout d’une constante pour autoriser la création/modification
        d’évènements orphelins par les admins :
        _AGENDA_AUTORISER_ORPHELINS

Bravo !
Cela m'évite de continuer à chercher/inventer la modification
Et merci...

Maintenant, va falloir éclaircir la documentation
(de quel article, entre SPIP 2 et SPIP 3 ?

--
YannX
http://www.spippourlesnuls.fr

----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

Julien_Tessier1 · Octobre 27, 2016, 8:59

Bonjour,

@YannX : je suis d’accord pour la gestion des statuts autonomes, mais il faudra rajouter un statut « brouillon » à mon avis, qu’en penses-tu ?

@Laurent : ce que l’on pourrait faire c’est définir un niveau de droit sur les rédacteurs comme quoi ils ont le droit de publier des évènements orphelins, qu’en penses-tu ?

@RastaPopoulos : j’ai bien noté pour les ID, je vais rajouter ça lors d’un prochain update (j’ai un petit bug à killer).

@les autres : des commentaires supplémentaires ?

Cordialement.

cerdic · Octobre 27, 2016, 4:56

RastaPopoulos a écrit :

Le 24/10/2016 à 15:01, Julien Tessier a écrit :

J'avais déjà contribué au plugin dans le passé donc je ne suis pas
nouveau sur le plugin ^^ J'ai aussi contacté Cédric Morin en parallèle
pour l'avertir.

Oui oui d'où ma phrase "mais peut-être ai-je raté une discussion ?"
(qui aurait été bien ici, enfin je trouve)

Moi aussi j'ai raté la discussion !
(ou alors ça s'est perdu dans les méandres de mon cerveau atteint, les prémices d'un alzheimer ?...)

J'ai pas lu le code, je suis trop loin et mal connecté, donc je ne vais rien dire plutôt que parler en l'air sur le fond.

Mais sur la forme, ça aurait été bien de brancher le trunk sur une branche stable avant d'y insérer ce genre de trucs, pour ne pas casser les sites en production.
(j'ai noté le coup de la constante, mais je sais aussi que ça peut quand même casser des choses, que ce n'est pas un petit chantier potentiellement, car ça va entrainer d'autres commits etc.)

--
Cédric

RealET · Octobre 27, 2016, 5:06

Julien Tessier a écrit le 27/10/2016 à 10:59 :

Bonjour,

Bonjour

@les autres : des commentaires supplémentaires ?

Chez moi ça marche pas™ :

--
RealET

Laurent3 · Octobre 27, 2016, 6:49

ben oui, ce serait super !

Laurent3 · Octobre 31, 2016, 1:51

*Après utilisation, un autre commentaire :*
Logiquement, une autre conséquence de la création d'événements sans article "parent" serait de permettre l'ajout d'événement existant dans les articles. Un peu comme les mots-clés, les auteurs, etc.

Le 27/10/2016 à 20:49, Laurent a écrit :

Le 27/10/2016 à 10:59, Julien Tessier a écrit :

Bonjour,

@YannX : je suis d'accord pour la gestion des statuts autonomes, mais il faudra rajouter un statut "brouillon" à mon avis, qu'en penses-tu ?

@Laurent : ce que l'on pourrait faire c'est définir un niveau de droit sur les rédacteurs comme quoi ils ont le droit de publier des évènements orphelins, qu'en penses-tu ?

ben oui, ce serait super !

@RastaPopoulos : j'ai bien noté pour les ID, je vais rajouter ça lors d'un prochain update (j'ai un petit bug à killer).

@les autres : des commentaires supplémentaires ?

Cordialement.

2016-10-25 22:26 GMT+04:00 Laurent <laurent@yohooo.org <mailto:laurent@yohooo.org>>:

    Merci pour cette initiative qui répond à un besoin récurent !

    je me demande même pourquoi cette fonctionnalité est réservée aux
    admins : il m'arrive de temps en temps de créer des interface
    pour proposer à des auteurs sur inscription de créer des
    événements sur un agenda commun. Je n'ai jusqu'à ce jour, pas
    trouvé d'autre manière de fonctionner que de créer un article
    "fourre tout" pour y déposer l'ensemble des événements entrés par
    les contributeurs.

    Merci !

    Le 24/10/2016 à 20:50, YannX SPIP a écrit :

    Pour info, la modif ne fait rien tant que la constante n'est
    pas définie et c'est bien le but : ne pas changer le
    comportement du plugin tant que les changements ne sont pas
    approuvés. Les utilisateurs n'en sauront rien tant que le mode
    opératoire exact ne sera pas en place.

    Cordialement.

    2016-10-24 15:38 GMT+04:00 RastaPopoulos
    <rastapopoulos@spip.org <mailto:rastapopoulos@spip.org>>:

        Le 24/10/2016 à 13:05, julien@cahri.com
        <mailto:julien@cahri.com> a écrit :

            Log:
            ajout d’une constante pour autoriser la
            création/modification d’évènements orphelins par les
            admins : _AGENDA_AUTORISER_ORPHELINS

    Bravo !
    Cela m'évite de continuer à chercher/inventer la modification
    Et merci...

    Maintenant, va falloir éclaircir la documentation
    (de quel article, entre SPIP 2 et SPIP 3 ?

    -- YannX
    http://www.spippourlesnuls.fr

    ----
    spip-zone@rezo.net <mailto:spip-zone@rezo.net> -http://listes.rezo.net/mailman/listinfo/spip-zone
    <http://listes.rezo.net/mailman/listinfo/spip-zone>

    ---- spip-zone@rezo.net <mailto:spip-zone@rezo.net> -
    http://listes.rezo.net/mailman/listinfo/spip-zone
    <http://listes.rezo.net/mailman/listinfo/spip-zone>

----
spip-zone@rezo.net -http://listes.rezo.net/mailman/listinfo/spip-zone

----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

Laurent3 · Octobre 31, 2016, 2:10

*Encore quelques retours suite à l'activation de _AGENDA_AUTORISER_ORPHELINS, mais cette fois ci, c'est des bugs :*

*1./* Si un événement n'est pas associé à un article lors de sa création, il n'est plus possible de l'associé à un article par la suite.
*2./* Lorsqu'un événement est associé à un article, sa date n'est plus affichée dans édition > événements, ainsi dans cet exemple :

"Un événement du centre culturel" et "troisième événement du centre culturel" sont associés à un article. On ne voit donc plus leur date.

Le 31/10/2016 à 14:51, Laurent a écrit :

*Après utilisation, un autre commentaire :*
Logiquement, une autre conséquence de la création d'événements sans article "parent" serait de permettre l'ajout d'événement existant dans les articles. Un peu comme les mots-clés, les auteurs, etc.

Le 27/10/2016 à 20:49, Laurent a écrit :

Le 27/10/2016 à 10:59, Julien Tessier a écrit :

Bonjour,

@YannX : je suis d'accord pour la gestion des statuts autonomes, mais il faudra rajouter un statut "brouillon" à mon avis, qu'en penses-tu ?

@Laurent : ce que l'on pourrait faire c'est définir un niveau de droit sur les rédacteurs comme quoi ils ont le droit de publier des évènements orphelins, qu'en penses-tu ?

ben oui, ce serait super !

@RastaPopoulos : j'ai bien noté pour les ID, je vais rajouter ça lors d'un prochain update (j'ai un petit bug à killer).

@les autres : des commentaires supplémentaires ?

Cordialement.

2016-10-25 22:26 GMT+04:00 Laurent <laurent@yohooo.org <mailto:laurent@yohooo.org>>:

    Merci pour cette initiative qui répond à un besoin récurent !

    je me demande même pourquoi cette fonctionnalité est réservée
    aux admins : il m'arrive de temps en temps de créer des
    interface pour proposer à des auteurs sur inscription de créer
    des événements sur un agenda commun. Je n'ai jusqu'à ce jour,
    pas trouvé d'autre manière de fonctionner que de créer un
    article "fourre tout" pour y déposer l'ensemble des événements
    entrés par les contributeurs.

    Merci !

    Le 24/10/2016 à 20:50, YannX SPIP a écrit :

    Pour info, la modif ne fait rien tant que la constante n'est
    pas définie et c'est bien le but : ne pas changer le
    comportement du plugin tant que les changements ne sont pas
    approuvés. Les utilisateurs n'en sauront rien tant que le mode
    opératoire exact ne sera pas en place.

    Cordialement.

    2016-10-24 15:38 GMT+04:00 RastaPopoulos
    <rastapopoulos@spip.org <mailto:rastapopoulos@spip.org>>:

        Le 24/10/2016 à 13:05, julien@cahri.com
        <mailto:julien@cahri.com> a écrit :

            Log:
            ajout d’une constante pour autoriser la
            création/modification d’évènements orphelins par les
            admins : _AGENDA_AUTORISER_ORPHELINS

    Bravo !
    Cela m'évite de continuer à chercher/inventer la modification
    Et merci...

    Maintenant, va falloir éclaircir la documentation
    (de quel article, entre SPIP 2 et SPIP 3 ?

    -- YannX
    http://www.spippourlesnuls.fr

    ----
    spip-zone@rezo.net <mailto:spip-zone@rezo.net> -http://listes.rezo.net/mailman/listinfo/spip-zone
    <http://listes.rezo.net/mailman/listinfo/spip-zone>

    ---- spip-zone@rezo.net <mailto:spip-zone@rezo.net> -
    http://listes.rezo.net/mailman/listinfo/spip-zone
    <http://listes.rezo.net/mailman/listinfo/spip-zone>

----
spip-zone@rezo.net -http://listes.rezo.net/mailman/listinfo/spip-zone

----
spip-zone@rezo.net -http://listes.rezo.net/mailman/listinfo/spip-zone

----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

nicod · Octobre 31, 2016, 6:35

Le 31/10/2016 à 15:10, Laurent a écrit :

*Encore quelques retours suite à l'activation de
_AGENDA_AUTORISER_ORPHELINS, mais cette fois ci, c'est des bugs :*

*1./* Si un événement n'est pas associé à un article lors de sa
création, il n'est plus possible de l'associé à un article par la suite.
*2./* Lorsqu'un événement est associé à un article, sa date n'est plus
affichée dans édition > événements, ainsi dans cet exemple :

Cf ce que disait Cedric :

Mais sur la forme, ça aurait été bien de brancher le trunk sur une branche stable avant d'y insérer ce genre de trucs, pour ne pas casser les sites en production.
(j'ai noté le coup de la constante, mais je sais aussi que ça peut quand même casser des choses, que ce n'est pas un petit chantier potentiellement, car ça va entrainer d'autres commits etc.)

--
nicod_

cerdic · Novembre 1, 2016, 10:37

Laurent a écrit :

*Encore quelques retours suite à l'activation de
_AGENDA_AUTORISER_ORPHELINS, mais cette fois ci, c'est des bugs :*

*2./* Lorsqu'un événement est associé à un article, sa date n'est plus
affichée dans édition > événements, ainsi dans cet exemple :

ça n'est pas un bug, c'est la même chose si tous les événements sont associés à un article : on affiche qu'une seule fois chaque date de la colonne de gauche, pour le premier événement à cette date.
La date exacte de chaque événement est affichée dans la colonne sous le titre

Cédric

cerdic · Novembre 1, 2016, 10:40

Hello,

je viens donc de créer une branche de maintenance pour la v3, qui ne contient pas ces modifications, et j'ai passé le trunk, qui contient ces modifications, en v4.0.0-dev

J'ai jeté un oeil rapide, et je pense que ces seuls commits ne peuvent suffire à traiter le problème dans son intégralité.
Notamment, il faut gérer la question du statut, des déplacements et rattachements...

Je te laisse poursuivre le chantier sur le trunk, je n'ai pas trop de temps pour regarder de plus près pour le moment.

Merci, et bon courage !

--
Cédric

Julien Tessier a écrit :

Bonjour,

J'avais déjà contribué au plugin dans le passé donc je ne suis pas
nouveau sur le plugin ^^ J'ai aussi contacté Cédric Morin en parallèle
pour l'avertir.

De plus, je pensais avoir respecté un des principes de la zone :

/Ya des trucs spécifiques à SPIP : /
/- « on code d’abord, on discute après » (pour donner du coeur à
l’ouvrage et éviter les discussions stériles) /
/- « gogogo » ! (version applicative du précédent précepte)/

Du coup, on peut discuter maintenant Tu verras le commit : je me base
sur des nouvelles autorisations qui permettent de changer le
comportement éventuel de qui peut créer/modifier/visualiser/instituer
des articles orphelins selon ce que la communauté pense être le mieux.

Pour info, la modif ne fait rien tant que la constante n'est pas définie
et c'est bien le but : ne pas changer le comportement du plugin tant que
les changements ne sont pas approuvés. Les utilisateurs n'en sauront
rien tant que le mode opératoire exact ne sera pas en place.

Cordialement.

2016-10-24 15:38 GMT+04:00 RastaPopoulos <rastapopoulos@spip.org
<mailto:rastapopoulos@spip.org>>:

    Le 24/10/2016 à 13:05, julien@cahri.com <mailto:julien@cahri.com> a
    écrit :

        Log:
        ajout d’une constante pour autoriser la création/modification
        d’évènements orphelins par les admins : _AGENDA_AUTORISER_ORPHELINS

    Euuuuh, ça aurait peut-être été bien d'en parler non ?

    C'est un truc voulu depuis longtemps ok, mais là tel quel, ça
    introduit du code qui est amené à devenir pérenne (puisque ajoutés
    dans le plugin central directement), donc est-ce que c'est comme ça
    qu'on veut faire ?

    Un avis des mainteneurs du plugin avant d'inscrire dans le marbre
    telle ou telle manière de faire, c'est peut-être pas mal ? (Mais
    peut-être ai-je raté une discussion ?)

    --
    RastaPopoulos

    ----
    spip-zone@rezo.net <mailto:spip-zone@rezo.net> -
    http://listes.rezo.net/mailman/listinfo/spip-zone
    <http://listes.rezo.net/mailman/listinfo/spip-zone>

----
spip-zone@rezo.net - http://listes.rezo.net/mailman/listinfo/spip-zone

jeanmarie · Décembre 9, 2016, 2:44

Salut,

@les autres : des commentaires supplémentaires ?

merci pour cette nouvelle fonctionnalité, je l'attendais depuis... longtemps

Juste un truc : dans un squelette sous forme de plugin, lorsque je définis _AGENDA_AUTORISER_ORPHELINS dans le fichier prefix_options.php à la racine de mon plugin, ça ne fonctionne pas, je dois impérativement le placer dans le dossier config.

Pour info, je suis sur un SPIP mutualisé (plugin Mutualisation facile), je ne sais pas si ça joue...

Bon week-end,

jean marie