Attachment 118883 Details for Bug 28043 – Bug 28043: Some patron clubs operations don't work from later pages of results

[patch] Bug 28043: Some patron clubs operations don't work from later pages of results

0001-Bug-28043-Some-patron-clubs-operations-don-t-work-fr.patch (text/plain), 2.53 KB, created by Amit Gupta on 2021-03-27 08:15:06 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Amit Gupta

Created: 2021-03-27 08:15:06 UTC

Size: 2.53 KB

patch

obsolete

>From 47c8d37036ab4a09cf2d5607aee60cb0fec9f8fd Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Fri, 26 Mar 2021 18:36:19 +0000
>Subject: [PATCH] Bug 28043: Some patron clubs operations don't work from later
> pages of results
>
>This page changes the way event handlers are defined for some operations
>on the patron clubs page. These changes make it possible for the
>controls to work on multi-page tables of clubs.
>
>To test, apply the patch and got to Tools -> Patron clubs.
>
>- If necessary, add enough patron club templates and patron clubs that
>  there are more than 10 of each.
>- Click to the second page of club templates and try to delete a
>  template. It should work correctly.
>- Click to the second page of clubs and try to delete a club. It should
>  work correctly.
>- Add one or more patrons to a club (from the checkout or patron details
>  page).
>- Click to the second page of clubs and try the "Search to hold" menu
>  item. It should redirect you to the search page.
>- After entering a search, the results page should include "Place hold
>  for <club>" links with each result.
>
>Signed-off-by: Amit Gupta <amitddng135@gmail.com>
>---
> koha-tmpl/intranet-tmpl/prog/en/modules/clubs/clubs.tt | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/clubs/clubs.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/clubs/clubs.tt
>index f7db215..799ff6f 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/clubs/clubs.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/clubs/clubs.tt
>@@ -160,21 +160,21 @@
>                 ]
>             } ));
> 
>-            $(".delete_club").on("click", function(e){
>+            $("body").on("click", ".delete_club", function(e){
>                 e.preventDefault();
>                 var club_id = $(this).data("id");
>                 var club_name = $(this).data("name");
>                 ConfirmDeleteClub( club_id, club_name );
>             });
> 
>-            $(".delete_template").on("click", function(e){
>+            $("body").on("click", ".delete_template", function(e){
>                 e.preventDefault();
>                 var template_id = $(this).data("id");
>                 var template_name = $(this).data("name");
>                 ConfirmDeleteTemplate( template_id, template_name );
>             });
> 
>-            $(".club_hold_search").on("click", function(e){
>+            $("body").on("click", ".club_hold_search", function(e){
>                 e.preventDefault;
>                 var club_id = $(this).data("id");
>                 SearchToHold( club_id );
>-- 
>2.7.4
>

Actions: View | Diff | Splinter Review

Attachments on bug 28043: 118876 | 118883 | 118890