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
Yohooo commented 4 months ago
Collaborator

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

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
Yohooo added 3 commits 4 months ago

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

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 : https://git.spip.net/spip-contrib-extensions/albums/issues/11
Poster
Collaborator

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)...

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)...
Collaborator

Pour rester simple, il faut créer une instance du pipeline calculer_rubriques
Cf https://programmer.spip.net/calculer_rubriques-1009

Pour rester simple, il faut créer une instance du pipeline `calculer_rubriques` Cf https://programmer.spip.net/calculer_rubriques-1009
Yohooo added 1 commit 3 months ago
Yohooo added 1 commit 3 months ago
Poster
Collaborator

Bonjour,

J'avais bien créé une instance calculer_rubriques Elle fonctionnait pour les organisations. Elle fonctionne maintenant pour les contacts.

Je pense que l'on peut fusionner.

Bonjour, J'avais bien créé une instance `calculer_rubriques` Elle 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 ?)

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.

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) : https://git.spip.net/spip-contrib-extensions/albums/issues/11#issuecomment-42646 Sachant comme dit dedans que le plugin Médias de la dist implémente bien *les deux*.
Collaborator

@Rastapopoulos il y a 2 problèmes différents :

  1. Faire en sorte qu'une rubrique soit publiée lorsqu'on lui associe au moins un contact ou une organisation.

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).

  1. Améliorer spip et sa documentation et corriger les ambiguités que tu décris.

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.

@Rastapopoulos il y a 2 problèmes différents : 1) Faire en sorte qu'une rubrique soit publiée lorsqu'on lui associe au moins un contact ou une organisation. 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). 2) Améliorer spip et sa documentation et corriger les ambiguités que tu décris. 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.
Poster
Collaborator

Bon benh du coup, dès le feu vert de @rastapopoulos, je fusionne.

Bon benh du coup, dès le feu vert de @rastapopoulos, je fusionne.
rastapopoulos reviewed 3 months ago
if($flux["args"]["table_lien"] == "spip_organisations_liens" && $flux["args"]["objet_source"] == "organisation"
&& $flux["args"]["objet"] == "rubrique" ){
include_spip('inc/rubriques');
calculer_rubriques($flux);
Owner

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 ?

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 ?
Poster
Collaborator

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 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 ?
Owner

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.

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.
Yohooo added 1 commit 3 months ago
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.