Bug 14265 - Use $.trim instead of trim() in admin/categorie.tt
authorDavid Cook <dcook@prosentient.com.au>
Mon, 25 May 2015 04:01:09 +0000 (14:01 +1000)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Mon, 1 Jun 2015 17:16:21 +0000 (14:16 -0300)
This patch replaces trim() with $.trim() in admin/categorie.tt, as
$.trim() is more cross-platform (ie it is supported in "< IE9").

_TEST PLAN_

Before applying patch:

0) Use IE 8 or use Document Mode 8 in a newer IE using F12 Developer Tools
1) Go to Administration > Patron categories
2) Click "Edit" next to a category
3) Click "Save" at the bottom
4) Note the form doesn't submit (you can also notice the error in the
console log)

Apply the patch:

5) Hold down shift + refresh the page
6) Click "Save" at the bottom"
7) Note that the form does submit and there are no errors reported

Signed-off-by: Indranil Das Gupta <indradg@gmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
No regression found using iceweasel.
I did not know the staff interface was IE compatible.
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>

koha-tmpl/intranet-tmpl/prog/en/modules/admin/categorie.tt

index 769a0bc..92a6a28 100644 (file)
@@ -47,7 +47,7 @@
            var ok=0;
                var _alertString=_("Form not submitted because of the following problem(s)");
                _alertString +="\n-------------------------------------------------------------------\n\n";
-                ff.categorycode.value = ff.categorycode.value.trim();
+                ff.categorycode.value = $.trim(ff.categorycode.value);
                 if (ff.categorycode.value.length==0) {
                    ok=1;
                    _alertString += _("- categorycode missing") + "\n";