http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9276 --- Comment #1 from Owen Leonard <oleonard@myacpl.org> --- Created attachment 14050 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14050&action=edit Bug 9276 - Display of biblio-level auth vals on OPAC search broken If you have your MARC framework configured to link a biblio-level field with an authorized value, and that value is linked to an image, and the AuthorisedValueImages system preference is turned on, the OPAC is supposed to display that image in search results much like item type images are displayed if item-level itemtypes are enabled. The switch to Template::Toolkit broke this feature with a variable scope error. This patch corrects the variable scope of the sytem preference check and adds a check for the existence of the image so that the template doesn't try to display broken images. To test: 1. Turn on the AuthorisedValueImages system preference. 2. If necessary, create or configure an authorized value with images. 3. Configure a MARC framework to link a field to the authorized value. I chose an arbitrary field, 942e. 4. Edit a bibliographic record and set a value for the MARC field you configured. 5. Make sure Zebra has reindexed, and perform an OPAC search which will include the record you edited. With AuthorisedValueImages ON, you should see the correct authorized value image. 6. With AuthorisedValueImages OFF, you should see no image. 7. Turn AuthorisedValueImages back on. Edit the authorized value again and configure it with no image. Perform your search again. You should see no image. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9276 Chris Cormack <chris@bigballofwax.co.nz> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #14050|0 |1 is obsolete| | --- Comment #2 from Chris Cormack <chris@bigballofwax.co.nz> --- Created attachment 14326 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=14326&action=edit Bug 9276 - Display of biblio-level auth vals on OPAC search broken If you have your MARC framework configured to link a biblio-level field with an authorized value, and that value is linked to an image, and the AuthorisedValueImages system preference is turned on, the OPAC is supposed to display that image in search results much like item type images are displayed if item-level itemtypes are enabled. The switch to Template::Toolkit broke this feature with a variable scope error. This patch corrects the variable scope of the sytem preference check and adds a check for the existence of the image so that the template doesn't try to display broken images. To test: 1. Turn on the AuthorisedValueImages system preference. 2. If necessary, create or configure an authorized value with images. 3. Configure a MARC framework to link a field to the authorized value. I chose an arbitrary field, 942e. 4. Edit a bibliographic record and set a value for the MARC field you configured. 5. Make sure Zebra has reindexed, and perform an OPAC search which will include the record you edited. With AuthorisedValueImages ON, you should see the correct authorized value image. 6. With AuthorisedValueImages OFF, you should see no image. 7. Turn AuthorisedValueImages back on. Edit the authorized value again and configure it with no image. Perform your search again. You should see no image. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9276 Jonathan Druart <jonathan.druart@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA CC| |jonathan.druart@biblibre.co | |m QA Contact| |jonathan.druart@biblibre.co | |m --- Comment #3 from Jonathan Druart <jonathan.druart@biblibre.com> --- QA Comment: Works as expected. Small patch, nothing to say. Marked as Passed QA. -- You are receiving this mail because: You are watching all bug changes.

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9276 Jared Camins-Esakov <jcamins@cpbibliography.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Passed QA |Pushed to Master --- Comment #5 from Jared Camins-Esakov <jcamins@cpbibliography.com> --- This patch has been pushed to master. -- You are receiving this mail because: You are watching all bug changes.