A couple scripts use constructs like this: eval "use C4::Acquisition qw/GetBasket/;"; I can think of no good reason to do this for standard C4 modules, especially since the code never checked to see if the eval succeeded.
This bug is mentioned in: bug 5399: remove useless eval 'use C4::Foo' http://lists.koha-community.org/pipermail/koha-patches/2010-November/012966.html
Signed patch (updated for current master)
Patch, and follow up patch pushed Would still like to see a fix to the use $pdfformat bit
This bug is mentioned in: Bug 5399: remove useless eval 'use C4::Foo' http://lists.koha-community.org/pipermail/koha-patches/2011-January/013608.html Bug 5399: remove useless eval 'use C4::Foo' http://lists.koha-community.org/pipermail/koha-patches/2011-January/013609.html