|
Description
Jonathan Druart
2021-08-31 13:22:20 UTC
Created attachment 124283 [details] [review] Bug 28931: Use EXPORT_OK from Koha::AuthUtils Created attachment 124750 [details] [review] Bug 28931: Use EXPORT_OK from Koha::AuthUtils It has been missed on bug 17600. Got the list using % grep --files-without-match "use Koha::DateUtils.*dt_from_string" `git grep dt_from_string|cut -d':' -f1|sort|uniq` Created attachment 124870 [details] [review] Bug 28931: Use EXPORT_OK from Koha::AuthUtils It has been missed on bug 17600. Signed-off-by: Nick Clemens <nick@bywatersolutions.com> There was no test plan...
Should these files be updated as well?
debian/templates/plack.psgi:use Koha::DateUtils;
t/db_dependent/Circulation/IsItemIssued.t:use Koha::DateUtils;
t/db_dependent/Items.t:use Koha::DateUtils;
t/db_dependent/Koha/SearchEngine/Indexer.t:use Koha::DateUtils;
t/db_dependent/Koha/Template/Plugin/TablesSettings.t:use Koha::DateUtils;
t/db_dependent/Log.t:use Koha::DateUtils;
t/db_dependent/SIP/ILS.t:use Koha::DateUtils;
t/db_dependent/Search/History.t:use_ok('Koha::DateUtils');
Created attachment 125686 [details] [review] Bug 28931: Remove unuse Koha::DateUtils from tests (In reply to Katrin Fischer from comment #5) > There was no test plan... It's impossible to provide a full test plan for this kind of patches. If the QA script passes it's already a good step for this one, as the compilation will be checked. > Should these files be updated as well? > t/db_dependent/Circulation/IsItemIssued.t:use Koha::DateUtils; > t/db_dependent/Items.t:use Koha::DateUtils; > t/db_dependent/Koha/SearchEngine/Indexer.t:use Koha::DateUtils; > t/db_dependent/Koha/Template/Plugin/TablesSettings.t:use Koha::DateUtils; > t/db_dependent/Log.t:use Koha::DateUtils; > t/db_dependent/SIP/ILS.t:use Koha::DateUtils; > t/db_dependent/Search/History.t:use_ok('Koha::DateUtils'); We can remove the use statement as it's not used. > debian/templates/plack.psgi:use Koha::DateUtils; I don't know about this one, we preload it but without the import. I prefer to keep it like that. Created attachment 125857 [details] [review] Bug 28931: Use EXPORT_OK from Koha::DateUtils It has been missed on bug 17600. Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Adjusted commit title. Created attachment 125858 [details] [review] Bug 28931: Remove unused Koha::DateUtils from tests Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Pushed to master for 21.11, thanks to everybody involved! |