Bug 41665

Summary: Only include Greybox in OPAC if IdRef is enabled
Product: Koha Reporter: David Cook <dcook>
Component: OPACAssignee: David Cook <dcook>
Status: Signed Off --- QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: lari.taskula, oleonard
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32333
GIT URL: Initiative type: ---
Sponsorship status: --- Comma delimited list of Sponsors:
Crowdfunding goal: 0 Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Bug Depends on:    
Bug Blocks: 34710, 38366    
Attachments: Bug 41665: Only include Greybox in OPAC if IdRef is enabled
Bug 41665: Only include Greybox in OPAC if IdRef is enabled

Description David Cook 2026-01-20 00:26:15 UTC
Greybox is only used in the OPAC for the IdRef feature in the UNIMARC XSLT. So we should only include it in that context.
Comment 1 David Cook 2026-01-20 00:31:58 UTC
Created attachment 191659 [details] [review]
Bug 41665: Only include Greybox in OPAC if IdRef is enabled

This patch wraps the Greybox include in the OPAC with a syspref
check on IdRef, so that it's only loaded when it's needed.

Test plan:
0. Apply the patch
1. Go to http://localhost:8080/cgi-bin/koha/opac-detail.pl?biblionumber=29
2. Search through the HTML or the Sources tab in F12 tools and note
that Greybox assets are not loaded
3. Enable the IdRef system preference
4. Note that the Greybox assets are loaded
Comment 2 Owen Leonard 2026-01-20 15:41:51 UTC
Created attachment 191708 [details] [review]
Bug 41665: Only include Greybox in OPAC if IdRef is enabled

This patch wraps the Greybox include in the OPAC with a syspref
check on IdRef, so that it's only loaded when it's needed.

Test plan:
0. Apply the patch
1. Go to http://localhost:8080/cgi-bin/koha/opac-detail.pl?biblionumber=29
2. Search through the HTML or the Sources tab in F12 tools and note
   that Greybox assets are not loaded
3. Enable the IdRef system preference
4. Note that the Greybox assets are loaded

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 David Cook 2026-01-20 23:29:38 UTC
Legend. Thanks, Owen!