Attachment 12181 Details for Bug 8454 – Bug 8454 - Show pull list on load. Otherwise, it looks like the pull list is empty - Amended

[patch] Bug 8454 - Show pull list on load. Otherwise, it looks like the pull list is empty - Amended

0001-Bug-8454-Holds-to-Pull-Show-pull-list-on-load.-Other.patch (text/plain), 1.46 KB, created by David Cook on 2012-09-12 21:33:22 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Cook

Created: 2012-09-12 21:33:22 UTC

Size: 1.46 KB

patch

obsolete

>From 569bc49d5ddf4eff59bba982600c065c44b96294 Mon Sep 17 00:00:00 2001
>From: David Cook <dcook@prosentient.com.au>
>Date: Tue, 17 Jul 2012 11:33:35 +1000
>Subject: [PATCH] Bug 8454 - Holds to Pull : Show pull list on load.
> Otherwise, it looks like the pull list is empty.
>Content-Type: text/plain; charset="utf-8"
>
>Currently, when you click "Holds to pull" from the Circulation menu, the only thing that you see is a white screen and a side navigation bar that says "Refine Results".
>
>Since there appear to be no results to refine, the holds to pull list appears completely empty. Staff have to know to click "Submit" on the "Refine Results" form to get anything.
>
>I propose that we have the "run_report" flag set to ON for the initial load. The report uses the default 2 days mentioned in the "Refine Results" form, so there is continuity across the board.
>---
> circ/pendingreserves.pl |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/circ/pendingreserves.pl b/circ/pendingreserves.pl
>index 61bace9..d5bd273 100755
>--- a/circ/pendingreserves.pl
>+++ b/circ/pendingreserves.pl
>@@ -36,7 +36,7 @@ use Date::Calc qw/Today Add_Delta_YMD/;
> my $input = new CGI;
> my $startdate=$input->param('from');
> my $enddate=$input->param('to');
>-my $run_report=$input->param('run_report');
>+my $run_report = ( not defined $input->param('run_report') ) ? 1 : $input->param('run_report');
> 
> my $theme = $input->param('theme');    # only used if allowthemeoverride is set
> 
>-- 
>1.7.7.4
>

Actions: View | Diff | Splinter Review

Attachments on bug 8454: 10880 | 11499 | 12181 | 12182