Projet

Général

Profil

Bug #13832

Le motif pour les noms de requêtes dans csvdatasource n'accepte pas tous les slugs (notamment avec tirets)

Ajouté par Benjamin Dauvergne il y a plus de 7 ans. Mis à jour il y a presque 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
03 novembre 2016
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers

Révisions associées

Révision 04af800f (diff)
Ajouté par Benjamin Dauvergne il y a plus de 7 ans

csvdatasource: allow dash in query names (fixes #13832)

Historique

#3

Mis à jour par Frédéric Péters il y a plus de 7 ans

La regex, pourquoi pas [\w-] ?

#4

Mis à jour par Thomas Noël il y a plus de 7 ans

Oui pour [\w-]+ et ne pas prendre les _.

(et dans une regex genre [A-Za-z0-9-_] je trouve que c'est plus clair de mettre le - à la fin)

#5

Mis à jour par Frédéric Péters il y a plus de 7 ans

(\w autorise _)

#7

Mis à jour par Benjamin Dauvergne il y a plus de 7 ans

  • Patch proposed changé de Non à Oui
#8

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Statut changé de Nouveau à En cours

"fix" dans la description de commit c'est nul; "allow dashes in query names".

#9

Mis à jour par Jean-Baptiste Jaillet il y a plus de 7 ans

On a eu le bug avec Mik, sur une query avec un -.
Si y'a moyen de changer le message de commit et ack !.
C'est pour faire un up comme je vois qu'on l'a oublié

#10

Mis à jour par Thomas Noël il y a plus de 7 ans

Jean-Baptiste Jaillet a écrit :

Si y'a moyen de changer le message de commit et ack !.

enjoy git --amend

#11

Mis à jour par Benjamin Dauvergne il y a plus de 7 ans

Patch à jour par rapport à master, message de commit corrigé.

#12

Mis à jour par Benjamin Dauvergne il y a plus de 7 ans

  • Statut changé de En cours à Nouveau
#13

Mis à jour par Frédéric Péters il y a plus de 7 ans

Ack.

#14

Mis à jour par Benjamin Dauvergne il y a plus de 7 ans

  • Statut changé de Nouveau à Résolu (à déployer)
  • % réalisé changé de 0 à 100
#15

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF