http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7310

Ian Walls <koha.sek...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Passed QA

--- Comment #65 from Ian Walls <koha.sek...@gmail.com> ---
Everything in the core patch looks really good.  While the exported subroutines
of C4/VirtualShelves.pm were changed greatly, no hanging calls were left in the
rest of the system.  The SQL and logic look clean.

One small thing: C4/Auth.pm removed the export of template param "bartotal",
but this is still used in one template, opac/opac-results.tt, to display a More
Lists option in the dropdown menu.  It doesn't appear that this param gets
exported anywhere else, so I believe it will fail to display the More Lists
option at any time.  I don't know if this is particularly relevant, and I'm not
sure how best to deal with it, but any fix for this can be handled with a
followup patch.

Marking as Passed QA.  Will reattach patches in correct order in a moment

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://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