Summary: | Add Koha::Biblio->opac_summary_html | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | martin.renvoize |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Small patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
23.11.00
|
Circulation function: | |
Bug Depends on: | 33953 | ||
Bug Blocks: | 33949, 33955, 33956 | ||
Attachments: |
Bug 33954: Add tests
Bug 33954: Koha::Biblio->opac_summary_html Bug 33954: Add tests Bug 33954: Koha::Biblio->opac_summary_html Bug 33954: Add tests Bug 33954: Koha::Biblio->opac_summary_html |
Description
Jonathan Druart
2023-06-08 10:11:49 UTC
Created attachment 152150 [details] [review] Bug 33954: Add tests Created attachment 152151 [details] [review] Bug 33954: Koha::Biblio->opac_summary_html On "Bug 33949: Replace GetAllIssues with Koha::Checkouts - opac" we are using this method from the template. This code is currently duplicated in controllers opac-readingrecord.pl and opac-user.pl. After bug 33949 it will be removed from opac-readingrecord.pl, and bug 33956 aims to remove it from opac-user.pl. Final situation will be: we have the code in a module, covered by tests \o/ Created attachment 153072 [details] [review] Bug 33954: Add tests Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 153073 [details] [review] Bug 33954: Koha::Biblio->opac_summary_html Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 153413 [details] [review] Bug 33954: Add tests Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Created attachment 153414 [details] [review] Bug 33954: Koha::Biblio->opac_summary_html Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Pushed to master for 23.11. Nice work everyone, thanks! Not required for anything in 23.05.x series yet, we can backport later if it becomes required. |