Hi,
On Wed, Feb 25, 2009 at 6:48 PM, Michael Hafen <[email protected]> wrote:
> This adds a system preference controlled feature. This controls how
> many returned items are shown on the check-in page.
This patch doesn't seem to be doing that - instead, it seems to be
establishing a limit on the number of items you can check in at once
from the patron details page, which is not desirable. See below:
> --- a/circ/returns.pl
> +++ b/circ/returns.pl
> @@ -82,11 +82,12 @@ my %returneditems;
> my %riduedate;
> my %riborrowernumber;
> my @inputloop;
> +my $returned_counter = ( C4::Context->preference('numReturnedItemsToShow') )
> ? C4::Context->preference('numReturnedItemsToShow') : 20;
> foreach ( $query->param ) {
> (next) unless (/ri-(\d*)/);
> my %input;
> my $counter = $1;
> - (next) if ( $counter > 20 );
> + (next) if ( $counter > $returned_counter );
This is in the query parameter processing loop, and says to stop
looking for additional items to return after the counter goes over the
limit. If you're looking to limit the number of returned items that
are displayed, should it not be doing in the loop that generates
@riloop?
Regards,
Galen
--
Galen Charlton
VP, Research & Development, LibLime
[email protected]
p: 1-888-564-2457 x709
skype: gmcharlt
_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha.org/mailman/listinfo/koha-patches
