This includes are not used anymore: use C4::Context; use C4::ClassSource; use C4::Acquisition qw/GetOrderFromItemnumber ModOrder GetOrder/;
Created attachment 125747 [details] [review] Bug 29179: Remove useless includes in moveitem.pl This patch moves a trivially unneeded set of includes from this file. To test: 1. Apply the patch 2. Try attaching an item to another biblio => SUCCESS: Things work as expected! 3. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Created attachment 125750 [details] [review] Bug 29179: Remove useless includes in moveitem.pl This patch moves a trivially unneeded set of includes from this file. To test: 1. Apply the patch 2. Try attaching an item to another biblio => SUCCESS: Things work as expected! 3. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 125855 [details] [review] Bug 29179: Remove useless includes in moveitem.pl This patch moves a trivially unneeded set of includes from this file. To test: 1. Apply the patch 2. Try attaching an item to another biblio => SUCCESS: Things work as expected! 3. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Well, there are a lot of useless includes in a lot of files... If you want to tackle them down, please fix them all at once, not one report per file.
Pushed to master for 21.11, thanks to everybody involved!