showmarc.tt is only called by modal dialogs for previewing MARC records or card views in acquisitions and cataloging. Since it is a minimal page loaded within a page it doesn't need to load all the CSS and JS a full page does. It also creates the possibility of unintended consequences if you have intranetuserjs triggering something on document load: the event will be triggered again when the dialog loads.
Created attachment 10438 [details] [review] Bug 8288 - showmarc.tt should not include full doc-head-close.inc showmarc.tt and opac-showmarc.tt are used to load a plain view of a MARC record and thus do not need the full set of page assets loaded with other pages (CSS and JavaScript). Removing the standard doc-head-close include will reduce the load time of these pages.
Created attachment 10448 [details] [review] Bug 8288 - showmarc.tt should not include full doc-head-close.inc showmarc.tt and opac-showmarc.tt are used to load a plain view of a MARC record and thus do not need the full set of page assets loaded with other pages (CSS and JavaScript). Removing the standard doc-head-close include will reduce the load time of these pages. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Template only change to remove unnecessary code loading in the showmarc.tt context. Marking Passed QA.
Bugfix pushed to 3.8.x, will be in 3.8.3