input type='hidden' dans pppsetup.cgi

[Gesp]

Est-ce que quelqu'un aurait une idée pourquoi quand je remplace les
input type='submit' par input type='hidden' (ce qui était à l'origine avant que je fasse les changements pour être compatible xhtml transitional) pour
name='PROFILE'
ou
name='BACKUPPROFILE'

la valeur que je récupère est '1|1' au lieu de '1'


Pour voir le problème, il suffit de tranformer la ligne
&log("$tr{'profile saved'} $pppsettings{'PROFILENAME'}");

en
&log("$tr{'profile saved'} $pppsettings{'PROFILENAME'} $pppsettings{'PROFILE'} $pppsettings{'BACKUPPROFILE'}");
mais testez d'abord que sur 'BACKUPPROFILE', cela évite d'avoir une page blanche.

Parce que je sèche un peu sur le sujet depuis quelques temps déjà.

[Franck78]

Tu remplaces TOUT les type='submit' par type='hidden' c'est ça ?

C'est pas clair, donnes moi le fichier modifié merdant.
Je viens de prendre le dernier sur SF.

[Gesp]

Non il faut juste remplacer les 2 inputs concernant PROFILE et BACKUPPROFILE (ligne 542 et 734)

N'arrivant plus à la faire fonctionner avec input='hidden', je me suis aperçu que cela fonctionnait avec 'submit' pour une raison que je me m'explique pas du tout.

Remplace
la ligne 734 en
<td><input type='hidden' name='BACKUPPROFILE' value='$pppsettings{'BACKUPPROFILE'}' />

la ligne 287
&log("$tr{'profile saved'} $pppsettings{'PROFILENAME'} $pppsettings{'PROFILE'} $pppsettings{'BACKUPPROFILE'}");

clique sur Rafraichir puis Sauvegarder et le problème commence : le contenu de BACKUPPROFILE est n°|n° au lieu de n°

[Franck78]

ok je regarde ca quand j'ai fini un petit truc sur le dyndns.
Tu peux vérifer chez toi:
choisis un provider dynamique, ne mets pas le mot depasse pour avoir une erreur.
Au réaffichage avec l'erreur, le provider est bon seulement pour ceux dont le libéllé est simple (dyndns-custom)
Si tu prend un hn.org par exemple, ca ne revient pas avec.

Donc je corriges ça et j'ajoute mon dyndns provider 'afraid.org' !

[Franck78]

Ok voila ce qui cloche en lisant pour moi:


Tu déclares une variable 'cachée' qui s'appelle 'PROFILE'
Dans le même <form>, tu déclares une autre variable avec aussi ce nom!
SELECT name ='BACKUPPROFILE'


Tu peux pas récupérer les deux valeurs dans une seule variable.

Quand tu changes le 'hidden' par 'submit', alors la première déclaration n'est plus une variable (donc tu obtiens le choix attendu du SELECT).

=>Supprimes ta ligne (aux deux endroits) qui ne te sert manifestement pas...

On change des noms pour éviter les collisions.

[Gesp]

Tu as regardé en CVS?
J'y ai déjà supprimé la seconde partie de déclaration des variables qui ne sert à rien, sans plus de résultat.

[Franck78]

1.17.2.22 chargé vers 12h?

fais moi le fichier buggé a télécharger...

[Franck78]

Tu peux pas récupérer les deux valeurs dans une seule variable.

En fait dans un language normal c'est vrai. Mais dans ce cas avec le cgi dont je ne connais pas toutes les subtilités, le post pour tout à fait contenir plusieur couples var=valeur.

Et getcgihash peut très bien concaténer les valeur de la même variable. getcgihash est écrite dans header.pl

Bingo

lit dans header.pl 522:
rewrite for multi var separated by "|"

[Gesp]

$Id: pppsetup.cgi,v 1.17.2.22 2004/07/15 20:54:45 gespinasse Exp $

Oui pour la version, c'est celui qui date d'une semaine.

Je ne comprend pas quand tu dis que la variable est définie 2 fois: je n'ai qu'un seul
<td><input type='submit' name='PROFILE' value='$pppsettings{'PROFILE'}' />
<select name='PROFILE'>

[Franck78]

$Id: pppsetup.cgi,v 1.17.2.22 2004/07/15 20:54:45 gespinasse Exp $

Oui pour la version, c'est celui qui date d'une semaine.

Je ne comprend pas quand tu dis que la variable est définie 2 fois: je n'ai qu'un seul
<td><input type='submit' name='PROFILE' value='$pppsettings{'PROFILE'}' />
<select name='PROFILE'>

name='PROFILE' apparait bien deux fois....


type='submit'
puis après aussi en
select

D'accord ?
En 'submit' ton objet est un BOUTON-ACTION dans le formulaire.
Quand tu le transformes en hidden (ce qui n'a pas de sens à priori)
cet objet devient une variable et elle est récupérée par getcgihash!

Donc si ton bouton ne sert à rien, supprimes le au lieu de vouloir le cacher (ce qui est impossible ici).

Ou change de nom à ton object SELECT

[Gesp]

Ok c'est un copier/coller malheureux.
Je ne sais pas quelle inspiration m'a poussée à mettre cela en double mais qu'est-ce que j'ai pu passer comme temps sur cette bétise.

[Franck78]

Ca ne fait que confirmer une règle absolue en développement. Quand tu merdoies trop longtemps sur un bug, fais relire par un autre...


J'ai posé un bug sur SF. Le coup du dyndns qui change. J'espère que cette fois ils accepteront l'ajout que j'ai effectué.

Bye

[Gesp]

Pour le active-filter, je vais essayer de finir cela vu que j'ai encore un autre changement à faire dans pppsetup.cgi

Pour ta modification, tu ne penses pas que ce serait mieux de faire
<td class='base'>$tr{'update string or'}$tr{'username'}</td>

plutôt que le message qui s'affiche après coup?

[Franck78]

Pour le active-filter, je vais essayer de finir cela vu que j'ai encore un autre changement à faire dans pppsetup.cgi

Pour ta modification, tu ne penses pas que ce serait mieux de faire
<td class='base'>$tr{'update string or'}$tr{'username'}</td>

plutôt que le message qui s'affiche après coup?

Houah, j'ai bien cherché 10 minutes le message de active filter avant de comprendre que tu avais changé de sujet !

Il faut prendre un des champs c'est sur, lequel est le plus adapté ???...??

Le plus simple serait de désactiver les champs inutiles, mais avec ce fonctionnement d'interface sans 'réactivité selon les cases cochée/choix' c'est pas facile d'avoir un fonctionnement ergonomique.

En fait le mieux serait un point bleu avec le commentaire toujours affiché en rappel!