Summary: | normalized_oclc not defined in opac-readingrecord.tt | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Architecture, internals, and plumbing | Assignee: | Bugs List <koha-bugs> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | nick, tomascohen |
Version: | unspecified | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
23.11.00,23.05.02,22.11.08
|
Circulation function: | |
Bug Depends on: | |||
Bug Blocks: | 33949 | ||
Attachments: |
Bug 33951: Set normalized_oclc in opac/opac-readingrecord.pl
Bug 33951: Set normalized_oclc in opac/opac-readingrecord.pl Bug 33951: (QA follow-up) Import GetNormalizedOCLCNumber |
Description
Jonathan Druart
2023-06-08 09:49:15 UTC
Created attachment 152141 [details] [review] Bug 33951: Set normalized_oclc in opac/opac-readingrecord.pl normalized_oclc is used in the template to build the link to syndetics, but it's not passed from the controller. Is this patch correct? Is syndetics broken on this page? Should we remove it or keep it? Is oclc parameter in the URL unecessary and should be removed? Created attachment 152157 [details] [review] Bug 33951: Set normalized_oclc in opac/opac-readingrecord.pl normalized_oclc is used in the template to build the link to syndetics, but it's not passed from the controller. Is this patch correct? Is syndetics broken on this page? Should we remove it or keep it? Is oclc parameter in the URL unecessary and should be removed? Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Created attachment 152158 [details] [review] Bug 33951: (QA follow-up) Import GetNormalizedOCLCNumber Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Trivial and straightforward, passing QA Pushed to master for 23.11. Nice work everyone, thanks! Thanks for all the hard work! Pushed to 23.05.x for the next release Nice work everyone! Pushed to oldstable for 22.11.x |