Bug 13612

Summary: Remove old YUI javacript libraries from opac-tmpl
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: jonathan.druart, tomascohen, veron
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13689
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 13689    
Bug Blocks:    
Attachments: Bug 13612 - Remove old YUI javacript libraries from opac-tmpl
[Signed-off] Bug 13612 - Remove old YUI javacript libraries from opac-tmpl
Bug 13612 - Remove old YUI javacript libraries from opac-tmpl

Description Katrin Fischer 2015-01-22 14:21:36 UTC
After checking with Owen it appears that the opac-tmpl/lib/yui directory can be removed now that we only support Bootstrap OPAC.
Comment 1 Owen Leonard 2015-01-22 16:38:55 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2015-02-04 13:35:24 UTC
I did git grep for the files all removed files, no references behalve of:

git grep 'opac-tmpl/lib/yui/reset-fonts-grids.css'
Result:
koha-tmpl/opac-tmpl/bootstrap/css/opac-old.css
koha-tmpl/opac-tmpl/bootstrap/css/sco-old.css

git grep 'opac-tmpl/lib/yui/skin.css'
Result:
koha-tmpl/opac-tmpl/bootstrap/css/opac-old.css
koha-tmpl/opac-tmpl/bootstrap/css/sco-old.css

Are opac-old.css and sco-old.css still used / present in the system or is this just a local issue in my dev installation?
Comment 3 Katrin Fischer 2015-02-08 21:13:44 UTC
Hi Marc, 

I have the same result, but the css files don't seem to be linked to from anywhere. My guess is, that we kept them around for look-up, but that they are not used any longer. Owen, could you confirm?
Comment 4 Owen Leonard 2015-02-10 14:06:53 UTC
> Owen, could you confirm?

I confirm. opac-old.css and sco-old.css were in-development reference files and shouldn't have made it into the original patch. They can be ignored for the purposes of this patch and deleted by another one.
Comment 5 Marc Véron 2015-02-10 14:44:20 UTC Comment hidden (obsolete)
Comment 6 Jonathan Druart 2015-02-11 10:20:51 UTC
Created attachment 35822 [details] [review]
Bug 13612 - Remove old YUI javacript libraries from opac-tmpl

YUI files were left behind in opac-tmpl/lib/yui after the removal of the
prog theme. These files are unused and can be safely removed.

To test, apply the patch and search for references to any of the YUI
files. You should find none in the OPAC templates.

No references found.
Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 7 Jonathan Druart 2015-02-11 10:22:05 UTC
(In reply to Owen Leonard from comment #4)
> > Owen, could you confirm?
> 
> I confirm. opac-old.css and sco-old.css were in-development reference files
> and shouldn't have made it into the original patch. They can be ignored for
> the purposes of this patch and deleted by another one.

I am marking this bug depends on bug 13689, IMO they should be pushed together.
Comment 8 Tomás Cohen Arazi 2015-02-24 17:25:11 UTC
Patch pushed to master.

Thanks Owen!