From fa87c4d8ff7fde20f846225532d698dce49f2622 Mon Sep 17 00:00:00 2001
From: Paul POULAIN <paul@koha-fr.org>
Date: Thu, 20 Sep 2007 10:31:15 +0200
Subject: [PATCH] BUGfix : return correctly the form checking

The check_form_borrowers showed an error in the user entry if needed,
but validated it anyway

Signed-off-by: Chris Cormack <crc@liblime.com>
---
 koha-tmpl/intranet-tmpl/prog/en/js/members.js                 | 1 +
 .../intranet-tmpl/prog/en/modules/members/memberentrygen.tmpl | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/members.js b/koha-tmpl/intranet-tmpl/prog/en/js/members.js
index bbb1e30e56..766495c817 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/js/members.js
+++ b/koha-tmpl/intranet-tmpl/prog/en/js/members.js
@@ -146,6 +146,7 @@ var statut=0;
 	if (statut==1){
 			//alert if at least 1 error
 				alert(message+"\n"+message_champ);
+				return false;
 	}
 	else 
 	{
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tmpl
index 6bfdeb2422..31e7a9291c 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tmpl
@@ -584,9 +584,9 @@ patron <!-- TMPL_VAR NAME="surname" --> <!-- TMPL_VAR name="firstname" -->
 <!-- /TMPL_IF -->
     <div id="action">
       <!-- TMPL_IF NAME="opadd" -->
-        <input type="button" name="step4" onclick="check_form_borrowers();unique();" value="Add member" />
+        <input type="button" name="step4" onclick="return check_form_borrowers();unique();" value="Add member" />
       <!-- TMPL_ELSE -->
-        <input type="button" name="step4" onclick="check_form_borrowers();" value="Modify member" />
+        <input type="button" name="step4" onclick="return check_form_borrowers();" value="Modify member" />
       <!--/TMPL_IF -->
     </div>
 </form>
-- 
2.39.2