Bug 2910 - greybox js library is incorrectly associated with Google Jackets pref
Summary: greybox js library is incorrectly associated with Google Jackets pref
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: PATCH-Sent (DO NOT USE) normal (vote)
Assignee: Owen Leonard
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-12 06:53 UTC by Owen Leonard
Modified: 2012-10-25 22:52 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Proposed patch (1.63 KB, patch)
2009-01-12 06:54 UTC, Chris Cormack
Details | Diff | Splinter Review
Proposed fix for 3.0.x (555.52 KB, patch)
2010-08-10 14:52 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Cormack 2010-05-21 01:02:14 UTC


---- Reported by oleonard@myacpl.org 2009-01-12 06:53:59 ----

The template loads the greybox library only if Google Jackets are enabled. The library should be loaded only if XSLTDetailsDisplay is on.



---- Additional Comments From oleonard@myacpl.org 2009-01-12 06:54:59 ----

Created an attachment
Proposed patch





---- Additional Comments From oleonard@myacpl.org 2009-10-07 18:58:02 ----

Alternate patch accepted on 26 Feb 2009. http://git.koha.org/cgi-bin/gitweb.cgi?p=Koha;a=commit;h=7d2875ae4dba4f1b12cad6951cff40951d9a7767



--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:02 UTC  ---

This bug was previously known as _bug_ 2910 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=2910
Imported an attachment (id=941)

Actual time not defined. Setting to 0.0
The original submitter of attachment 941 [details] [review] is unknown.
   Reassigning to the person who moved it here: chris@bigballofwax.co.nz.

Comment 1 Owen Leonard 2010-08-10 14:52:38 UTC
Created attachment 2498 [details] [review]
Proposed fix for 3.0.x

Patch also includes a fix for Bug 5102