Development #66348
action liaison fonction/rôle : ne fonctionne pas si la variable indiquée est un entier
Début:
16 juin 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Si on utilise vraiment un simple nom de variable (et pas un gabarit) alors wcs utilise la variable telle quelle.
Or quand la variable est un entier (genre form_var_truc_id), ça ne passe pas, parce qu'on compare avec des chaînes de caractères.
(Note : un remplacement par form_var_truc_raw qui est une chaine de caractère fait que ça passe, mais ça n'est pas évident à comprendre)
(Note 2 : j'ai écrit #66347 pour plutôt inciter à l'usager d'un gabarit)
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 2 ans
- Fichier 0001-wf-dispatch-allow-an-integer-value-in-variable-66348.patch 0001-wf-dispatch-allow-an-integer-value-in-variable-66348.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 2 ans
Cette partie alimente variable_values avec différentes représentations, pour augmenter les chances de match, pourquoi le if/else ici, pourquoi pas juste
variable_values = [variables.get(self.variable)]
if not variable_values[0]:
variable_values.append(None)
+ if variables.get(self.variable) is not None:
+ variable_values.append(str(variable_values[0]))
try:
variable_values.append(int(variable_values[0]))
except (ValueError, TypeError):
?
Mis à jour par Thomas Noël il y a presque 2 ans
- Fichier 0001-wf-dispatch-allow-non-string-value-in-variable-66348.patch 0001-wf-dispatch-allow-non-string-value-in-variable-66348.patch ajouté
Yep, c'est effectivement plus générique ainsi, merci.
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Thomas Noël
(une fois jenkins)
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 2301da5e0170187fbe6ddb93b796bca28c1d84f8
Author: Thomas NOËL <tnoel@entrouvert.com>
Date: Thu Jun 16 18:41:28 2022 +0200
wf/dispatch: allow non-string value in variable (#66348)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
wf/dispatch: allow non-string value in variable (#66348)