From f916aa7a5fd121f1d38bddf0c1e09bdf3b06f616 Mon Sep 17 00:00:00 2001
From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Date: Tue, 26 Aug 2014 11:14:33 +0200
Subject: [PATCH] Bug 12823: Hint to define the SRU search field mappings
Content-Type: text/plain; charset=utf-8

This is a follow-up for report 6536 (SRU search targets).
It will alert a user that saves a SRU server without field mappings.

Test plan:
Add a Z39.50 server. No confirm message.
Add a SRU server without field mappings. Cancel the confirm.
Add one field mapping. No confirm message.
---
 .../prog/en/modules/admin/z3950servers.tt          |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/z3950servers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/z3950servers.tt
index cc8e25a..46704b3 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/z3950servers.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/z3950servers.tt
@@ -70,6 +70,10 @@
         [% END %]
 
         $("#serverentry").submit(function( event ) {
+            // first test if show_sru_fields exists
+            if( $('#show_sru_fields').length && $('#show_sru_fields').val()=='' && !confirm( _("No SRU search field mappings have been defined. This means that all field searches will go through the whole record. Continue?"))) {
+                return false;
+            }
             // copy show_sru_fields to hidden counterpart
             $('#sru_fields').val( $('#show_sru_fields').val() );
             // enable recordtype to include field in post
-- 
1.7.7.6