From c900a47aa8c7133036138cf79a81e70b827fe61f Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 7 Oct 2013 10:41:24 +0200 Subject: [PATCH] Bug 11002: simplify patron mandatory field validation When the pref BorrowerMandatoryField is set and a librarian forget to fill fields, a js popup appears with the following message: "The following fields are mandatory: surname, etc." The fields are not translatable. Test plan: - fill the pref BorrowerMandatoryField with something like: title|zipcode|surname|cardnumber|branchcode - go the members/memberentry.pl?op=add page - verify you are not allowed to add the patron without filling all mandatory fields. Signed-off-by: Paola Rossi Signed-off-by: Brendan Gallagher Signed-off-by: Galen Charlton --- koha-tmpl/intranet-tmpl/prog/en/js/members.js | 38 +++---------------- .../prog/en/modules/members/memberentrygen.tt | 1 - 2 files changed, 5 insertions(+), 34 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/members.js b/koha-tmpl/intranet-tmpl/prog/en/js/members.js index 28bf5c7ebe..d7c29476f2 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/js/members.js +++ b/koha-tmpl/intranet-tmpl/prog/en/js/members.js @@ -98,38 +98,6 @@ function check_form_borrowers(nav){ } } - if (document.form.BorrowerMandatoryField.value==='') - {} - else - { - var champ_verif = document.form.BorrowerMandatoryField.value.split ('|'); - message += MSG_MISSING_MANDATORY; - message += "\n"; - for (var i=0; i -- 2.39.5