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