De mémoire, moi j'étais plutôt contre l'introduction de nouvelles
constantes dans ce plugin, c'était en réponse à un besoin assez
spécifique si je me souviens bien, qui aurait sans doute put être réglé
avec une simple surcharge.
Quoiqu'il en soit, est-ce que tu peux au moins corriger la notice ? Merci.
Le 27/06/2018 à 16:07, spip-zone-commit@rezo.net a écrit :
Author: jeanmarie.listes@cousumain.info
Date: 2018-06-27 14:07:26 +0000 (Wed, 27 Jun 2018)
New Revision: 110901
Modified:
_plugins_/metasplus/trunk/inclure/metasplus/dist.html
_plugins_/metasplus/trunk/paquet.xml
Log:
V2.1.4: Ajout d'une constante _METASPLUS_MASQUER_AUTEURS pour masquer les auteurs des articles et afficher le #NOM_SITE_SPIP à la place
De mémoire, moi j'étais plutôt contre l'introduction de nouvelles
constantes dans ce plugin, c'était en réponse à un besoin assez
spécifique si je me souviens bien, qui aurait sans doute put être réglé
avec une simple surcharge.
Quoiqu'il en soit, est-ce que tu peux au moins corriger la notice ? Merci.
Un peu de même, si on se met à faire des constantes et des tests à tout
va pour le moindre morceau, je trouve ça moyen bof.
Le plugin est déjà surchargeable pour chaque site : à la fois le dist
commun, et ensuite par type de contenu. Ça laisse déjà beaucoup de
liberté pour chacun.
Soit ya des trucs qui sont clairement configurable, et pour ce cas c'est
pas technique, ça m'a bien l'air d'un choix éditorial : "est-ce que je
veux que mes auteurs soient visibles de l'extérieur ou pas ?", ça peut
parfaitement être un choix décidé par les admins, sans aucune compétence
technique. Soit c'est que pour les techniciens et on peut déjà
surcharger plein de choses.
Là si jamais ça doit être configurable, ça serait mieux en vrai config
avec interface non ?
Oui à ce compte là, autant ajouter une case à cocher dans la config.
Le 12/01/2019 à 20:20, RastaPopoulos a écrit :
Un peu de même, si on se met à faire des constantes et des tests à tout
va pour le moindre morceau, je trouve ça moyen bof.
Le plugin est déjà surchargeable pour chaque site : à la fois le dist
commun, et ensuite par type de contenu. Ça laisse déjà beaucoup de
liberté pour chacun.
Soit ya des trucs qui sont clairement configurable, et pour ce cas c'est
pas technique, ça m'a bien l'air d'un choix éditorial : "est-ce que je
veux que mes auteurs soient visibles de l'extérieur ou pas ?", ça peut
parfaitement être un choix décidé par les admins, sans aucune compétence
technique. Soit c'est que pour les techniciens et on peut déjà
surcharger plein de choses.
Là si jamais ça doit être configurable, ça serait mieux en vrai config
avec interface non ?
Oui à ce compte là, autant ajouter une case à cocher dans la config.
et sinon, et pour info générale sur cette notice,
il suffirait de remplacer #EVAL par #CONST pour qu’il n’y ait plus de notice
(ainsi que j'ai commenté sur le forum)
pour ce qui est du besoin, je n'ai pas l'impression qu'il soit si spécifique (dans le sens peu courant) : les auteurs d'un article n'ont pas forcément vocation à être visibles. Par ex sur le site d'une structure où il n'est pas pertinent (voir même pas logique) de savoir que c'est Machine ou Machin qui a créé l'actu ou l'article alors que c'est la structure qui s'exprime.
La discussion à l'époque : https://www.mail-archive.com/spip-zone@rezo.net/msg46002.html
Pour ce qui est de la constante vs config, c'était dans l'idée de garder la page de config la plus simple possible (éviter le mode tableau de bord d'avion de chasse à terme) mais pourquoi pas...
Du coup, si j'ajoute une case dans la config, on vire complètement la constante ? Ça ne risque pas de poser des problèmes pour les sites l'utilisant (s'il y en a). Bon, le plug était en test jusqu'à présent...
Et #CONST n'est dispo qu'à partir de 3.2 alors que le plugin est 3.1
jean marie
Le 12/01/2019 à 20:50, Charles Razack a écrit :
Oui à ce compte là, autant ajouter une case à cocher dans la config.
Le 12/01/2019 à 20:20, RastaPopoulos a écrit :
Un peu de même, si on se met à faire des constantes et des tests à tout
va pour le moindre morceau, je trouve ça moyen bof.
Le plugin est déjà surchargeable pour chaque site : à la fois le dist
commun, et ensuite par type de contenu. Ça laisse déjà beaucoup de
liberté pour chacun.
Soit ya des trucs qui sont clairement configurable, et pour ce cas c'est
pas technique, ça m'a bien l'air d'un choix éditorial : "est-ce que je
veux que mes auteurs soient visibles de l'extérieur ou pas ?", ça peut
parfaitement être un choix décidé par les admins, sans aucune compétence
technique. Soit c'est que pour les techniciens et on peut déjà
surcharger plein de choses.
Là si jamais ça doit être configurable, ça serait mieux en vrai config
avec interface non ?
Oui tu as raison, le besoin me semble assez courant pour être ajouté
dans la config.
Je serais d'avis de virer la constante quitte à ajouter un message
d'avertissement dans la doc pour les quelques personnes qui l'utiliseraient.
Quant à la constante qui exclue des groupes de mots-clés, je suis plus
circonspect.
Il y a déjà un plugin qui sert à exclure des mots technique
automatiquement partout, on va pas réimplémenter la fonctionnalité dans
chaque plugin.
Donc celle là, je préfererais qu'elle soit retirée (sans la passer en
config).
Pour la balise #CONST, on peut faire aussi [(#VAL{machin}|constant)] et
ça fait pareil, sans notif si pas définie.
Le 14/01/2019 à 14:25, Jean Marie Grall a écrit :
Salut,
pour ce qui est du besoin, je n'ai pas l'impression qu'il soit si
spécifique (dans le sens peu courant) : les auteurs d'un article n'ont
pas forcément vocation à être visibles. Par ex sur le site d'une
structure où il n'est pas pertinent (voir même pas logique) de savoir
que c'est Machine ou Machin qui a créé l'actu ou l'article alors que
c'est la structure qui s'exprime.
La discussion à l'époque : https://www.mail-archive.com/spip-zone@rezo.net/msg46002.html
Pour ce qui est de la constante vs config, c'était dans l'idée de
garder la page de config la plus simple possible (éviter le mode
tableau de bord d'avion de chasse à terme) mais pourquoi pas...
Du coup, si j'ajoute une case dans la config, on vire complètement la
constante ? Ça ne risque pas de poser des problèmes pour les sites
l'utilisant (s'il y en a). Bon, le plug était en test jusqu'à présent...
Et #CONST n'est dispo qu'à partir de 3.2 alors que le plugin est 3.1
Pour ce qui est de la constante vs config, c'était dans l'idée de garder
la page de config la plus simple possible (éviter le mode tableau de
bord d'avion de chasse à terme) mais pourquoi pas...
La page de configuration actuel de Metas+ est actuellement très loin de
l'états "tableau de bord d'avions de chasse". Et quand bien même, je
préfère un tableau de bord d'avions de chasse bien expliqué et bien
rangé qu'une constante (buggée qui plus est).
C'est clairement le genre de plugin qui mérite un tableau de bord
d'avion de chasse pour un maximum de flexibilité.
Jean Marie Grall <jeanmarie.listes@cousumain.info> writes:
Salut,
pour ce qui est du besoin, je n'ai pas l'impression qu'il soit si
spécifique (dans le sens peu courant) : les auteurs d'un article n'ont
pas forcément vocation à être visibles. Par ex sur le site d'une
structure où il n'est pas pertinent (voir même pas logique) de savoir
que c'est Machine ou Machin qui a créé l'actu ou l'article alors que
c'est la structure qui s'exprime.
La discussion à l'époque : https://www.mail-archive.com/spip-zone@rezo.net/msg46002.html
Pour ce qui est de la constante vs config, c'était dans l'idée de garder
la page de config la plus simple possible (éviter le mode tableau de
bord d'avion de chasse à terme) mais pourquoi pas...
Du coup, si j'ajoute une case dans la config, on vire complètement la
constante ? Ça ne risque pas de poser des problèmes pour les sites
l'utilisant (s'il y en a). Bon, le plug était en test jusqu'à présent...
Et #CONST n'est dispo qu'à partir de 3.2 alors que le plugin est 3.1
jean marie
Le 12/01/2019 à 20:50, Charles Razack a écrit:
Oui à ce compte là, autant ajouter une case à cocher dans la config.
Le 12/01/2019 à 20:20, RastaPopoulos a écrit:
Un peu de même, si on se met à faire des constantes et des tests à tout
va pour le moindre morceau, je trouve ça moyen bof.
Le plugin est déjà surchargeable pour chaque site : à la fois le dist
commun, et ensuite par type de contenu. Ça laisse déjà beaucoup de
liberté pour chacun.
Soit ya des trucs qui sont clairement configurable, et pour ce cas c'est
pas technique, ça m'a bien l'air d'un choix éditorial : "est-ce que je
veux que mes auteurs soient visibles de l'extérieur ou pas ?", ça peut
parfaitement être un choix décidé par les admins, sans aucune compétence
technique. Soit c'est que pour les techniciens et on peut déjà
surcharger plein de choses.
Là si jamais ça doit être configurable, ça serait mieux en vrai config
avec interface non ?
Ok pour remplacer _METASPLUS_MASQUER_AUTEURS par une config, je regarde ça rapidement…
Et oui, pour _METASPLUS_MASQUER_MOTSCLEFS, j’aurais dû faire un revert, j’ai zappé (cf ). Par contre, je n’ai jamais fait ça, comment ça se passe ? (avec Tortoise SVN)
