Bug 31321

Summary: Remove GetItemsInfo from catalogue/detail.pl
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Joonas Kylmälä <joonas.kylmala>
Severity: enhancement    
Priority: P5 - low CC: joonas.kylmala, lucas, nick
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
22.11.00
Bug Depends on: 31306    
Bug Blocks: 27272, 32908    
Attachments: Bug 31321: Remove GetItemsInfo from catalogue/detail.pl
Bug 31321: Remove GetItemsInfo from catalogue/detail.pl
Bug 31321: Fix 'Used in'
Bug 31321: Fix copynumber
Bug 31321: Remove GetItemsInfo from catalogue/detail.pl

Description Jonathan Druart 2022-08-08 14:23:17 UTC
Bug 27272 is going to remove C4::Items::GetItemsInfo in favour of Koha::Items->search_ordered.

Here we are going to deal with catalogue/detail.pl
Comment 1 Jonathan Druart 2022-08-08 14:53:43 UTC
Created attachment 138831 [details] [review]
Bug 31321: Remove GetItemsInfo from catalogue/detail.pl

Bug 27272 is going to remove C4::Items::GetItemsInfo in favour of Koha::Items->search_ordered.

Here we are going to deal with catalogue/detail

Test plan:
List items on the modified view and confirm that all the info is
displayed correctly
Comment 2 Owen Leonard 2022-08-09 13:30:24 UTC
Created attachment 138913 [details] [review]
Bug 31321: Remove GetItemsInfo from catalogue/detail.pl

Bug 27272 is going to remove C4::Items::GetItemsInfo in favour of Koha::Items->search_ordered.

Here we are going to deal with catalogue/detail

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>
Comment 3 Joonas Kylmälä 2022-08-14 14:50:46 UTC
After applying this patch the link to analytical record is not shown in the items list of the detail.pl page. To reproduce:

1) Apply patch
2) Enable EasyAnalyticalRecords syspref
3) Go to any biblio's page with items
4) Click New => analyze items.
5) Click on the link on any item that says "Create analytics" and proceed with the record creation.
6) Go back to the biblio page and notice "Used in" tab doesn't contain the link to the analytics record (without this patch it contains.)
Comment 4 Jonathan Druart 2022-08-15 08:06:22 UTC
Created attachment 139115 [details] [review]
Bug 31321: Fix 'Used in'
Comment 5 Jonathan Druart 2022-08-15 08:06:59 UTC
Thanks, Joonas!
Feel free to squash the follow-up patch.
Comment 6 Jonathan Druart 2022-08-15 08:08:10 UTC
Created attachment 139116 [details] [review]
Bug 31321: Fix copynumber
Comment 7 Joonas Kylmälä 2022-08-19 20:58:25 UTC
Created attachment 139523 [details] [review]
Bug 31321: Remove GetItemsInfo from catalogue/detail.pl

Bug 27272 is going to remove C4::Items::GetItemsInfo in favour of Koha::Items->search_ordered.

Here we are going to deal with catalogue/detail

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>
JK: Amend follow-up fixes from Joubu to this patch

Signed-off-by: Joonas Kylmälä <joonas.kylmala@iki.fi>
Comment 8 Tomás Cohen Arazi 2022-08-22 16:19:06 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!
Comment 9 Lucas Gass 2022-10-14 19:52:03 UTC
Enhancement will not be backported to 22.05.x series