bug fix :add new condition in javascript to eliminate error when field not exist.

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/javascript/members.js b/koha-tmpl/intranet-tmpl/prog/en/includes/javascript/members.js
index d3311ac..40fd625 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/javascript/members.js
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/javascript/members.js
@@ -80,7 +80,6 @@ var myDate2=document.form.dateexpiry.value.split ('/');
 \r
 // function to test all fields in forms and nav in different forms(1 ,2 or 3)\r
  function check_form_borrowers(nav){\r
-\r
 var statut=0;\r
 if (nav < document.form.step.value) {\r
        document.form.step.value=nav;\r
@@ -107,34 +106,36 @@ if (nav < document.form.step.value) {
        var champ_verif = document.form.BorrowerMandatoryfield.value.split ('|');\r
        var champ_form= document.form.FormFieldList.value.split('|');\r
                var message ="The following fields are mandatory :\n";\r
-       var message_champ="";\r
-               for (var j=0; j<champ_form.length; j++){ \r
+               var message_champ="";\r
+       for (var j=0; j<champ_form.length; j++){ \r
+               if (document.getElementsByName(""+champ_form[j]+"")[0]){\r
                        for (var i=0; i<champ_verif.length; i++) {\r
-                               if (eval("document.form.elements['"+champ_verif[i]+"']") && eval("document.form.elements['"+champ_verif[j]+"']")) {\r
-                                       var val_champ=eval("document.form."+champ_verif[i]+".value");\r
-                                       var ref_champ=eval("document.form."+champ_verif[i]);\r
-                                       var val_form=eval("document.form."+champ_form[j]+".value");\r
-                                       if (champ_verif[i] == champ_form[j]){\r
-                                               //check if it's a select\r
-                                               if (ref_champ.type=='select-one'){\r
-                                                       if (ref_champ.options[0].selected ){\r
-                                                               // action if field is empty\r
-                                                               message_champ+=champ_verif[i]+"\n";\r
-                                                               //test to konw if u must show a message with error\r
-                                                               statut=1;\r
+                                       if (eval("document.form.elements['"+champ_verif[i]+"']") ) {\r
+                                               var val_champ=eval("document.form."+champ_verif[i]+".value");\r
+                                               var ref_champ=eval("document.form."+champ_verif[i]);\r
+                                               var val_form=eval("document.form."+champ_form[j]+".value");\r
+                                               if (champ_verif[i] == champ_form[j]){\r
+                                                       //check if it's a select\r
+                                                       if (ref_champ.type=='select-one'){\r
+                                                               if (ref_champ.options[0].selected ){\r
+                                                                       // action if field is empty\r
+                                                                       message_champ+=champ_verif[i]+"\n";\r
+                                                                       //test to konw if u must show a message with error\r
+                                                                       statut=1;\r
+                                                               }\r
+                                                       }else {\r
+                                                               if ( val_champ == '' ) {\r
+                                                                       // action if the field is not empty\r
+                                                                       message_champ+=champ_verif[i]+"\n";\r
+                                                                       statut=1;\r
+                                                               }       \r
                                                        }\r
-                                               }else {\r
-                                                       if ( val_champ == '' ) {\r
-                                                               // action if the field is not empty\r
-                                                               message_champ+=champ_verif[i]+"\n";\r
-                                                               statut=1;\r
-                                                       }       \r
                                                }\r
                                        }\r
-                               }\r
                        }\r
                }\r
        }\r
+       }\r
 //borrowers form 2 test if u chcked no to the quetsion of double \r
        if (document.form.step.value==2 && statut!=1 && document.form.check_member.value > 0 )\r
        {\r