Attachment 38052 Details for Bug 13886 – [PASSED QA] Bug 13886 - Add ability to merge records directly from cataloging search

[patch] [PASSED QA] Bug 13886 - Add ability to merge records directly from cataloging search

PASSED-QA-Bug-13886---Add-ability-to-merge-records.patch (text/plain), 2.89 KB, created by Kyle M Hall (khall) on 2015-04-17 13:14:47 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2015-04-17 13:14:47 UTC

Size: 2.89 KB

patch

obsolete

>From 9c371efbcefc0606ed500d33c7366c6504d8238f Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@quecheelibrary.org>
>Date: Sat, 21 Mar 2015 11:31:14 -0400
>Subject: [PATCH] [PASSED QA] Bug 13886 - Add ability to merge records directly from cataloging search
>
>This patch adds a merge button and checkboxes to the cataloging search results screen to allow librarians to merge directly from these reults
>
>To test:
>1 - Apply patch
>2 - Perform a cataloging search
>3 - Try to select 1 or more than 2 records to merge and check that you get an error
>4 - Select 2 records to merge and confirm the correct records are passed to the merge tool
>
>Signed-off-by: Nicole Engard <nengard@bywatersolutions.com>
>
>Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
>---
> .../prog/en/modules/cataloguing/addbooks.tt        |   21 ++++++++++++++++++++
> 1 files changed, 21 insertions(+), 0 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbooks.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbooks.tt
>index 5619816..59380e2 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbooks.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbooks.tt
>@@ -18,6 +18,10 @@
>             PopupZ3950($(this).prop('id'));
>             return false;
>         });
>+        $(".merge-items").on("click",function(e){
>+            e.preventDefault();
>+            MergeItems();
>+        });
>      });
> 
>     /* this function open a popup to search on z3950 server.  */
>@@ -36,6 +40,20 @@
>         return strQuery;
>     }
> 
>+    /**
>+     * This function checks if the adequate number of records are checked for merging
>+     */
>+    function MergeItems() {
>+	var checkboxes = $("input:checkbox:checked");
>+        var nbCheckbox = checkboxes.length;
>+	if (nbCheckbox != 2) {
>+	    alert(_("Two records must be selected for merging."));
>+	} else {
>+	    location.href='/cgi-bin/koha/cataloguing/merge.pl?biblionumber=' + checkboxes[0].value + '&amp;biblionumber=' + checkboxes[1].value;
>+	}
>+	return false;
>+    }
>+
>     //]]>
> </script>
> </head>
>@@ -73,6 +91,7 @@
>                 [% END %]
>             </ul>
>         </div>
>+        <div class="btn-group"><button type="submit" class="btn btn-small merge-items"><i class="icon-merge"></i> Merge selected</button></div>
>   </div>
> [% END %]
> 
>@@ -89,6 +108,7 @@
> <div class="searchresults">
>     <table>
>         <tr>
>+            <th>&nbsp;</th>
>             <th>Title</th>
>             <th>Location</th>
>             <th>Preview</th>
>@@ -101,6 +121,7 @@
>         [% ELSE %]
>             <tr>
>         [% END %]
>+            <td><input type="checkbox" class="selection" id="bib[% resultsloo.biblionumber %]" name="biblionumber" value="[% resultsloo.biblionumber %]" /></td>
>             <td>
>                 <p>
>                     <a href="/cgi-bin/koha/catalogue/MARCdetail.pl?biblionumber=[% resultsloo.biblionumber %]">[% resultsloo.title |html %]</a>
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 13886: 37079 | 37346 | 37347 | 37348 | 37349 | 38052 | 38054 | 38817