At the time this bug being filed there are only a handful of pages in the staff client which still require YUI assets. Pending patches cover most of these (listed as dependencies on this bug). There is one page, basketgroups.pl, which still doesn't have a patch to re-implement the drag-and-drop functionality in something other than YUI. I propose to remove YUI assets from the global include file and include them explicitly in the basketgroups template until that conversion can be done.
Created attachment 18562 [details] [review] Bug 9779 - Remove global include of YUI assets from the staff client There are no pages in the staff client which use YUI JS assets other than basketgroups.tt. The links to YUI assets can therefore be removed from the global include and put directly into the basket groups template. To test, view the basketgroups page and confirm using a JS debugger that no JS errors are triggered. Perform various basket group options: Open, close, modify, drag-and-drop. Operations should complete successfully and without JS errors.
Created attachment 18636 [details] [review] [SIGNED-OFF] Bug 9779 - Remove global include of YUI assets from the staff client There are no pages in the staff client which use YUI JS assets other than basketgroups.tt. The links to YUI assets can therefore be removed from the global include and put directly into the basket groups template. Signed-off-by: Srdjan <srdjan@catalyst.net.nz>
Created attachment 19047 [details] [review] [PASSED QA] Bug 9779 - Remove global include of YUI assets from the staff client There are no pages in the staff client which use YUI JS assets other than basketgroups.tt. The links to YUI assets can therefore be removed from the global include and put directly into the basket groups template. Signed-off-by: Srdjan <srdjan@catalyst.net.nz> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Passes all tests and QA script. Really nice to see YUI go. Checked that the drag&drop functionality in basket groups still works.
Pushed to master. Thanks, Owen! Besides basketgroup.tt, there are a few remaining references to JavaScript widgets in the YHAOO namespace: - koha-tmpl/intranet-tmpl/prog/en/modules/admin/stopwords.tt This template is used by a script that is no longer linked to the administration page, and per bug 9819 it looks like there's an active effort underway to remove the last of the legacy stopwords code. - koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt The only reference is in in a FIXME comment that can probably just be removed. - koha-tmpl/intranet-tmpl/prog/en/modules/course_reserves/course-reserves.tt The references are in commented-out code that appears to be the beginning of an attempt to add a course reserve search option to the global search box. This code can probably just be removed; adding such a search option for real can be the topic of a separate bug.