Bug 12103 - Move ajaxfileupload jQuery plugin outside of language-specific directory
Summary: Move ajaxfileupload jQuery plugin outside of language-specific directory
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: Main
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Owen Leonard
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-04-17 17:11 UTC by Owen Leonard
Modified: 2014-12-07 20:07 UTC (History)
2 users (show)

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


Attachments
Bug 12103 - Move ajaxfileupload jQuery plugin outside of language-specific directory (1.81 KB, patch)
2014-04-17 18:39 UTC, Owen Leonard
Details | Diff | Splinter Review
[Signed-off] Bug 12103 - Move ajaxfileupload jQuery plugin outside of language-specific directory (2.01 KB, patch)
2014-04-21 18:15 UTC, Marc Véron
Details | Diff | Splinter Review
[PASSED QA] Bug 12103 - Move ajaxfileupload jQuery plugin outside of language-specific directory (1.94 KB, patch)
2014-04-25 13:33 UTC, Kyle M Hall
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Owen Leonard 2014-04-17 17:11:29 UTC
The ajaxfileupload jQuery plugin should be moved to intranet-tmpl/lib/jquery/plugins so that it doesn't have to be duplicated for each translation.
Comment 1 Owen Leonard 2014-04-17 18:39:29 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2014-04-21 18:15:55 UTC Comment hidden (obsolete)
Comment 3 Kyle M Hall 2014-04-25 13:33:43 UTC
Created attachment 27625 [details] [review]
[PASSED QA] Bug 12103 - Move ajaxfileupload jQuery plugin outside of language-specific directory

This patch moves the ajaxfileupload jQuery plugin to
intranet-tmpl/prog/lib/jquery/plugins so that it will not be duplicated
for each set of translated templates.

To test, apply the patch and test a page which requires the file upload
plugin. For instance: Tools -> Upload local cover image. The upload
process should finish correctly.

Tested with patron and local cover images. Works as expected.
Signed-off-by: Marc Veron <veron@veron.ch>

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 4 Galen Charlton 2014-04-25 15:39:03 UTC
Pushed to master.  Thanks, Owen!