Re: [Spip-zone-commit] r114271 - in _core_/plugins/mots

Le 03/03/2019 à 02:48, Cerdic a écrit :
> Hello Touti,
>
> je comprend bien l’objectif poursuivi, mais ça semble un peu brutal et
> pas très satisfaisant de forker les squelettes listes
> articles/rubriques, car du coup ceux qui les ont personnalisés perdent
> des morceaux, et cela pose aussi plusieurs questions de généricité :
>
> Ce que tu fais ici pour les mots, faut il le faire pour tout objet qui
> vient enrichir les articles ?
> Et dans l’autre sens tu as changé le nommage générique en demandant
> donc une liste {table}-mot pour les autres objets ne faisant pas
> partie du core.
> Donc on casse plein de compat sur les objets perso, mais au delà est
> il raisonnable de se retrouver à dupliquer des listes ad-nauseam partout ?

ce n'est pas l'idéal, je le reconnais et c'était effectivement le plus
simple à coder. Celleux qui ont personnalisés ces 2 listes sauront
surement s'en sortir, sachant que modifier l'espace privé n'est pas une
recommandation et empêche même les évolutions. Pour le nommage générique
on ne casse pas la compat, voir dans la ligne suivante il y a toujours
liste-objets dont parle Nicod_ qui permet de lister les
liste-patate.html que l'on ajouterai par des plugins de ce mot, merci de
regarder le code. J'ai dû garder les appels à listes articles et
rubriques et les mettre en dupli car ils ne sont pas générique par
exemple avec une variable supplémentaire session de l'auteur pour
distinguer les autorisations.

>
> Pour le moment c’était resté comme ça car il n’a jamais été considéré
> comme critique de se débarasser de ce {id_mot?} dans ces listes, mais
> si on veut vraiment se débarasser de la dépendance je pense qu’on doit
> faire mieux que ça en évitant tous ces dédoublement.
Bonne idée !
>
> Une solution à laquelle je pense serait de faire un critère spécifique
> genre {selection_conditionnelle} (nomenklatura si tu as mieux on prend
> !) qui regarderait les id_xxx connus dans l’environnement et pour
> chaque chercherait une fonction du type
> inc_generer_condition_selection_{id_xx}_dist qu’on appelerait sur le mode
> $generer_condition_selection =
> charger_fonction(‘generer_condition_selection_’.$primary,’inc’);
> $boucle->where[] =
> $generer_condition_selection($type_boucle,champ_sql($primary));
>
> a charge pour chaque plugin de déclarer la fonction de selection
> correspondante et d’y gerer tous les cas (ici si pas de
> $_PILE[0][‘id_mot’] ça renvoie un 1=1)
>
> Bref, c’est juste les grandes lignes, il y a surement des problèmes à
> résoudre et de la mise au point à faire, mais ça me semble une
> direction plus générique et souhaitable pour à la fois résoudre la
> dépendance de ces listes d’un plugin en particulier et éviter de faire
> 36 copies dans tous les coins de la zone…
> (en évitant de la rupture de compat, puisque les surcharges
> continueraient de fonctionner, de même que les listes existantes
> d'autres objets éditoriaux)

voir au-dessus pour la non rupture, il n'y a pratiquement que le
{id_mot?} de ces listes /critère uniquement utilisé dans la page d'un
mot/ qui crée un warning sur l'espace privé lorsqu'on ne souhaite pas
activer les plugins-dist et obtenir un SPIP-light. Le prix d'une infâme
dupli n'est pas si terrible :-)

++

touti


>
> Bises
>
> -- 
> Cédric
> Le 2 mars 2019 à 19:21 +0100, [email protected], a écrit :
>> Author: [email protected]
>> Date: 2019-03-02 18:21:41 +0000 (Sat, 02 Mar 2019)
>> New Revision: 114271
>>
>> Added:
>> _core_/plugins/mots/prive/objets/liste/articles-mot.html
>> _core_/plugins/mots/prive/objets/liste/articles-mot_fonctions.php
>> _core_/plugins/mots/prive/objets/liste/rubriques-mot.html
>> Modified:
>> _core_/plugins/mots/paquet.xml
>> _core_/plugins/mots/prive/squelettes/contenu/mot.html
>> Log:
>> Création des fichiers qui listent les articles et les rubriques d'un mot
>> dans l'espace privé.
>>
>> Duplication de ceux du core, deviennent articles-mot et rubriques-mot.
>>
>>
>> Details: https://zone.spip.org/trac/spip-zone/changeset/114271
>>
>> _______________________________________________
>> [email protected] -
>> https://listes.rezo.net/mailman/listinfo/spip-zone-commit