Discussion :

[yolepro]

Bonjour,

J'ai des questions plutot basique sur validator, mais je ne trouve malheureusement pas de tutorial complet sur developpez (il faudra qu'on s'y mette ).

Je fais des tests via le validator en required sur des champs hidden et il ne semble pas prendre en compte cette contrainte, est ce normal?
(pour info cela fonctionne si je change le champs de hidden à text)

exemple :
validation.xml

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
<form name="creerForm">
    <field  property="codeObjet"  depends="required">
        <msg name="required" key="errors.required"/>
        <arg0 key="errors.codeObjet"/>
    </field>
...
</form>

maJsp.jsp

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
 
<html:javascript formName="creerForm" dynamicJavascript="true" staticJavascript="false"/>
<html:form action="creerAction">
<html:hidden property="codeObjet" name="creerForm"/>
<html:text property="libelleObjet" name="creerForm"/>
...
</html:form>

Merci d'avance pour vos réponses.

[fnobb]

Bonjour,

En principe cela devrait marcher, mais juste un question :

à quoi sert un control sur un champ caché ? (non accessible à l'utilisateur)
que ce passera-t-il si un jour le champ est vide ?
l'utilisateur à un message d'erreur (bloquant) mais ne peut rien y faire car le champ n'est pas saisissable !

[yolepro]

Le champ caché est rempli par une autre action (l'utilisateur doit rechercher l'objet via un lien).

Si jamais ce champs est vide c'est que l'utilisateur n'a pas recherché l'objet et il faut donc le lui dire.

[fnobb]

Ah okay okay !

donc pour en revenir à la problématique, cela devrait marcher car même caché le champs est soumis.

si les controles sont executé sur le serveur. est ce qu'il n'y aurait pas une erreur java ?

[yolepro]

Merci fnobb pour ta réponse.

Suite à des recherches un peu plus poussée, il se trouve qu'avec la version de Struts que j'utilises (v1.1) et le validatior-rules.xml; les champs hidden ne sont pas gérés pour la dépendance hidden.

j'ai quelque chose qui reseemble à ca :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
 
<validator name="requiredHidden"  
           classname="org.apache.struts.webapp.validator.FieldsCheck"  
              method="validateRequiredHidden"  
        methodParams="java.lang.Object,  
                      org.apache.commons.validator.ValidatorAction,  
                      org.apache.commons.validator.Field,  
                      org.apache.struts.action.ActionMessages,  
                      javax.servlet.http.HttpServletRequest"  
                 msg="errors.required">  
	<javascript>  
       <![CDATA[  
       function validateRequiredHidden(form) {  
         var isValid = true;  
         var focusField = null;  
         var i = 0;  
         var fields = new Array();  
         oRequired = new required();  
         for (x in oRequired) {  
           var field = form[oRequired[x][0]];  
 
           if (field.type == 'text' ||  
               field.type == 'textarea' ||  
               field.type == 'file' ||  
               field.type == 'select-one' ||  
               field.type == 'radio' ||  
               field.type == 'password') {  
 
             var value = '';  
 
             // get field's value  
             if (field.type == "select-one") {  
               var si = field.selectedIndex;  
               if (si >= 0) {  
                 value = field.options[si].value;  
               }  
             } else {  
               value = field.value;  
             }  
 
             if (trim(value).length == 0) { {  
               if (i == 0) {  
                 focusField = field;  
               }  
               fields[i++] = oRequired[x][1];  
               isValid = false;  
             }  
           }  
         }  
 
         if (fields.length > 0) {  
           focusField.focus();  
           alert(fields.join('\n'));  
         }  
 
         return isValid;  
       }  
 
       // Trim whitespace from left and right sides of s.  
       function trim(s) {  
         return s.replace( /^\s*/, "" ).replace( /\s*$/, "" );  
       }  
 
       ]]>  
     </javascript>  
</validator>

J'essai alors de faire ma propre classe FieldCheck avec une méthode validateRequiredHidden() mais elle ne s'arrete pas sur mon point d'arret (premiere ligne de la methode en question) et elle me genere une erreur assez incomprehensible :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
GRAVE: org.apache.struts.webapp.validator.FieldCheck.validateRequiredHidden(java.lang.Object, org.apache.commons.validator.ValidatorAction, org.apache.commons.validator.Field, org.apache.struts.action.ActionMessages, javax.servlet.http.HttpServletRequest)
org.apache.commons.validator.ValidatorException: org.apache.struts.webapp.validator.FieldCheck.validateRequiredHidden(java.lang.Object, org.apache.commons.validator.ValidatorAction, org.apache.commons.validator.Field, org.apache.struts.action.ActionMessages, javax.servlet.http.HttpServletRequest)
	at org.apache.commons.validator.ValidatorAction.loadValidationMethod(ValidatorAction.java:627)
	at org.apache.commons.validator.ValidatorAction.executeValidationMethod(ValidatorAction.java:557)
	at org.apache.commons.validator.Field.validateForRule(Field.java:827)
	at org.apache.commons.validator.Field.validate(Field.java:906)
	at org.apache.commons.validator.Form.validate(Form.java:174)
	at org.apache.commons.validator.Validator.validate(Validator.java:367)
	at org.apache.struts.validator.ValidatorForm.validate(ValidatorForm.java:110)
	at org.apache.struts.action.RequestProcessor.processValidate(RequestProcessor.java:950)
	at org.apache.struts.action.RequestProcessor.process(RequestProcessor.java:207)
	at org.apache.struts.action.ActionServlet.process(ActionServlet.java:1196)
	at org.apache.struts.action.ActionServlet.doPost(ActionServlet.java:432)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:709)
	at javax.servlet.http.HttpServlet.service(HttpServlet.java:802)
	at org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:252)
	at org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:173)
	at org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:213)
	at org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:178)
	at org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:126)
	at org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:105)
	at org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:107)
	at org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:148)
	at org.apache.coyote.http11.Http11Processor.process(Http11Processor.java:869)
	at org.apache.coyote.http11.Http11BaseProtocol$Http11ConnectionHandler.processConnection(Http11BaseProtocol.java:664)
	at org.apache.tomcat.util.net.PoolTcpEndpoint.processSocket(PoolTcpEndpoint.java:527)
	at org.apache.tomcat.util.net.LeaderFollowerWorkerThread.runIt(LeaderFollowerWorkerThread.java:80)
	at org.apache.tomcat.util.threads.ThreadPool$ControlRunnable.run(ThreadPool.java:684)
	at java.lang.Thread.run(Unknown Source)

Je ne sais pas trop ce qui cloche.

Comment s'y prendre? faut-il etendre certaine methodes?

Est ce que quelqu'un aurait un example simple à me montrer?

[yolepro]

Bon j'ai finalement résolu mon problème.

Je m'y prenais correctement, mais le javascript était généré dans une JSP via un cache (il ne prenait donc pas mes modifications en compte).

J'ai quand même modifié le required car il me fallait remonter les erreurs du champs hidden dans la même popup javascript.

Voila le code au final :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
            function validateRequired(form) {
                var isValid = true;
                var focusField = null;
                var i = 0;
                var fields = new Array();
                oRequired = new required();
                for (x in oRequired) {
                	var field = form[oRequired[x][0]];
 
                    if (field.type == 'text' ||
                        field.type == 'textarea' ||
                        field.type == 'file' ||
                        field.type == 'select-one' ||
                        field.type == 'radio' ||
                        field.type == 'password') {
 
                        var value = '';
						// get field s value
						if (field.type == "select-one") {
							var si = field.selectedIndex;
							if (si >= 0) {
								value = field.options[si].value;
							}
						} else {
							value = field.value;
						}
 
                        if (trim(value).length == 0) {
 
	                        if (i == 0) {
	                            focusField = field;
	                        }
	                        fields[i++] = oRequired[x][1];
	                        isValid = false;
                        }
                    }
					if (field.type == 'hidden') {
						alert('validateRequired(). if hidden');
						var value = field.value;
						if (trim(value).length == 0) {
	                        fields[i++] = oRequired[x][1];
	                        isValid = false;
                        }
					}
                }
                if (fields.length > 0) {
		   focusField.style.color="red";
                   focusField.focus();
                   alert(fields.join('\n'));
                }
                return isValid;
            }
 
            // Trim whitespace from left and right sides of s.
            function trim(s) {
                return s.replace( /^\s*/, "" ).replace( /\s*$/, "" );
            }

pour ton aide fnobb