validation d'un formulaire indexé

Version imprimable

Bonjour,
J'ai fait des recherches et je me suis aidé de cette discussion en particulier: http://www.developpez.net/forums/sho...d.php?t=353992, mais je ne trouve pas la solution à mon problème...

J'essaie de valider un formulaire indexé (13 lignes identiques) mais il ne se passe rien à la validation, la page se réaffiche simplement. Dans le javascript généré, j'ai vu que cette fonction était vide, qui ne me semble pas normal:
Code:

1 2 function saisieForm_required () { }
Quelqu'un peut-il m'aider? Merci d'avance.
Voici des extraits de mon code.
Dans validation.xml:
Code:

1 2 3 4 5 <form name="saisieForm"> <field property="motif" indexedListProperty="lignes" depends="required"> <arg key="saisieForm.motif" /> </field> </form>
Dans la jsp:
Code:

1 2 3 4 5 6 7 8 9 <html:form action="/valideSaisie" onsubmit="return validateSaisieForm(this);"> [...] <logic:iterate name="saisieForm" property="lignes" id="ligneDecompte"> <html:text name="ligneDecompte" property="motif" indexed="true"/> <html:text name="ligneDecompte" property="acte" indexed="true"/> <br/> </logic:iterate> [...] </html:form>
Dans struts-config.xml:
Code:

1 2 3 4 5 <action path="/valideSaisie" name="saisieForm" scope="session" type="fr.irpauto.ipsa.saisiedecomptes.web.action.SaisieAction" validate="false"> <forward name="valideSaisie" path="/jsp/saisie.jsp" /> </action>

09/07/2007, 15h00
c_nvy

Il me semble que la validation sur les propriétés indexées via le Validator fonctionne seulement côté serveur.

Peux-tu faire un test en enlevant l'attribut onsubmit du tag html:form et en mettant validate="true" dans le mapping de l'Action SaisieAction ?
09/07/2007, 15h17
khilik
Quand je mets validate="true" dans le mapping de l'Action SaisieAction, j'obtiens cette erreur:
Code:

1 2 java.lang.NullPointerException org.apache.commons.validator.Field.getIndexedProperty(Field.java:796)
09/07/2007, 15h21
c_nvy

Peux-tu montrer le code de l'ActionForm ?

Voici l'ActionForm:

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
32
33
public class SaisieForm extends ValidatorForm {
 
	private static final long serialVersionUID = 1L;
 
	private String numeroSS = null;
 
	public String getNumeroSS() {
		return numeroSS;
	}
 
	public void setNumeroSS(String numeroSS) {
		this.numeroSS = numeroSS;
	}
 
	private List<LigneDecompte> lignes;
 
	public List<LigneDecompte> getLignes() {
		return lignes;
	}
 
	public void setLignes(List<LigneDecompte> lignes) {
		this.lignes = lignes;
	}
 
	public LigneDecompte getLigneDecompte(int i) {
		return (LigneDecompte)this.lignes.get(i);
	}
 
	public void setLigneDecompte(int i, LigneDecompte ligneDecompte) {
		this.lignes.set(i, ligneDecompte);
	}
 
}

J'ai trouvé cette discussion: http://www.developpez.net/forums/showthread.php?t=90055, où les personnes disent aussi que c'est impossible. Il vaut peut-être mieux que j'utilise la méthode validate dans l'ActionForm?

09/07/2007, 16h36
c_nvy
Citation:
Envoyé par khilik

Quand je mets validate="true" dans le mapping de l'Action SaisieAction, j'obtiens cette erreur:

Code:

1 2 java.lang.NullPointerException org.apache.commons.validator.Field.getIndexedProperty(Field.java:796)
Je pense savoir pourquoi tu as ce message d'erreur.
Si je me souviens bien ce que tu as fait dans une précédente discussion, la première fois que tu charges la page, tu n'as pas encore constitué la collection lignes.
Il ne peut donc pas faire la validation sur la collection étant donné que celle-ci n'existe pas encore, ce qui explique le NullPointerException.
09/07/2007, 17h25
khilik

Je regarde de ce côté, tu as sûrement raison, mais j'ai du mal à voir à quel endroit il faut constituer la collection.
09/07/2007, 18h38
c_nvy
Dans ton cas, la solution serait peut-être de mettre validate à false dans le mapping de l'Action et de faire appel à la validation dans l'Action au moment voulu via ce code :
Code:

1 2 3 4 5 6 ActionMessages errors = form.validate(mapping,request); if ( !errors.isEmpty() ) { saveErrors(request,errors); return mapping.getInputForward() ; }
Sinon, dans l'ActionForm, il serait bien aussi de déclarer la collection lignes comme ceci :

Code:

private List<LigneDecompte> lignes = new ArrayList();

Si tu ne t'en sors pas, montre-moi le code de ton Action.;)

Citation:

Si tu ne t'en sors pas, montre-moi le code de ton Action.

J'abuse de ta gentillesse...

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
public class SaisieAction extends Action {
 
	public ActionForward execute(ActionMapping mapping, ActionForm form, HttpServletRequest request,
			HttpServletResponse response) throws Exception {
		String forward;
		SaisieForm saisieForm = (SaisieForm)form;
		// récupération de l'individu
		String numeroSS = saisieForm.getNumeroSS();
		IndividuService service = IndividuService.getInstance();
		List<Individu> listIndividus = service.get(numeroSS);
		// si nbSolutions=0, on affiche un message
		// sinon, on affiche l'individu et les lignes de décompte
		request.setAttribute("nbSolutions", listIndividus.size());
		if (listIndividus.size() == 1) {
			request.setAttribute("individu", listIndividus.iterator().next());
		}
		// constitution des 13 lignes de décompte
		ArrayList<LigneDecompte> lignes = (ArrayList<LigneDecompte>)saisieForm.getLignes();
		if (lignes==null) {
			lignes = new ArrayList<LigneDecompte>();
			for (int i = 1; i <= 13; i++)
				lignes.add(new LigneDecompte());
			saisieForm.setLignes(lignes);
		}
		forward = "valideSaisie";
		return mapping.findForward(forward);
	}
}

Je cherche avec ta solution, mais sans succès pour l'instant.

Avant de tester l'appel de la validation dans l'Action, pourrais-tu faire un test en initialisant l'ArrayList dans la méthode reset de SaisieForm :
Code:

1 2 3 4 5 6 7 8 public void reset(ActionMapping mapping, HttpServletRequest request) { this.lignes = new ArrayList<LigneDecompte>(); for (int i = 1; i <= 13; i++) { this.lignes.add(new LigneDecompte()); } }
Dans le constructeur de la classe LigneDecompte, comment initialises-tu les propriétés motif et acte ?

10/07/2007, 16h30
khilik

Merci pour cette solution! La validation fonctionne maintenant. L'initialisation de l'ArrayList dans la méthode reset, c'est sans doute ce qu'il me manquait le plus.
Je reviendrai pour décrire précisément ce qui a changé dans mon code par rapport à ma question de départ, je veux juste mettre tout au clair après mes nombreux tests...

Pour le constructeur de la classe LigneDecompte, en fait je n'en ai pas :oops: . C'est vrai qu'il serait peut-être bon que j'initialise les propriétés motif et acte.

Donc voici les changements par rapport à ma question de départ:
validation.xml n'a pas changé, cela signifie que les 13 champs motif (pour les 13 lignes du formulaire) sont requis:
Code:

1 2 3 4 5 <form name="saisieForm"> <field property="motif" indexedListProperty="lignes" depends="required"> <arg key="saisieForm.motif" /> </field> </form>
la jsp:
Code:

1 2 3 4 5 6 7 8 9 <html:form action="/valideSaisie"> [...] <logic:iterate name="saisieForm" property="lignes" id="ligneDecompte"> <html:text name="ligneDecompte" property="motif" indexed="true"/> <html:text name="ligneDecompte" property="acte" indexed="true"/> <br/> </logic:iterate> [...] </html:form>
struts-config.xml:
Code:

1 2 3 4 5 6 7 <action path="/valideSaisie" name="saisieForm" scope="session" type="fr.irpauto.ipsa.saisiedecomptes.web.action.SaisieAction" input="input" validate="true"> <forward name="input" path="/jsp/saisie.jsp" /> <forward name="valideSaisie" path="/jsp/saisie.jsp" /> </action>
La méthode reset de l'ActionForm a été implémentée comme proposé par c_nvy.
Par contre, je ne comprends pas bien comment fonctionne ici le plug-in Validator. Contrairement à ce qui est indiqué ici: http://java.developpez.com/faq/strut...on#validplugin, dans la jsp je ne mets pas:

Code:

<html:javascript formName="saisieForm" />

ni:

Code:

onsubmit="return validateSaisieForm(this);"

et la validation se fait quand même conformément à la règle écrite dans validation.xml.

11/07/2007, 11h00
c_nvy

Citation:

Envoyé par khilik

Par contre, je ne comprends pas bien comment fonctionne ici le plug-in Validator. Contrairement à ce qui est indiqué ici: http://java.developpez.com/faq/strut...on#validplugin, dans la jsp je ne mets pas:

Code:

<html:javascript formName="saisieForm" />

ni:

Code:

onsubmit="return validateSaisieForm(this);"

et la validation se fait quand même conformément à la règle écrite dans validation.xml.

Le tag html:javascript et l'attribut onsubmit du tag html:form sont utiles seulement dans le cas où la validation doit se faire côté client.

Or, dans ton cas, comme tu as des propriétés indexées, tu peux faire la validation seulement côté serveur, ce qui explique pourquoi tu n'as pas besoin de les coder.;)

Il va falloir qu'on donne plus de précisions dans la FAQ.;)