Sens de tri avec multi #5213

Closed
opened 2 months ago by tcharlss · 3 comments
Owner

Le sens de tri n'est pas pris en compte avec multi xxx.

Exemple sur contrib :

@marcimat propose un simple fix :

--- a/ecrire/public/fonctions.php
+++ b/ecrire/public/fonctions.php
@@ -522,7 +522,7 @@ function tri_protege_champ($t) {
  */
 function tri_champ_order($t, $from = null, $senstri = '') {
        if (strncmp($t, 'multi ', 6) == 0) {
-               return 'multi';
+               return 'multi' . $senstri;
        }
 
        $champ = $t;

Trop simple pour être honnête ? Foin, je fais une PR.

(Nb : c'est dans la foulée de !5210)

Le sens de tri n'est pas pris en compte avec `multi xxx`. Exemple sur contrib : * https://contrib.spip.net/ecrire/?exec=auteurs&tri_liste_aut=multi%20nom&sens_liste_aut=1 * https://contrib.spip.net/ecrire/?exec=auteurs&tri_liste_aut=multi%20nom&sens_liste_aut=-1 @marcimat propose un simple fix : ```diff --- a/ecrire/public/fonctions.php +++ b/ecrire/public/fonctions.php @@ -522,7 +522,7 @@ function tri_protege_champ($t) { */ function tri_champ_order($t, $from = null, $senstri = '') { if (strncmp($t, 'multi ', 6) == 0) { - return 'multi'; + return 'multi' . $senstri; } $champ = $t; ``` Trop simple pour être honnête ? Foin, je fais une PR. (Nb : c'est dans la foulée de !5210)
b_b added the
bug
label 2 months ago
b_b added this to the 4.2 milestone 2 months ago
Owner

À voir si on reporte en 4.1.

À voir si on reporte en 4.1.

Pour moi c'est bien un bug, et 4.1 est maintenue, donc oui à backporter.

Pour moi c'est bien un bug, et 4.1 est maintenue, donc oui à backporter.
b_b modified the milestone from 4.2 to 4.1 2 months ago
marcimat closed this issue 2 months ago
marcimat closed this issue 2 months ago
Owner

Reporté en 4.1.

Reporté en 4.1.
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.