dev_publier_rubrique Publier une rubrique lorsqu'on lui assosie au moins un contact ou une organisation #11
Open
Yohooo
wants to merge 6 commits from dev_publier_rubrique into master
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'dev_publier_rubrique'
Deleting a branch is permanent. It CANNOT be undone. Continue?
Coucou,
Petite demande d'ajout de fonctionnalités qui permettent de publier des rubriques quand il y a des contacts ou organisations dedans, et de la dépublier dans le cas contraire.
Voili
Il y a un grand flou sur la "vraie", ou meilleure, ou plus complète manière de faire ça. Normalement il devrait y avoir le pipeline "objet_compte_enfants" aussi, et ensuite quelle fonction doit être appelée c'est un mystère yen a plein possible. Cf cette discussion, et les questions en bas :
spip-contrib-extensions/albums#11
Rôlala ! Tu m'a perdu... Pas sûr que je trouve le temps de faire tout ça (et que je me trouve une légitimité pour arbitrer sur une méthode qui convienne à tout le monde)...
JLuc referenced this pull request from spip-contrib-extensions/albums 4 months agoPour rester simple, il faut créer une instance du pipeline
calculer_rubriquesCf https://programmer.spip.net/calculer_rubriques-1009
Bonjour,
J'avais bien créé une instance
calculer_rubriquesElle fonctionnait pour les organisations. Elle fonctionne maintenant pour les contacts.Je pense que l'on peut fusionner.
En quoi ça "reste simple" @jluc ? On a "documenté" plus haut, avec le lien dans albums qu'il y a AUSSI le pipeline "objet_compte_enfants" qui sert aussi explicitement à ça, et que c'est absolument pas clair pourquoi des fois faut l'un des fois faut l'autre, qu'il n'y a aucune vraie doc d'à quoi ça sert et pourquoi ya les deux.
Donc bah non c'est toujours pas très clair pour moi, et il devrait y avoir le "objet_comptes_enfants" d'implémenté aussi (ou pas ? mais pourquoi alors ?)
Et résumé plus précisément dans ce commentaire, où je fais le tour (des pipelines et aussi des fonctions de recalcul où là aussi c'est le bordel, yen a 3) : spip-contrib-extensions/albums#11
Sachant comme dit dedans que le plugin Médias de la dist implémente bien les deux.
@Rastapopoulos il y a 2 problèmes différents :
C'est le titre de ce ticket et c'est ça qui est simple. Il faut probablement valider cette PR qui semble solutionner ce problème de la bonne manière puisque de manière cohérente avec le reste du plugin (= pareil que pour les organisations).
Je n'ai pas étudié ce sujet, mais au vu de tes explications ça mériterait certainement un ticket, pas dans un plugin comme C&O ou Album, mais dans le core.
Bon benh du coup, dès le feu vert de @rastapopoulos, je fusionne.
if($flux["args"]["table_lien"] == "spip_organisations_liens" && $flux["args"]["objet_source"] == "organisation"&& $flux["args"]["objet"] == "rubrique" ){include_spip('inc/rubriques');calculer_rubriques($flux);Cette fonction n'attend absolument pas d'argument : https://git.spip.net/spip/spip/src/branch/master/ecrire/inc/rubriques.php#L273
Par ailleurs, pourquoi demander un recalculer de toutes les rubriques du monde (il peut y en avoir des centaines, des milliers, dans le site), alors qu'à cet endroit là, on connait LA rubrique (ou sa branche mais juste autour d'elle quoi) qu'on veut recalculer ?
Pour l'argument, c'est modifié.
Par contre, pour le calcul d'une seule rubrique, tu me demandes de créer une fonction calculer_rubrique($id_rubrique) qui prend en compte tous les pipelines calculer_rubrique (par exemple, celui ajouté par medias).
Ce serait interressant mais je pense que ce type de focntion a sa place dans le core. Et une fois dispo, on pourrait l'utiliser dans ce type de plugin.
Non ?
Pour une rubrique unique une fonction existe déjà dans le core, quoique la nomenclature m'échappe, c'est calculer_rubriques_if() : https://code.spip.net/spip/namespaces/default.html#function_calculer_rubriques_if
Et oui toutes ces fonctions mériteraient des clarifications.