Patch from Jeff Godin:

The "First 5 characters" transform appears to be broken due to a typo,
patch attached. To fix previously broken templates created using this
transform, clone, re-select the First 5 characters transform on any fields
where you were using it, and save the template.

Also in this patch, change from confirm() to alert() for "Template NAME
was successfully saved." No point in using confirm(), nothing was checking
the return value, etc.

git-svn-id: svn://svn.open-ils.org/ILS/branches/rel_1_4_0@12438 dcc99617-32d9-48b4-a31d-7c20da2025e4

diff --git a/Open-ILS/web/reports/xul/template-config.js b/Open-ILS/web/reports/xul/template-config.js
index 842c930..86a4bd6 100644 (file)
--- a/Open-ILS/web/reports/xul/template-config.js
+++ b/Open-ILS/web/reports/xul/template-config.js
@@ -804,7 +804,7 @@ function save_template () {
                                alertILSEvent(res);
                        } else {
                                if( res && res != '0' ) {
-                                       confirm(dojo.string.substitute( rpt_strings.TEMPLATE_CONF_SUCCESS_SAVE, [tmpl.name()] ));
+                                       alert(dojo.string.substitute( rpt_strings.TEMPLATE_CONF_SUCCESS_SAVE, [tmpl.name()] ));
                                        _l('../oils_rpt.xhtml');
                                }
                        }

diff --git a/Open-ILS/web/reports/xul/transforms.js b/Open-ILS/web/reports/xul/transforms.js
index 5191185..86abb53 100644 (file)
--- a/Open-ILS/web/reports/xul/transforms.js
+++ b/Open-ILS/web/reports/xul/transforms.js
@@ -91,7 +91,7 @@ var OILS_RPT_TRANSFORMS = {
                label : rpt_strings.TRANSFORMS_UPPER
        },
 
-       firt5 : {
+       first5 : {
                datatype : [ OILS_RPT_DTYPE_STRING ],
                label : rpt_strings.TRANSFORMS_FIRST5
        },