Ticket #2190 (closed enhancement: fixed)
[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: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: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...)