Project

General

Profile

Anomalie #4704

Par sinum tri le nombre par ordre alpha

Added by Maïeul Rouquette 3 months ago. Updated 23 days ago.

Status:
En cours
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
03/25/2021
Due date:
% Done:

0%

Resolution:
Navigateur:

Description

Soit un site avec deux articles.

- 20. Doublon
- 3. Article 3

Soit le squelette suivant

par sinum titre
<BOUCLE_articles(ARTICLES) {par sinum titre, titre}>
<li>#TITRE*</li>
</BOUCLE_articles>

Par num titre
<BOUCLE_articles2(ARTICLES) {par num titre}>
<li>#TITRE*</li>
</BOUCLE_articles2>

Le resultat est le suivant

par sinum titre
20. Doublon
3. Article 3
Par num titre
3. Article 3
20. Doublon

Visiblement, contrairement à par num, par sinum ne tient pas compte du fait que 20 > 3.

History

#1 Updated by marcimat 🌻 3 months ago

sinum renvoie 0 ou 1. Ça sert juste à mettre les titres sans numéros en premier ou dernier.
Si tu veux trier en plus, bah tu ajoutes un autre tri.

{par sinum titre, num titre, titre}

#2 Updated by b b 3 months ago

  • Status changed from Nouveau to En cours

Et c'est documenté ici https://www.spip.net/fr_article5531.html#Ordonner-par-numero

{par sinum titre, num titre, titre} mettra les éléments sans numéro en fin de liste, contrairement à {par num titre, titre} .

Mais ça mériterait peut-être de le mentionner ici aussi https://www.spip.net/fr_article2035.html#par-num---- ?

#3 Updated by cedric - 3 months ago

Moi je me demande si on devrait pas surtout changer le {par num ...} pour que par défaut il mette bien en dernier ceux qui n'ont pas de numéro
(et les rares cas où on voudrait que ceux sans numéros soit en premier, on utilisera {sinum...}

#4 Updated by b b 3 months ago

+100 je galère toujours pour retrouver la bonne syntaxe de {par sinum titre, num titre, titre} qui est bien trop verbeuse, et j'ai souvenir de ma surprise le jour où je l'ai découvert car sinum me semblait faire "morte quoi" comme dirait quelqu'un :p

#5 Updated by marcimat 🌻 3 months ago

Pour que {par num} mette en dernier les sans numéros, il faut qu'il fasse automatiquement le "par sinum" du coup... (en ajoutant sinum dans le select...)
Mais du coup, comment tu le désactiverais si tu veux l'ancien comportement ?

#6 Updated by RastaPopoulos ♥ 3 months ago

Si techniquement c'est possible de faire les sans numéros à la fin de base par défaut : je +1 aussi

#7 Updated by Maïeul Rouquette 3 months ago

AH ! Donc je comprend enfion cette syntaxe absconne ! Pour moi "par sinum" faisait les deux, vu la phrase

{sinum nom_champ} permet d’avoir un tri par numéro avec les éléments numérotés avant ...

il faudrait plutot dire

{sinum nom_champ} met les éléments numérotés avant ... il doit être ensuite combiné avec le critère par num

parce que moi je ne comprenait par pourquoi on disait "par tri par numéro si il y a un numéro, puis par numero".

après... why not changer le comportement de par num, on peut se l'assumer en 3.3, non ?

Par contre cedric tu dis

(et les rares cas où on voudrait que ceux sans numéros soit en premier, on utilisera {sinum...}

si on fait cela, cela voudrait dire que tout les squelettes où l'on a sinum mettrait les numéros... à la fin. Ce qui non seulement ne serait pas cohérent avec le nom du critère, mais en plus casserait les choses. Mais je n'ai pas d'idée de bonne syntaxe

#8 Updated by JLuc - 3 months ago

J'ai actualisé la doc pour refléter le comportement actuel : « Ce critère ne fait que placer les éléments numérotés avant les non-numérotés : il ne trie pas plus les éléments à l’intérieur de ces 2 groupes. Il doit donc être utilisé en conjonction avec un autre critère de tri. » Ça fait du bien de comprendre la logique du truc.

#9 Updated by b b 3 months ago

Désolé jluc, mais je n'arrive pas à comprendre la mention "il ne trie pas plus les éléments à l’intérieur de ces 2 groupes" même après plusieurs lectures :\ Amha il faut reformuler plus simplement et peut-être se passer des "ne fait que" qui semble indiquer qu'il manque un truc au critère ou que les gens ne devraient pas l'utiliser.

#10 Updated by Maïeul Rouquette 3 months ago

j'ai modifié pour rendre plus clair b_b jluc

#11 Updated by JLuc - 3 months ago

Impec la nouvelle formulation. J'y ai ajouté "par" car il faut pas oublier "par" avant "sinum" dans le critère.

Quand on sait, avec cette nouvelle doc, c'est assez clair ce que fait sinum.
Mais je me demande si un autre terme que "sinum" serait plus clair spontanément : même quand on ne sait PAS encore ce que ça fait...

#12 Updated by cedric - about 2 months ago

  • Target version set to 4.0

#13 Updated by JLuc - 23 days ago

ticket traité ?

#14 Updated by RastaPopoulos ♥ 23 days ago

Je ne crois pas non, mais il faudrait changer le titre qui devrait désormais être : {par num} devrait trier par défaut les sans numéros à la fin (+ trouver un autre critère pour désactiver)

Also available in: Atom PDF