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.
Associated revisions
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
History
#1 Updated by Emmanuel GARETTE almost 12 years ago
<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 Updated by Emmanuel GARETTE almost 12 years ago
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 Updated by Emmanuel GARETTE almost 12 years ago
- Assigned To set to Emmanuel GARETTE
- Target version set to 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 Updated by Emmanuel GARETTE almost 12 years ago
- Status changed from Nouveau to Résolu
- % Done changed from 0 to 100
Appliqué par commit fc65f5cff4c0eafd27bfdfad94e69a59e6c79139.
#5 Updated by Joël Cuissinat almost 12 years ago
- Target version changed from Mises à jour 2.3 - 02 RC to Mises à jour 2.3 - 01 Stable
#6 Updated by Joël Cuissinat almost 12 years ago
- Status changed from Résolu to Fermé
OK pour l'exemple fourni avec les paquets recompilés ce jour (17/08/2011) en eole-2.3-updates
#7 Updated by Emmanuel GARETTE (2) almost 12 years ago
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 Updated by Joël Cuissinat almost 12 years ago
- Target version changed from Mises à jour 2.3 - 01 Stable to Mises à jour 2.3 - 02 RC
#9 Updated by Joël Cuissinat almost 12 years ago
- Status changed from Fermé to En attente d'informations
#10 Updated by Joël Cuissinat over 11 years ago
- Assigned To changed from Emmanuel GARETTE to Bruno Boiget
- Target version changed from Mises à jour 2.3 - 02 RC to Mises à jour 2.3 - 02 Stable
- % Done changed from 100 to 50
- Distribution set to EOLE 2.3
La correction de la correction n'a pas (encore) été appliquée !
#11 Updated by Bruno Boiget over 11 years ago
- Status changed from En attente d'informations to Résolu
- % Done changed from 50 to 100
Appliqué par commit c5ff615d92a6557f59b51e0132aa1f13257a4251.