Ticket #472 (closed enhancement: fixed)
Afficher les catégories vides avec <tpl:Categories>
Reported by: | Moe | Owned by: | franck |
---|---|---|---|
Priority: | normal | Milestone: | 2.6 |
Component: | module:public | Version: | 2.0 |
Severity: | normal | Keywords: | |
Cc: |
Description
Bonjour,
voici un patch pour que l'on puisse afficher les catégories vides dans les templates avec le code suivant :
<tpl:Categories without_empty="0"> {{tpl:CategoryTitle encode_html="1"}} </tpl:Categories>
Le second patch renomme without_empty en with_empty pour simplifier le code. Il faut alors utiliser le paramètre with_empty="1".
Attachments
Change History
Changed 15 years ago by Moe
-
attachment
patch_without_empty.diff
added
Changed 15 years ago by Moe
-
attachment
patch_with_empty.diff
added
Remplacement de without_empty par with_empty
comment:1 Changed 15 years ago by olivier
- Status changed from new to closed
- Resolution set to wontfix
C'est n'est pas souhaitable dans le mesure où ça conduira à des erreurs 404 si on fait un lien vers les-dites catégories.
comment:2 Changed 15 years ago by biou
- Status changed from closed to reopened
- Resolution wontfix deleted
Je vais aussi avoir besoin de ce type de fonctionnement. En supposant qu'une catégorie soit créée avec un nom et une description, on devrait pouvoir afficher ce genre d'info (c'est du contenu après tout) et ne pas tomber sur une 404...
comment:4 Changed 15 years ago by biou
du texte dans un champ description c'est du contenant ? j'avoue que je ne comprends pas bien...
comment:6 Changed 15 years ago by biou
ok ça c'est ton point de vue :) mais une catégorie a aussi une vie propre, vu qu'elle a un nom et une description. C'est un concept à part entière. On a une url pour une catégorie, et coté backend on a même une table qui gère des catégories. A partir du moment qu'on a créé une catégorie dans le logiciel, qu'une url a été déterminée, je ne vois pas pourquoi on ne pourrait pas l'afficher. Entre autres, ceci viole les principes basiques des architectures REST.
Je ne pense pas qu'après la nouvelle gestion des catégories qu'Olivier vient de mettre en place, on puisse garder des vestiges de l'ancien fonctionnement.
J'utilise toujours dotclear sur des utilisateurs "de base" (i.e. qui ne sont pas intéressés par la technologie) et la première chose qu'on m'a demandé lorsque j'ai présenté les catégories et sous catégories, c'était un moyen d'afficher les catégories vides.
Pour l'utilisateur lambda, le nouveau mécanisme permet d'afficher une arborescence de site (on s'en fout de savoir que ce sont des catégories) A chaque noeud de cette arborescence, on a une page de contenu. Si on veut on peut y ranger des posts, mais ce n'est pas une obligation.
La nouvelle gestion des catégories permet en gros de dessiner un plan du site. Si on le dessine du côté admin, et que du côté public on n'obtienne pas la même chose, ceci est clairement déroutant.
Je ne dis pas que ce comportement doit être par défaut, mais le patch de Moe me semblait sympa.
comment:7 Changed 15 years ago by kozlika
Il me semble qu'on peut considérer que la description d'une catégorie lui donne un contenu, même si ce contenu n'est pas des billets. La page pourrait simplement afficher dans son content "cette catégorie ne contient pas de billets".
Par exemple, pour faire du teasing et que les gens se préabonnent au flux rss, ou autres utilisations que je n'ai pas en tête.
Par ailleurs, ça pourrait être pratique pour une catégorie ne comprenant que des billets avec mot de passe, de façon à la "promouvoir" sans la dévoiler : "si vous voulez lire les billets de cette catégorie, blah blah".
comment:8 Changed 15 years ago by biou
juste pour revenir sur le 404, en supposant qu'on reste là dessus, l'erreur ne me paraît pas appropriée.
Si la catégorie n'existe pas, ok 404. Si la catégorie existe, mais on ne veut pas l'afficher parce qu'elle est vide, une erreur 405 ou une 403 me paraissent plus appropriées.
comment:10 Changed 15 years ago by olivier
- Milestone 2.1 deleted
On peut aussi ajouter un status de publication sur la catégorie. C'est la solution la plus propre, avec le problème qu'à 36h de la 2.1 ce ne sera pas pour celle-ci :)
comment:11 Changed 15 years ago by olivier
Mauvaise idée :) Je vais appliquer le patch tel quel et on verra.
comment:12 Changed 15 years ago by olivier
Ton patch ne fonctionne pas puisque la page de la catégorie ne s'affiche que si elle possède des billets. Ça m'embête un peu de faire passer ça dans la 2.1
comment:13 Changed 15 years ago by olivier
- Milestone set to 2.2
Ah et puis, sans oublier l'affichage à gérer dans la liste des enfants...
2.2 définitivement
comment:14 Changed 14 years ago by biou
j'ai ajouté un patch, complémentaire de celui de moe (j'avais oublié qu'il existait...) Finalement en relisant les commentaires, je me dis que ce n'est pas encore prêt. Il faudrait voir ce que l'on fait vraiment : status de publication sur une catégorie ? Je n'ai pas compris la question de l'affichage à gérer dans la liste des enfants...
comment:15 Changed 13 years ago by xave
Patch et contre-patch, moi je m'y perds un peu. Qui me fait un truc propre ?
comment:16 Changed 13 years ago by xave
Afficher les catégories vides, c'est sympa, mais en l'état, les pages de catégories vont renvoyer une 404. Plutôt qu'une modif des templates, il faudrait d'abord penser à un réglage par blog.
comment:18 Changed 12 years ago by bruno
- Priority changed from lowest to normal
- Milestone 2.3 deleted
comment:20 Changed 10 years ago by franck
Est-ce qu'il ne suffirait pas que dans le cas de catégorie sans contenu (ie sans billet) on ne mette aucune URL ?
J'ai l'impression que c'est le seul point bloquant, non ?
comment:22 Changed 10 years ago by franck
- Owner changed from olivier to franck
- Status changed from reopened to new
comment:23 Changed 10 years ago by franck <carnet.franck.paul@…>
- Status changed from new to closed
- Resolution set to fixed
(In [2d5ebb2fcdfd]) Fixes #472, <tpl:Categories> (new without_empty="0" attribute) and categories' widget ("Include empty categories" checkbox) may includes empty categories.
comment:24 Changed 10 years ago by franck <carnet.franck.paul@…>
(In [1c463226bd1c]) Replace without_empty attribute of <tpl:Categories> by with_empty, avoiding double-negation. Addresses #472
Patch simple