Dotclear

Ticket #2190 (closed enhancement: fixed)

Opened 8 years ago

Last modified 8 years ago

[tpl:Entries] Mauvaise gestion de no_category ?

Reported by: Welsh Owned by: team
Priority: normal Milestone: 2.10
Component: module:core Version: 2.9.1
Severity: normal Keywords:
Cc:

Description

Pour ne pas retrouver ces billets dans le reste de la page, je fais une boucle avec tpl:Entries et no_category="0", sous-entendu ayant une catégorie.

Or cela ne fonctionne pas.

Dans class.dc.template.php, ligne 1143, on a :

if (isset($attr['no_category']) && $attr['no_category']) {
			$p .= "@\$params['sql'] .= ' AND P.cat_id IS NULL ';\n";
			$p .= "unset(\$params['cat_url']);\n";
		}

De la sorte, si no_cateogry est positionné à 1, on filtre sur cat_id NULL. Si no_category est positionné à 0, on ne filtre pas au lieu de filtrer sur cat_id non NULL.

Change History

comment:1 Changed 8 years ago by franck

  • Type changed from defect to enhancement

comment:2 Changed 8 years ago by franck

Effectivement l'attribut no_category n'a pas été prévu pour ce cas particulier, mais on peut l'envisager (c'est la raison pour laquelle ce ticket est passé de « defect » à « enhancement ».

comment:3 Changed 8 years ago by Welsh

Merci pour les changements. Je ne savais pas bien...
Quant au fonctionnement cible, la documentation ( https://fr.dotclear.org/documentation/2.0/resources/themes/tags/entries) indique :

Cet attribut permet d'obtenir les billets n'ayant aucune catégorie. Sa valeur est 0 ou 1. 

Le fait que deux valeurs soient proposées fait penser à un fonctionnement en toggle.

comment:4 follow-up: ↓ 7 Changed 8 years ago by franck

Je pense qu'utiliser

no_category = "0"

est difficile à comprendre à la première lecture.

Du coup, plutôt que de modifier et attribut — ce qui au passage permet de ne pas casser ceux qui l'utilisent tel que prévu — je vais ajouter un autre attribut :

with_category = "1"

Fonctionnant sur le même principe.

comment:5 Changed 8 years ago by franck <carnet.franck.paul@…>

  • Status changed from new to closed
  • Resolution set to fixed

(In [5a42f6cb893f]) Add with_category attribute for tpl:Entries — fixes #2190

comment:6 Changed 8 years ago by franck

  • Milestone changed from A definir to 2.10

comment:7 in reply to: ↑ 4 Changed 8 years ago by Welsh

Replying to franck:

Je pense qu'utiliser

no_category = "0"

est difficile à comprendre à la première lecture.

Pas faux. Dans ce cas, il faut alors enlever de la documentation la possibilité de valoriser à 0. Non ?

Replying to franck:

Du coup, plutôt que de modifier et attribut — ce qui au passage permet de ne pas casser ceux qui l'utilisent tel que prévu — je vais ajouter un autre attribut :

with_category = "1"

Fonctionnant sur le même principe.

Bien vu. Et je pense la même chose sur la doc.

comment:8 follow-up: ↓ 9 Changed 8 years ago by franck

Plus qu'à modifier la doc en conséquence (en précisant que pas avant la 2.10)

Mesdames, messieurs, à vos claviers…

comment:9 in reply to: ↑ 8 ; follow-up: ↓ 10 Changed 8 years ago by Welsh

Replying to franck:

Plus qu'à modifier la doc en conséquence (en précisant que pas avant la 2.10)

Mesdames, messieurs, à vos claviers…

Par où qu'on passe pour mettre à jour la doc ? (Jamais fait...)

comment:10 in reply to: ↑ 9 Changed 8 years ago by Welsh

Trouvé...

Replying to Welsh:

Replying to franck:

Plus qu'à modifier la doc en conséquence (en précisant que pas avant la 2.10)

Mesdames, messieurs, à vos claviers…

Par où qu'on passe pour mettre à jour la doc ? (Jamais fait...)

Note: See TracTickets for help on using tickets.

Sites map