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