Anomalie #1967
calc_multi_val avec un fill ne fonctionne pas correctement
100%
Description
Si on utilise la fonction calc_multi_val dans un fill dans des variables, elles sont chargés au démarrage gen_config et ne sera plus recalculé lorsque les valeurs seront correctement renseignées.
Révisions associées
ajout de hidden dans les params + retourne chaine vide si un des paramètres sont vides pour concat ou calc_multi_val (fixes #1967)
mise à niveau des tests unitaires : ajout des hidden manquants (ref #1967)
correction sur la prise en compte des variables cachées dans calc_multi (fixes #1967)
Improve mail settings
Add new parameters; fixes #1967:
- smtp_sender may be different than admin email,
- add global "from" and "from name" that overrides admins infos
Set debug level, store debug infos in specific file (mail-debug)
Rework config form; dispaly SMTP config only if relevant
Drop not used options parameter from getSender method
Historique
#1 Mis à jour par Emmanuel GARETTE il y a plus de 12 ans
<fill name='calc_multi_val' target='ssl_subjectaltname_ip'>
<param type='eole' optional='True'>adresse_ip_eth0</param>
<param type='eole' optional='True'>alias_ip_eth0</param>
</fill>
A quoi est censé servir les "optional" ?
#2 Mis à jour par Emmanuel GARETTE il y a plus de 12 ans
L'attribut optional sert si la valeur n'existe pas. Ce qui n'est pas le cas de adresse_ip_eth0 et alias_ip_eth0.
#3 Mis à jour par Emmanuel GARETTE il y a plus de 12 ans
- Assigné à mis à Emmanuel GARETTE
- Version cible mis à Mises à jour 2.3 - 02 RC
Les variables présentes dans une fonction calc_multi_val doivent être maintenant obligatoires (ce qui est le cas pour les fonctions calc_multi_val aujourd'hui) !
Si la valeur est obligatoire mais peut être caché, il faut ajouté l'attribut hidden="False" (et non optional="False").
#4 Mis à jour par Emmanuel GARETTE il y a plus de 12 ans
- Statut changé de Nouveau à Résolu
- % réalisé changé de 0 à 100
Appliqué par commit fc65f5cff4c0eafd27bfdfad94e69a59e6c79139.
#5 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Version cible changé de Mises à jour 2.3 - 02 RC à Mises à jour 2.3 - 01 Stable
#6 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Statut changé de Résolu à Fermé
OK pour l'exemple fourni avec les paquets recompilés ce jour (17/08/2011) en eole-2.3-updates
#7 Mis à jour par Emmanuel GARETTE (2) il y a plus de 12 ans
J'ai honte ... ;)
Il y a un gros bug dans cette correction !
Voici la correction de la correction :
# diff -u typeole.py.orig typeole.py --- typeole.py.orig 2011-09-02 20:41:07.842182121 +0200 +++ typeole.py 2011-09-02 20:41:23.022163615 +0200 @@ -199,7 +199,7 @@ continue else: if param['hidden'] == 'False': - if self.context[param['value']].hidden == False: + if self.context[param['value']].hidden == True: continue vals = self.context[param['value']].get_value() if len(vals) == 0:
#8 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Version cible changé de Mises à jour 2.3 - 01 Stable à Mises à jour 2.3 - 02 RC
#9 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Statut changé de Fermé à En attente d'informations
#10 Mis à jour par Joël Cuissinat il y a plus de 12 ans
- Assigné à changé de Emmanuel GARETTE à Bruno Boiget
- Version cible changé de Mises à jour 2.3 - 02 RC à Mises à jour 2.3 - 02 Stable
- % réalisé changé de 100 à 50
- Distribution mis à EOLE 2.3
La correction de la correction n'a pas (encore) été appliquée !
#11 Mis à jour par Bruno Boiget il y a plus de 12 ans
- Statut changé de En attente d'informations à Résolu
- % réalisé changé de 50 à 100
Appliqué par commit c5ff615d92a6557f59b51e0132aa1f13257a4251.