From c5f603c8404f2b2825d228d8697aef9ec1bd7e55 Mon Sep 17 00:00:00 2001
From: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Date: Wed, 15 Jul 2015 11:50:18 +0100
Subject: [PATCH] Bug 12525: Prevent adding several patron lists with the same
 name

If you add patron to a patron list, from the patron search result, a
list is created when you click on "Save".
The list is considered as new each time.

To reproduce:
1/ Launch a patron search
2/ Select 1 patron, and create a new list 'aaa'
3/ Select another patron and click Save again
2 lists are created

Test plan:
1/ Launch a patron search
2/ Select 1 patron, and create a new list 'aaa'
The dropdown list should be populated with this new list, and should be
selected
3/ Select another patron and click Save again
Only 1 list should be created

NOTE: And the single list doesn't duplicate patrons that were
      saved both times. Both patches needed to make the list
      functionality work more sanely.

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>
---
 koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt
index aac6657..0d50f4a 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/member.tt
@@ -76,6 +76,13 @@ $(document).ready(function() {
                 $("#patron_list_dialog > span.patrons-length").html(data.patrons_added_to_list);
                 $("#patron_list_dialog > a").attr("href", "/cgi-bin/koha/patron_lists/list.pl?patron_list_id=" + data.patron_list.patron_list_id);
                 $("#patron_list_dialog > a").html(data.patron_list.name);
+                var new_patron_list_added = $("<option>", {
+                    value: data.patron_list.patron_list_id,
+                    text: data.patron_list.name
+                });
+                $("#add_to_patron_list optgroup").append(new_patron_list_added);
+                $("#add_to_patron_list").val(data.patron_list.patron_list_id);
+                $("#new_patron_list").val('');
             },
             error: function() {
                 alert("an error occurred");
-- 
2.1.4