| Summary: | Remove GetItemsInfo from catalogue/detail.pl | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
| Component: | Architecture, internals, and plumbing | Assignee: | 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
|
|
| Circulation function: | |||
| 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
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 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> 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.) Created attachment 139115 [details] [review] Bug 31321: Fix 'Used in' Thanks, Joonas! Feel free to squash the follow-up patch. Created attachment 139116 [details] [review] Bug 31321: Fix copynumber 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> Pushed to master for 22.11. Nice work everyone, thanks! Enhancement will not be backported to 22.05.x series |