Bug 27272 is going to remove C4::Items::GetItemsInfo in favour of Koha::Items->search_ordered. Here we are going to deal with opac-sendbasket
Created attachment 138826 [details] [review] Bug 31316: Remove GetItemsInfo from opac-sendbasket Bug 27272 is going to remove C4::Items::GetItemsInfo in favour of Koha::Items->search. Here we are going to deal with opac-sendbasket Test plan: List items on the modified view and confirm that all the info is displayed correctly
Created attachment 138833 [details] [review] Bug 31316: Remove GetItemsInfo from opac-sendbasket Bug 27272 is going to remove C4::Items::GetItemsInfo in favour of Koha::Items->search. Here we are going to deal with opac-sendbasket Test plan: List items on the modified view and confirm that all the info is displayed correctly
Created attachment 138898 [details] [review] Bug 31316: Remove GetItemsInfo from opac-sendbasket Bug 27272 is going to remove C4::Items::GetItemsInfo in favour of Koha::Items->search. Here we are going to deal with opac-sendbasket Test plan: List items on the modified view and confirm that all the info is displayed correctly Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 138903 [details] [review] Bug 31316: Remove GetItemsInfo from opac-sendbasket Bug 27272 is going to remove C4::Items::GetItemsInfo in favour of Koha::Items->search. Here we are going to deal with opac-sendbasket Test plan: List items on the modified view and confirm that all the info is displayed correctly Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Created attachment 138904 [details] [review] Bug 31316: (QA follow-up) Remove stray template close tag Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
This actually improves the cart/basket - before these patches location was not translated from AV, now it is - I think this is correct
Pushed to master for 22.11. Nice work everyone, thanks!
(In reply to Nick Clemens (kidclamp) from comment #6) > This actually improves the cart/basket - before these patches location was > not translated from AV, now it is - I think this is correct Correct, also needed for staff interface I've opened Bug 36980