[checkbox] soumission d'une valeur en fonction de l'etat du checkbox

Version imprimable

Voir 40 message(s) de cette discussion en une page

08/04/2009, 15h41
OButterlin

Si tu veux renvoyer la valeur "R" quand la case est cochée, tu mets l'attribut value="R".
Comme une case non cochée n'est pas envoyée (c'est comme ça), il faut systématiquement réinitialiser les propriétés représentées par une checkbox dans la méthode reset(...) du formulaire (ActionForm)

ok, sinon, j'ai ajoute ds le reset ce bout de code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
	public void reset(ActionMapping mapping, HttpServletRequest request) {
		if (!ajustContrat.isEmpty()) {
			for (Iterator<AjustContrat> iterator = ajustContrat.iterator(); iterator
					.hasNext();) {
				AjustContrat ajustContrat = (AjustContrat) iterator.next();
				if (((ajustContrat.getCAjustTypeAjust() != null) && (!ajustContrat
						.getCAjustTypeAjust().equals("R")))
						|| (ajustContrat.getCAjustTypeAjust() == null)) {
					ajustContrat.setCAjustTypeAjust("");
				}
			}
		}
	}

Cdlt
Sallemel

08/04/2009, 15h57
sallemel

comme ca, une case qui n'est pas coche par defaut
si je la coche et je sauvegarde, sa valeur reste egale a "", elle ne prend pas la valeur R

auriez vous une idee pourquoi?

Merci
Cdlt
Sallemel

Comme je l'ai dit précédemment, il faut initialiser la propriété CAjustTypeAjust de tous les objets de la liste ajustContrat à "" dans la méthode reset de l'ActionForm.
Comme la méthode reset est appelée avant le setter, toutes les valeurs sont initialisées à "" et ensuite, il affectera la valeur R pour les checkbox qui ont été cochées.

Il faut donc enlever la clause if dans la boucle for :
Code:

1 2 3 4 5 6 if (!ajustContrat.isEmpty()) { for (Iterator<AjustContrat> iterator = ajustContrat.iterator(); iterator.hasNext();) { AjustContrat ajustContrat = (AjustContrat) iterator.next(); ajustContrat.setCAjustTypeAjust(""); } }

08/04/2009, 17h14
sallemel

ok, merci c_nvy pour votre reponse
ceci etant, j'ai resette ds tous les ajust_contrat le type a "" au niveau du reset
mais , j'ai le meme probleme
si le type vaut R, en editant la ligne, j'ai bien une case cohee
mais lorsque je decoche la case et je fais le save
ca garde la valeur R
de meme si le type vaut "" et j'edite la ligne
la case apparait bien decochee
mais si je la cohe et je fais le save
ca reste =""

juste a titre d'info, les donnes je les recuperes de la bd et les modifs aussi
so sauvegardes au niveau de la bd
Cdlt
Sallemel
08/04/2009, 18h03
c_nvy

As-tu le même résultat si tu n'exécutes plus la fonction javascript ?
Quand tu fais save, tu cliques bien sur un bouton de type submit ?

Bonjour,
en fait, lors de la soumission, ca garde l'ancienne valeur

voici la fonction associee au click

Code:

1
2
3
4
5
6
7
8
9
 
function verifierReversible(i){
if (document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked=="false"){
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="";
} else {
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="R";
}
 
}

et voici le code lors de la soumission :

Code:

1
2
3
4
5
6
7
8
9
10
11
 
		document.forms[0].elements["ajustcontModifieId"].value = i+1;
		document.forms[0].elements["ajustContrat[" + i + "].contratAjustEditmode"].value=0;
		if (document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked=="true"){
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="R";
} else {
document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="";
}
		document.forms[0].action = 'EditAjustContrat.do?do=validateAjustContrat';
 
		document.forms[0].submit();

Cdlt
Sallemel

Mais pour moi, le code Javascript suivant ne sert à rien :
Code:

1 2 3 4 if (document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked=="true"){ document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="R"; } else { document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].value="";}
étant donné que tu as initialisé toutes les valeurs de CAjustTypeAjust à "" dans la méthode reset() de l'ActionForm et que tu as ajouté un attribut value="R" dans le tag nested:checkbox.
Donc, automatiquement, après submit du formulaire déclenché par document.forms[0].submit() , toutes checkbox cochées aura pour valeur R et toutes checkbox non cochées aura pour valeur "".

Peux-tu donc faire un test en enlevant ce code ?

09/04/2009, 10h57
sallemel

d'accord, mais le probleme c'est que si je mets la valeur du checkbox a "R" , j'enleve l'evenement onclick, je supprime le bout de code de test ds la validation et j'ajoute le reset au niveau de l'actionform
le resultat est que je garde toujours l'ancienne valeur du type
si la case est cochee , je la decoche et je valide => j'ai en resultat R (ancienne valeur)
si j'ai la case decohee et je la coche , une fois je valide => j'ai "" ds type

a titre d'information, le scope pour cette action, est sette a session

Cdlt
Sallemel
09/04/2009, 11h14
c_nvy

Le form-bean est-il bien dans le scope session ?

oui, tout a fait, voici la defintition de l'action
Code:

1 2 3 4 5 6 7 8 9 <action name="saisiAjustContratEditForm" parameter="do" path="/EditAjustContrat" scope="session" type="com.thales.project.action.SaisiAjustContratAction"> <forward name="gererSaisiAjustContrats" path="/pages/CPFA_Mensuel/saisi_ajustement/ajust_contrat.jsp" /> <forward name="ajouterCommentaire" path="/pages/CPFA_Mensuel/saisi_ajustement/addComment.jsp" /> </action>
par ailleurs, j'ai debuggue l'action lors de la validation, je confirme que pour l'enregistrement a valider , je recupere toujours l'ancienne valeur du type au niveau de l'action
mais lors de l'edition d'un enregistrement, j'ai bien une case cochee si type vaut "R" et decochee si type vaut ""
Cdlt
Sallemel

et juste pour debugguer,
j'ai remis l'evenement onclick
Code:

1 2 <nested:checkbox property="CAjustTypeAjust" value="R" onclick='<%="verifierReversible("+Integer.parseInt(numLigne)+" )"%>' />
et ds la fonction javascript, j'ai mis juste ca
Code:

1 2 3 4 5 6 function verifierReversible(i){ alert(document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked); }
si le type vaut "", et j'edite l'enregistrement, que je coche ou decoche la case, j'ai toujours false ds
Code:

1 2 document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked
et ca renvoie , undefined , si la valeur par defaut du type est "R"

Cdlt
Sallemel

09/04/2009, 15h06
c_nvy

Je ne suis pas spécialiste en Javascript mais peut-être que checked ne peut renvoyer true que si CAjustTypeAjust est de type booleen.

Mais bon, ça devrait fonctionner sans javascript.

Peux-tu montrer le code de la méthode validateAjustContrat de l'Action et de la méthode reset() de l'ActionForm ?
09/04/2009, 15h37
OButterlin
Citation:
Envoyé par sallemel

Code:

1 2 document.forms[0].elements["ajustContrat[" + i + "].CAjustTypeAjust"].checked
C'est quoi ça ?

L'indice s'applique à CAjustTypeAjust, non ?
09/04/2009, 15h43
c_nvy

Non, CAjustTypeAjust est une propriété des objets de la liste ajustContrat.
09/04/2009, 15h50
OButterlin
Sauf que là, on est sur le client en javascript... la seule propriété de formulaire qu'on connait est CAjustTypeAjust
(D'après ce que je vois là)
Code:

1 2 <nested:checkbox property="CAjustTypeAjust" value="R" onclick='<%="verifierReversible("+Integer.parseInt(numLigne)+" )"%>' />
Il serait plus esthétique de coder ceci d'ailleurs
Code:

1 2 onclick="verifierReversible(<%=numLigne%>)"
09/04/2009, 16h03
c_nvy

Oui mais le nested:checkbox doit être dans un nested:iterate. Enfin, je suppose vu le code html généré.

Sinon, si tu codes ceci :

Code:

onclick="verifierReversible(<%=numLigne%>)"

la scriptlet n'est pas interprétée.
09/04/2009, 16h11
OButterlin
Citation:

Envoyé par c_nvy

Sinon, si tu codes ceci :

Code:

onclick="verifierReversible(<%=numLigne%>)"

la scriptlet n'est pas interprétée.

Oui, effectivement... autant pour moi...:oops:
Par contre tu peux faire
Code:

1 2 onclick="verifierReversible(${idx})"
avec
Code:

1 2 <nested:iterate property="list" indexId="idx">
09/04/2009, 16h15
c_nvy

Je lui ai déjà proposé précédemment dans la discussion mais il faut qu'il déclare son web.xml en J2EE 1.4.

Bonjour,

Citation:

Je ne suis pas spécialiste en Javascript mais peut-être que checked ne peut renvoyer true que si CAjustTypeAjust est de type booleen.

Mais bon, ça devrait fonctionner sans javascript.

Peux-tu montrer le code de la méthode validateAjustContrat de l'Action et de la méthode reset() de l'ActionForm ?

oui biensur, voici le code de validation sachant que j'ai debuggue l'action avec eclipse, et ds ajustContratIt, je ne trouve pas la bonne valeur modifie, je trouve toujours l'ancienne valeur, donc c'est plutot un pb lors du renvoi de la page jsp

ici la methode validation de l'action
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 public ActionForward validateAjustContrat(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) throws Exception { SaisiAjustContratEditForm ajustContratEditForm = (SaisiAjustContratEditForm) form; List<AjustContrat> ajustContrat = ajustContratEditForm .getAjustContrat(); for (Iterator<AjustContrat> iterator = ajustContrat.iterator(); iterator .hasNext();) { AjustContrat ajustContratIt = (AjustContrat) iterator.next(); if (ajustContratIt.getCAjustNLigne() == ajustContratEditForm .getAjustcontModifieId()) { HibernateAjustContratDAO dao = new HibernateAjustContratDAO(); dao.updateUjustContrat(ajustContratIt); HibernateUtilisateurDAO daoUser = new HibernateUtilisateurDAO(); if (ajustContratIt.getContratAjustEditmode() == 0) { MessageResources mr = getResources(request); String operationLib = mr .getMessage("mensuel.ajustContrat.operationLib"); DataMonitoringHome daoDataMonit = new DataMonitoringHome(); DataMonitoring dataMonit = new DataMonitoring(1, ajustContratEditForm.getSelectedPeriod(), daoUser .findByIdentifiantDAO("SallemEl"), new Date(), operationLib); daoDataMonit.insertDatamonitoringDAO(dataMonit); } break; } } return mapping.findForward("gererSaisiAjustContrats"); }
et ici la methode reset de l'actionForm
Code:

1 2 3 4 5 6 7 8 9 10 public void reset(ActionMapping mapping, HttpServletRequest request) { if (!ajustContrat.isEmpty()) { super.reset(mapping, request); for (Iterator<AjustContrat> iterator = ajustContrat.iterator(); iterator.hasNext();) { AjustContrat ajustContrat = (AjustContrat) iterator.next(); ajustContrat.setCAjustTypeAjust(""); } } }
Citation:

Oui mais le nested:checkbox doit être dans un nested:iterate. Enfin, je suppose vu le code html généré.

Sinon, si tu codes ceci :

onclick="verifierReversible(<%=numLigne%>)"

la scriptlet n'est pas interprétée.

j'ai essaye, mais ce n'est pas interprete
par contre , sous cette forme, j'ai bien la bonne valeur
Code:

1 2 onclick='<%="verifierReversible("+Integer.parseInt(numLigne)+" )"%>'
Cdlt
Sallemel

Voir 40 message(s) de cette discussion en une page