[Koha-bugs] [Bug 6875] New: de-nesting C4 packages

[bugzilla-daemon at bugs.koha-community.org]

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

             Bug #: 6875
           Summary: de-nesting C4 packages
    Classification: Unclassified
 Change sponsored?: ---
           Product: Koha
           Version: rel_3_6
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P5
         Component: Architecture, internals, and plumbing
        AssignedTo: gmcharlt at gmail.com
        ReportedBy: paul.poulain at biblibre.com
         QAContact: koha-bugs at lists.koha-community.org


Created attachment 5442
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=5442
script to check which package used in a package is useless

C4 packages are using use C4::another package a lot. It result in all packages
being loaded (& compiled) on each page.
This bugzilla entry will contain the work to remove as much as possible the
dependancies

That will be achieved through :
* removing use C4::otherpackage and using require C4::otherpackage in the subs
that need C4::otherpackage. Require is loaded at run time only. Most of the
time, the C4::otherpackage is loaded everytime, just to be used once, in one
sub (that is rarely used)
* removing useless subs that use a C4::otherpackage::sub, because there are
some

Cleaning for each package will result in a patch, with a lot of
comments/details/feedback.
To discover which sub is used by which script, I'll use the attached script.

-- 
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.