Dotclear

Ticket #472 (closed enhancement: fixed)

Opened 15 years ago

Last modified 10 years ago

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

patch_without_empty.diff Download (1.5 KB) - added by Moe 15 years ago.
Patch simple
patch_with_empty.diff Download (1.6 KB) - added by Moe 15 years ago.
Remplacement de without_empty par with_empty
patch-empty-cat.txt Download (2.4 KB) - added by biou 14 years ago.
patch empty cat

Change History

Changed 15 years ago by Moe

Patch simple

Changed 15 years ago by Moe

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:3 Changed 15 years ago by xave

Pour moi, ça serait plutôt du contenant.

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:5 Changed 15 years ago by xave

De la catégorie, c'est du contenant pour des billets.

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:9 Changed 15 years ago by olivier

Si c'est fait, ça devient le comportement par défaut.

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

Changed 14 years ago by biou

patch empty cat

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:17 Changed 13 years ago by xave

  • Milestone changed from 2.2 to 2.3

comment:18 Changed 12 years ago by bruno

  • Priority changed from lowest to normal
  • Milestone 2.3 deleted

comment:19 Changed 11 years ago by franck

  • Milestone set to A definir

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:21 Changed 10 years ago by franck

  • Milestone changed from A definir to 2.6

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

Note: See TracTickets for help on using tickets.

Sites map