Project

General

Profile

Anomalie #1967

calc_multi_val avec un fill ne fonctionne pas correctement

Added by Emmanuel GARETTE about 11 years ago. Updated almost 11 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Category:
-
Start date:
08/17/2011
Due date:
% Done:

100%

Distribution:
EOLE 2.3

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

Revision fc65f5cf (diff)
Added by Emmanuel Garette about 11 years ago

ajout de hidden dans les params + retourne chaine vide si un des paramètres sont vides pour concat ou calc_multi_val (fixes #1967)

Revision 6b9f549f (diff)
Added by Joël Cuissinat about 11 years ago

mise à niveau des tests unitaires : ajout des hidden manquants (ref #1967)

Revision c5ff615d (diff)
Added by Bruno Boiget almost 11 years ago

correction sur la prise en compte des variables cachées dans calc_multi (fixes #1967)

Revision df577018 (diff)
Added by Johan Cwiklinski about 5 years ago

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 about 11 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 about 11 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 about 11 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 about 11 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100

#5 Updated by Joël Cuissinat about 11 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 about 11 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) about 11 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 about 11 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 about 11 years ago

  • Status changed from Fermé to En attente d'informations

#10 Updated by Joël Cuissinat almost 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 almost 11 years ago

  • Status changed from En attente d'informations to Résolu
  • % Done changed from 50 to 100

#12 Updated by Joël Cuissinat almost 11 years ago

  • Status changed from Résolu to Fermé

semble OK

Also available in: Atom PDF