Bug 6875 de nesting C4::Biblio
authorPaul Poulain <paul.poulain@biblibre.com>
Fri, 16 Sep 2011 17:37:15 +0000 (19:37 +0200)
committerPaul Poulain <paul.poulain@biblibre.com>
Mon, 20 Feb 2012 15:35:17 +0000 (16:35 +0100)
commit49b167e8486e19a20f055e1ce9cde30f82cfc55e
tree4add8998012f5674002691b8bb97c292c36ddce5
parent1ddc9322dfc13ef7f1e930692bfd2cb3771b102b
Bug 6875 de nesting C4::Biblio

C4::Biblio is used in many many places. The goal of this cleaning is to do from C4::Biblio a package with as many dependancies as possible.

* C4::Heading is called only in 1 place, highly rarely used (only in 1 misc/link_bibs_to_authorities.pl), moving to require
* PrepareItemrecordDisplay is a sub that is more related to Items, moving it here. It means some scripts that used this sub must be checked against use C4::Items
* C4::Items is needed in EmbedItemsInMarcBiblio, moving it only in this sub, and switching to require
* 2 subs are totally useless z3950_extended_services and set_service_options, removing them

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
No test plan included, but tested some opac and cataloging functions.
Code looks good. Marked as Passed QA.
C4/Biblio.pm
C4/Items.pm
C4/Serials.pm
acqui/neworderempty.pl
t/db_dependent/lib/KohaTest/Biblio.pm
t/db_dependent/lib/KohaTest/Items.pm