Une rubrique est considérée comme publiée si elle a un logo #4424

Closed
opened 2 years ago by nicod_ · 6 comments
nicod_ commented 2 years ago
Owner

Une rubrique sans article passe au statut "publie" dès qu'on lui ajoute un logo (qui sont maintenant des documents).

Je pense que c'est à la fois côté SPIP, qui compte tous les enfants d'une rubrique, et du côté du plugin médias, qui renvoie le nombre de documents d'une rubrique qu'il faut intervenir.

Mais medias_objet_compte_enfants() renvoie juste un total (sql_countsel), du coup il faudrait peut être lui passer un paramètre pour qu'elle exclue les logos dans certains cas ?

Une rubrique sans article passe au statut "publie" dès qu'on lui ajoute un logo (qui sont maintenant des documents). Je pense que c'est à la fois côté SPIP, qui compte tous les enfants d'une rubrique, et du côté du plugin médias, qui renvoie le nombre de documents d'une rubrique qu'il faut intervenir. Mais medias_objet_compte_enfants() renvoie juste un total (sql_countsel), du coup il faudrait peut être lui passer un paramètre pour qu'elle exclue les logos dans certains cas ?
Poster
Owner

J'ai un patch qui corrige (chez moi en tout cas), à voir si c'est suffisant.

J'ai un patch qui corrige (chez moi en tout cas), à voir si c'est suffisant.
b_b commented 2 years ago
Owner

Paye ta PR mignon ;)
Statut changé à En cours

Paye ta PR mignon ;) **Statut changé à En cours**
Poster
Owner

Quoi, plus personne sait lire un patch ou quoi ? ^^

Quoi, plus personne sait lire un patch ou quoi ? ^^
b_b commented 2 years ago
Owner

Lu et validé, d'où la demande de PR, je ne vais pas le commiter pour toi :p

Lu et validé, d'où la demande de PR, je ne vais pas le commiter pour toi :p
Poster
Owner

C'est commité et pushé.
Statut changé à Résolu

C'est commité et pushé. **Statut changé à Résolu**
b_b commented 2 years ago
Owner

Pour faire le lien (car tu as oublié de mentionner #4424 dans tes commits) :

https://core.spip.net/projects/spip/repository/revisions/24480
https://zone.spip.org/trac/spip-zone/changeset/119908/

Et statut Fermé + résolution fixed ;)
Statut changé à Fermé

Pour faire le lien (car tu as oublié de mentionner #4424 dans tes commits) : https://core.spip.net/projects/spip/repository/revisions/24480 https://zone.spip.org/trac/spip-zone/changeset/119908/ Et statut Fermé + résolution fixed ;) **Statut changé à Fermé**
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.