https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=38177

Pedro Amorim <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         QA Contact|[email protected] |[email protected]
                   |m                           |
           Assignee|[email protected]   |[email protected]
                   |                            |m
             Status|In Discussion               |Needs Signoff

--- Comment #7 from Pedro Amorim <[email protected]> ---
(In reply to Jonathan Druart from comment #4)
> I am not sure what you are doing here.

$c->stash('koha.pagination.per_page')
and
$c->stash('koha.pagination.page')
are undefined.

I'm not sure why, or if they should ever be defined. But I'm changing that to
use the params coming directly from the URL instead. This fixes the problem.
I'm obv open to other options.

(In reply to Jonathan Druart from comment #4)
> "All" results in 500

This problem exists before the patch, i.e. I think it's another problem.

(In reply to Jonathan Druart from comment #4)
> the problem still exists for packages and resources.

I missed this but it makes sense since it's the same code duplicated. Whatever
fix is pushed to Titles should also be done for Resources+Packages I reckon.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to