Attachment #157716 for bug #35139

View | Details | Raw Unified | Return to bug 35139
Collapse All | Expand All




my $op         = $input->param('op') || '';
my $borrowernumber = $input->param('borrowernumber');
my $reserve_id = $input->param('reserve_id');
my $all_branches   = $input->param('allbranches') || '';

my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
    {

    $where{'itembib.damaged'} = 0;
}

if ( C4::Context->preference('IndependentBranches') ) {
    undef $all_branches;
} else {
    my $all_branches_link = '/cgi-bin/koha/circ/pendingreserves.pl?allbranches=1';
    $all_branches_link .= '&from=' . $startdate if $startdate;
    $all_branches_link .= '&to=' . $enddate if $enddate;
    $template->param( all_branches_link => $all_branches_link ) unless $all_branches;
}
if ($all_branches) {
    $template->param( allbranches => 1 );
} else {
    $where{'me.branchcode'} = C4::Context->userenv->{'branch'};
}


Lines 58-63 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt (-1 / +8 lines)
58	[% END %]	58	[% END %]
59		59
60	<h1>Holds to pull placed between [% from \| $KohaDates %] and [% to \| $KohaDates %]</h1>	60	<h1>Holds to pull placed between [% from \| $KohaDates %] and [% to \| $KohaDates %]</h1>
		61	<h2>
		62	[% IF all_branches_link %]
		63	<span><a href="[% all_branches_link \| url %]">View all libraries</a></span>
		64	[% END %]
		65	</h2>
61	<h3>Reported on [% todaysdate \| $KohaDates %]</h3>	66	<h3>Reported on [% todaysdate \| $KohaDates %]</h3>
62	<p>The following holds have not been filled. Please retrieve them and check them in.</p>	67	<p>The following holds have not been filled. Please retrieve them and check them in.</p>
63	<div id="searchresults">	68	<div id="searchresults">
Lines 207-212 Link Here
207	<form name="cancelReserve" action="/cgi-bin/koha/circ/pendingreserves.pl" method="post">	212	<form name="cancelReserve" action="/cgi-bin/koha/circ/pendingreserves.pl" method="post">
208	<input type="hidden" name="op" value="cancel_reserve" />	213	<input type="hidden" name="op" value="cancel_reserve" />
209	<input type="hidden" name="reserve_id" value="[% hold.reserve_id \| html %]" />	214	<input type="hidden" name="reserve_id" value="[% hold.reserve_id \| html %]" />
		215	<input type="hidden" name="allbranches" value="[% allbranches \| html %]" />
210		216
211	[% SET hold_cancellation = AuthorisedValues.GetAuthValueDropbox('HOLD_CANCELLATION') %]	217	[% SET hold_cancellation = AuthorisedValues.GetAuthValueDropbox('HOLD_CANCELLATION') %]
212	[% IF hold_cancellation.count %]	218	[% IF hold_cancellation.count %]
Lines 232-237 Link Here
232	[% IF hold.itemnumber %]	238	[% IF hold.itemnumber %]
233	<form name="cancelReserve" action="/cgi-bin/koha/circ/pendingreserves.pl" method="post">	239	<form name="cancelReserve" action="/cgi-bin/koha/circ/pendingreserves.pl" method="post">
234	<input type="hidden" name="reserve_id" value="[% hold.reserve_id \| html %]" />	240	<input type="hidden" name="reserve_id" value="[% hold.reserve_id \| html %]" />
		241	<input type="hidden" name="allbranches" value="[% allbranches \| html %]" />
235	[% IF Koha.Preference('CanMarkHoldsToPullAsLost') == 'allow' %]	242	[% IF Koha.Preference('CanMarkHoldsToPullAsLost') == 'allow' %]
236	<input type="hidden" name="op" value="mark_as_lost" />	243	<input type="hidden" name="op" value="mark_as_lost" />
237	<input type="submit" class="btn btn-primary" value="Mark item as lost" />	244	<input type="submit" class="btn btn-primary" value="Mark item as lost" />
Lines 285-290 Link Here
285	<div id="filters">	292	<div id="filters">
286		293
287	<form action="/cgi-bin/koha/circ/pendingreserves.pl" method="post" >	294	<form action="/cgi-bin/koha/circ/pendingreserves.pl" method="post" >
		295	<input type="hidden" name="allbranches" value="[% allbranches \| html %]" />
288	<fieldset class="brief">	296	<fieldset class="brief">
289	<h4>Refine results</h4>	297	<h4>Refine results</h4>
290	<ol>	298	<ol>
291	-

Return to bug 35139

Lines 42-47 my $theme = $input->param('theme'); # only used if allowthemeoverride is set Link Here

(-)a/circ/pendingreserves.pl (-1 / +12 lines)
42	my $op = $input->param('op') \|\| '';	42	my $op = $input->param('op') \|\| '';
43	my $borrowernumber = $input->param('borrowernumber');	43	my $borrowernumber = $input->param('borrowernumber');
44	my $reserve_id = $input->param('reserve_id');	44	my $reserve_id = $input->param('reserve_id');
		45	my $all_branches = $input->param('allbranches') \|\| '';
45		46
46	my ( $template, $loggedinuser, $cookie ) = get_template_and_user(	47	my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
47	{	48	{
Lines 181-187 if ( !C4::Context->preference('AllowHoldsOnDamagedItems') ){ Link Here
181	$where{'itembib.damaged'} = 0;	182	$where{'itembib.damaged'} = 0;
182	}	183	}
183		184
184	if ( C4::Context->only_my_library() ){	185	if ( C4::Context->preference('IndependentBranches') ) {
		186	undef $all_branches;
		187	} else {
		188	my $all_branches_link = '/cgi-bin/koha/circ/pendingreserves.pl?allbranches=1';
		189	$all_branches_link .= '&from=' . $startdate if $startdate;
		190	$all_branches_link .= '&to=' . $enddate if $enddate;
		191	$template->param( all_branches_link => $all_branches_link ) unless $all_branches;
		192	}
		193	if ($all_branches) {
		194	$template->param( allbranches => 1 );
		195	} else {
185	$where{'me.branchcode'} = C4::Context->userenv->{'branch'};	196	$where{'me.branchcode'} = C4::Context->userenv->{'branch'};
186	}	197	}
187		198