[Koha-bugs] [Bug 13247] opacuserjs is not compatible with tracking's tools like Piwik

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13247

--- Comment #2 from M. Tompsett <mtompset at hotmail.com> ---
Comment on attachment 33521
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=33521
proposed patch

Review of attachment 33521:
 --> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=13247&attachment=33521)
-----------------------------------------------------------------

::: koha-tmpl/opac-tmpl/bootstrap/en/includes/opac-bottom.inc
@@ +238,5 @@
>  [% PROCESS jsinclude %]
>  
> +<script type="text/javascript">
> +    //<![CDATA[
> +    [% IF ( opacuserjs ) %][% opacuserjs %][% END %]

Wouldn't it make more sense to put the IF and END around the entire script? Why
generate a hollow script section?

-- 
You are receiving this mail because:
You are watching all bug changes.