From 3c7edd1cbef811de89d6ee0b54cd690dd36d6ff4 Mon Sep 17 00:00:00 2001 From: Owen Leonard Date: Fri, 14 Feb 2014 12:22:19 -0500 Subject: [PATCH] Bug 11764 - Move repeatedly redefined function toUC() into staff-global.js toUC() is repeatedly declared on many administration templates. This function, used to transform user input to uppercase, can be added to staff-global.js to prevent repetition. To test, confirm that transformation to uppercase is working on the following Administration pages when text is entered in a form field and focus is moved to the next field: - Authority types -> New: Test the "Authority type" field. - MARC bibliographic framework -> New framework: Test the "Framework code" field. - Patron types and categories -> New category: Test the "Category code" field. - Currencies and exchange rates -> New currency: Test the "Currency" field. - Item types -> New item type: Test the "Item type" field. - Z39.50 client targets -> New Z39.50 server: Test the "Z39.50 server" field. The following pages do not call the toUC function despite the fact that they included it: auth_tag_structure.tt printers.tt roadtype.tt stopwords.tt systempreferences.tt Signed-off-by: Chris Cormack Signed-off-by: Jonathan Druart tools/letter.tt declares this js function and don't use it. Signed-off-by: Galen Charlton --- koha-tmpl/intranet-tmpl/prog/en/js/staff-global.js | 6 ++++++ .../prog/en/modules/admin/auth_tag_structure.tt | 6 ------ koha-tmpl/intranet-tmpl/prog/en/modules/admin/authtypes.tt | 6 ------ .../intranet-tmpl/prog/en/modules/admin/biblio_framework.tt | 4 ---- koha-tmpl/intranet-tmpl/prog/en/modules/admin/categorie.tt | 6 ------ koha-tmpl/intranet-tmpl/prog/en/modules/admin/currency.tt | 5 ----- koha-tmpl/intranet-tmpl/prog/en/modules/admin/itemtypes.tt | 6 ------ koha-tmpl/intranet-tmpl/prog/en/modules/admin/printers.tt | 6 ------ koha-tmpl/intranet-tmpl/prog/en/modules/admin/roadtype.tt | 6 ------ koha-tmpl/intranet-tmpl/prog/en/modules/admin/stopwords.tt | 5 ----- .../prog/en/modules/admin/systempreferences.tt | 6 ------ .../intranet-tmpl/prog/en/modules/admin/z3950servers.tt | 6 ------ 12 files changed, 6 insertions(+), 62 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/js/staff-global.js b/koha-tmpl/intranet-tmpl/prog/en/js/staff-global.js index 234a8710f2..c97007f844 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/js/staff-global.js +++ b/koha-tmpl/intranet-tmpl/prog/en/js/staff-global.js @@ -86,3 +86,9 @@ function openWindow(link,name,width,height) { function removeFocus() { $(':focus').blur(); } + +function toUC(f) { + var x=f.value.toUpperCase(); + f.value=x; + return true; +} \ No newline at end of file diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt index aed41b61eb..7ecb5658fa 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/auth_tag_structure.tt @@ -28,12 +28,6 @@ return false; return true; } ///////////////////////////////////////////////////////////////////////////////////////////////////////////////// - function toUC(f) { - var x=f.value.toUpperCase(); - f.value=x; - return true; - } - ///////////////////////////////////////////////////////////////////////////////////////////////////////////////// function isNum(v,maybenull) { var n = new Number(v.value); if (isNaN(n)) { diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authtypes.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authtypes.tt index dab813e087..7823108360 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authtypes.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/authtypes.tt @@ -17,12 +17,6 @@ function isNotNull(f,noalert) { return true; } ///////////////////////////////////////////////////////////////////////////////////////////////////////////////// -function toUC(f) { - var x=f.value.toUpperCase(); - f.value=x; - return true; -} -///////////////////////////////////////////////////////////////////////////////////////////////////////////////// function Check(f) { var _alertString=""; var alertString2; diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt index e6ce9d8d3f..ec458b30e0 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/biblio_framework.tt @@ -16,10 +16,6 @@ function isNotNull(f,noalert) { } return true; } -function toUC(f) { - f.value = f.value.toUpperCase(); - return true; -} ///////////////////////////////////////////////////////////////////////////////////////////////////////////////// function Check(f) { var ok=1; diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/categorie.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/categorie.tt index ce7f0141c0..49b359a45a 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/categorie.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/categorie.tt @@ -35,12 +35,6 @@ return true; } // - function toUC(f) { - var x=f.value.toUpperCase(); - f.value=x; - return true; - } - // function isNum(v,maybenull) { var n = new Number(v.value); if (isNaN(n)) { diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/currency.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/currency.tt index 99f693b2f8..def9a023a8 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/currency.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/currency.tt @@ -7,11 +7,6 @@ [% INCLUDE 'doc-head-close.inc' %]