[Koha-bugs] [Bug 7025] New: C4::Items depends on C4::Reserves (which in turn depends on C4::Items...)
bugzilla-daemon at bugs.koha-community.org
bugzilla-daemon at bugs.koha-community.org
Thu Oct 13 17:29:49 CEST 2011
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7025
Bug #: 7025
Summary: C4::Items depends on C4::Reserves (which in turn
depends on C4::Items...)
Classification: Unclassified
Change sponsored?: ---
Product: Koha
Version: rel_3_6
Platform: All
OS/Version: All
Status: NEW
Severity: normal
Priority: P5
Component: Architecture, internals, and plumbing
AssignedTo: ian.walls at bywatersolutions.com
ReportedBy: ian.walls at bywatersolutions.com
QAContact: koha-bugs at lists.koha-community.org
C4::Items uses a single subroutine from C4::Reserves, CheckReserves(), in a
single on of it's subroutines, GetItemsInfo(). The only data exported is in
the field "count_reserves" which gives the reserve status. This in turn is
only used in opac/opac-detail.pl to display "on hold" in the holdings table if
the item is currently waiting on the holds shelf to be picked up. There are no
other uses of this information.
We can remove CheckReserves() from GetItemsInfo(), and add a different call in
opac/opac-details.pl to get whether the particular item is on hold or not.
This will remove a circular dependency, likely speeding up the overall code (as
we won't need to load all of C4::Reserves every time we load C4::Items).
--
Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.
More information about the Koha-bugs
mailing list