Attachment 21202 Details for Bug 2693 – Bug 2693 - Tagging: Add filter by name

[patch] Bug 2693 - Tagging: Add filter by name

Bug-2693---Tagging-Add-filter-by-name.patch (text/plain), 3.25 KB, created by Owen Leonard on 2013-09-18 12:47:08 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2013-09-18 12:47:08 UTC

Size: 3.25 KB

patch

obsolete

>From ba94137bb6801c953b65f0f7e37829ce2a9bf567 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 21 Aug 2013 10:53:20 -0400
>Subject: [PATCH] Bug 2693 - Tagging: Add filter by name
>Content-Type: text/plain; charset="utf-8"
>
>If you want to filter results on the tags management page by the
>reviewer you have to know their borrowernumber. It would be better to be
>able to search by name, but in the meantime this patch adds an
>autocomplete widget to the form field so that you can do a name search
>and populate the field with the borrowernumber.
>
>To test you must have approved or rejected tags. From the tags review
>page (tags/review.pl), view all, approved, or rejected tags. Type a name
>in the "reviewer" form field in the left sidebar filter. You should get
>an autocomplete dropdown with names matching your search. When you
>select one the form field should be populated by the borrowernumber.
>---
> circ/ysearch.pl                                    |    5 +++--
> .../intranet-tmpl/prog/en/modules/tags/review.tt   |   15 +++++++++++++++
> 2 files changed, 18 insertions(+), 2 deletions(-)
>
>diff --git a/circ/ysearch.pl b/circ/ysearch.pl
>index 4a3663e..7429c06 100755
>--- a/circ/ysearch.pl
>+++ b/circ/ysearch.pl
>@@ -45,7 +45,7 @@ if ($auth_status ne "ok") {
> 
> my $dbh = C4::Context->dbh;
> my $sql = q(
>-    SELECT surname, firstname, cardnumber, address, city, zipcode, country
>+    SELECT borrowernumber, surname, firstname, cardnumber, address, city, zipcode, country
>     FROM borrowers
>     WHERE ( surname LIKE ?
>         OR firstname LIKE ?
>@@ -68,7 +68,8 @@ print "[";
> my $i = 0;
> while ( my $rec = $sth->fetchrow_hashref ) {
>     if($i > 0){ print ","; }
>-    print "{\"surname\":\"" . $rec->{surname} . "\",\"" .
>+    print "{\"borrowernumber\":\"" . $rec->{borrowernumber} . "\",\"" .
>+          "surname\":\"".$rec->{surname} . "\",\"" .
>           "firstname\":\"".$rec->{firstname} . "\",\"" .
>           "cardnumber\":\"".$rec->{cardnumber} . "\",\"" .
>           "address\":\"".$rec->{address} . "\",\"" .
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/tags/review.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/tags/review.tt
>index 89c998a..19f3e49 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/tags/review.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/tags/review.tt
>@@ -121,6 +121,21 @@ td input,td input[type="submit"] { font-size: 85%; padding: 1px; }
> 		$("*").ajaxError(function(evt, request, settings){
> 			if ((alerted +=1) <= 1){ window.alert(_("AJAX error")+" (" + alerted + " alert)"); }
> 		});
>+        var reviewerField = $("#approver");
>+        reviewerField.autocomplete({
>+            source: "/cgi-bin/koha/circ/ysearch.pl",
>+            minLength: 3,
>+            select: function( event, ui ) {
>+                reviewerField.val( ui.item.borrowernumber );
>+                return false;
>+            }
>+        })
>+        .data( "autocomplete" )._renderItem = function( ul, item ) {
>+            return $( "<li></li>" )
>+            .data( "item.autocomplete", item )
>+            .append( "<a>" + item.surname + ", " + item.firstname + " (" + item.cardnumber + ") <small>" + item.address + " " + item.city + " " + item.zipcode + " " + item.country + "</small></a>" )
>+            .appendTo( ul );
>+        };
> 	});
> //]]>
> </script>
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 2693: 796 | 797 | 20539 | 21202 | 22085 | 22289 | 22780