You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ms...@apache.org on 2011/08/18 20:26:53 UTC

svn commit: r1159348 - in /myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main: java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/ java/org/apache/myfaces/trinidadinternal/resource/ javascript/META-INF/adf/jsLibs/ reso...

Author: mstarets
Date: Thu Aug 18 18:26:52 2011
New Revision: 1159348

URL: http://svn.apache.org/viewvc?rev=1159348&view=rev
Log:
TRINIDAD-2100 - Trinidad's Library Scriptlet Mechansim Should Support More Selective JS Inclusion

Added:
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreFormatScriptsResourceLoader.java
      - copied unchanged from r1102874, myfaces/trinidad/trunk/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreFormatScriptsResourceLoader.java
Modified:
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/XhtmlScriptletFactory.java
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreCommonScriptsResourceLoader.java
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreRenderKitResourceLoader.java
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/LocaleElementsResourceLoader.java
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/ColorFormat.js
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/CoreFormat.js
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/DateFormat.js
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Locale.js
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js
    myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/resources/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/javascriptmap.properties

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/XhtmlScriptletFactory.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/XhtmlScriptletFactory.java?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/XhtmlScriptletFactory.java (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/XhtmlScriptletFactory.java Thu Aug 18 18:26:52 2011
@@ -100,7 +100,7 @@ public class XhtmlScriptletFactory
   static
   {
     _sLocaleScriptlet =
-        new AliasedScriptlet(LOCALE_LIB, new String[]{
+        new LibraryScriptlet(LOCALE_LIB, new String[]{
                               "getUserLanguage()",
                               "getJavaLanguage()",
                               "TrConverter()", 
@@ -113,11 +113,8 @@ public class XhtmlScriptletFactory
                               "isUpperCase()",
                               "isNotUpperCase()",
                               "isLetter()",
-                              "getLocaleSymbols()"},
-                            new String[]
-                            {
-                              CORE_LIB
-                            });
+                              "getLocaleSymbols()"}
+                             );
 
     //    new LocaleInfoScriptlet();
     //new DateFormatInfoScriptlet();
@@ -139,7 +136,8 @@ public class XhtmlScriptletFactory
                            },
                            new String[]
                            {
-                             GlobalVariablesScriptlet.GLOBAL_VARIABLES_KEY
+                             GlobalVariablesScriptlet.GLOBAL_VARIABLES_KEY,
+                             LOCALE_LIB
                            })
       {
         // =-=AEW  The Core library needs to be rendered even
@@ -153,7 +151,7 @@ public class XhtmlScriptletFactory
       };
 
     _sDateFormatScriptlet =
-      new AliasedScriptlet(DATE_FORMAT_LIB,
+      new LibraryScriptlet(DATE_FORMAT_LIB,
                            new String[]
                            {
                              "TrDateTimeConverter()",
@@ -216,7 +214,7 @@ public class XhtmlScriptletFactory
                              "TableProxy()",
                            });
    _sColorFormatScriptlet =
-      new AliasedScriptlet(COLOR_FORMAT_LIB,
+      new LibraryScriptlet(COLOR_FORMAT_LIB,
                            new String[]
                            {
                              "TrColor()",

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreCommonScriptsResourceLoader.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreCommonScriptsResourceLoader.java?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreCommonScriptsResourceLoader.java (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreCommonScriptsResourceLoader.java Thu Aug 18 18:26:52 2011
@@ -18,6 +18,9 @@
  */
 package org.apache.myfaces.trinidadinternal.resource;
 
+import java.io.IOException;
+
+import java.net.URL;
 import java.net.URLConnection;
 import org.apache.myfaces.trinidad.resource.AggregatingResourceLoader;
 import org.apache.myfaces.trinidad.resource.ClassLoaderResourceLoader;
@@ -29,10 +32,9 @@ import org.apache.myfaces.trinidad.resou
 public class CoreCommonScriptsResourceLoader extends AggregatingResourceLoader
 {
   public CoreCommonScriptsResourceLoader(
-    String  path,
     boolean debug)
   {
-    super(path,
+    super("",
           debug ? _DEBUG_LIBRARIES : _LIBRARIES,
           new ClassLoaderResourceLoader());
 
@@ -42,6 +44,16 @@ public class CoreCommonScriptsResourceLo
     // block comment "/*"
     setSeparator(_NEWLINE_SEPARATOR);
   }
+  
+  /**
+   * Since CoreRenderKitResourceLoader already does the matching, this method is overridden
+   * to just call getURL()
+   */
+  @Override
+  protected URL findResource(String path) throws IOException
+  {
+    return getURL(path);
+  }
 
   @Override
   protected String getContentType(
@@ -53,15 +65,8 @@ public class CoreCommonScriptsResourceLo
   // List of all libraries
   static private final String[] _LIBRARIES =
   {
-    "META-INF/adf/jsLibs/CharSets.js",
-    "META-INF/adf/jsLibs/NumberFormat.js",
-    "META-INF/adf/jsLibs/NumberConverter.js",
-    "META-INF/adf/jsLibs/CoreFormat.js",
-    "META-INF/adf/jsLibs/TrCollections.js",
     "META-INF/adf/jsLibs/DateField.js",
     "META-INF/adf/jsLibs/DateFieldFormat.js",
-    "META-INF/adf/jsLibs/DateFormat.js",
-    "META-INF/adf/jsLibs/Locale.js",
     "META-INF/adf/jsLibs/MessageBox.js",
     "META-INF/adf/jsLibs/Core.js",
     "META-INF/adf/jsLibs/Window.js",
@@ -70,7 +75,6 @@ public class CoreCommonScriptsResourceLo
     "META-INF/adf/jsLibs/Poll.js",
     "META-INF/adf/jsLibs/ColorField.js",
     "META-INF/adf/jsLibs/ColorFieldFormat.js",
-    "META-INF/adf/jsLibs/ColorFormat.js",
     "META-INF/adf/jsLibs/Shuttle.js",
     "META-INF/adf/jsLibs/PanelPopup.js",
     "META-INF/adf/jsLibs/PopupDialog.js",
@@ -86,14 +90,14 @@ public class CoreCommonScriptsResourceLo
   // List of all libraries
   static private final String[] _DEBUG_LIBRARIES =
   {
-    "META-INF/adf/jsLibsDebug/CharSets.js",
-    "META-INF/adf/jsLibsDebug/NumberFormat.js",
-    "META-INF/adf/jsLibsDebug/NumberConverter.js",
+    //"META-INF/adf/jsLibsDebug/CharSets.js",
+    //"META-INF/adf/jsLibsDebug/NumberFormat.js",
+    //"META-INF/adf/jsLibsDebug/NumberConverter.js",
     "META-INF/adf/jsLibsDebug/TrCollections.js",
-    "META-INF/adf/jsLibsDebug/CoreFormat.js",
+    //"META-INF/adf/jsLibsDebug/CoreFormat.js",
     "META-INF/adf/jsLibsDebug/DateField.js",
     "META-INF/adf/jsLibsDebug/DateFieldFormat.js",
-    "META-INF/adf/jsLibsDebug/DateFormat.js",
+    //"META-INF/adf/jsLibsDebug/DateFormat.js",
     "META-INF/adf/jsLibsDebug/Locale.js",
     "META-INF/adf/jsLibsDebug/MessageBox.js",
     "META-INF/adf/jsLibsDebug/Core.js",
@@ -103,7 +107,7 @@ public class CoreCommonScriptsResourceLo
     "META-INF/adf/jsLibsDebug/Poll.js",
     "META-INF/adf/jsLibsDebug/ColorField.js",
     "META-INF/adf/jsLibsDebug/ColorFieldFormat.js",
-    "META-INF/adf/jsLibsDebug/ColorFormat.js",
+    //"META-INF/adf/jsLibsDebug/ColorFormat.js",
     "META-INF/adf/jsLibsDebug/Shuttle.js",
     "META-INF/adf/jsLibsDebug/PanelPopup.js",
     "META-INF/adf/jsLibsDebug/PopupDialog.js",

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreRenderKitResourceLoader.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreRenderKitResourceLoader.java?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreRenderKitResourceLoader.java (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/CoreRenderKitResourceLoader.java Thu Aug 18 18:26:52 2011
@@ -1,4 +1,4 @@
-/*
+ /*
  *  Licensed to the Apache Software Foundation (ASF) under one
  *  or more contributor license agreements.  See the NOTICE file
  *  distributed with this work for additional information
@@ -6,9 +6,9 @@
  *  to you under the Apache License, Version 2.0 (the
  *  "License"); you may not use this file except in compliance
  *  with the License.  You may obtain a copy of the License at
- * 
+ *
  *  http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  *  Unless required by applicable law or agreed to in writing,
  *  software distributed under the License is distributed on an
  *  "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
@@ -47,27 +47,22 @@ public class CoreRenderKitResourceLoader
   public CoreRenderKitResourceLoader(ResourceLoader parent)
   {
     register("(/.*/Common.*\\.js)",
-             new CoreCommonScriptsResourceLoader(_getCommonLibraryURI(false),
-                                                 false));
+             new CoreCommonScriptsResourceLoader(false));
     register("(/.*/DebugCommon.*\\.js)",
-             new CoreCommonScriptsResourceLoader(_getCommonLibraryURI(true),
-                                                 true));
+             new CoreCommonScriptsResourceLoader(true));
+    
+    register("(/.*/CoreFmt.*\\.js)",
+             new CoreFormatScriptsResourceLoader(false));
+    register("(/.*/DebugCoreFmt.*\\.js)",
+             new CoreFormatScriptsResourceLoader(true));
+    
     register("(/.*LocaleElements.*\\.js)",
-             new LocaleElementsResourceLoader(getLocaleElementsURI("LocaleElements", true))); 
+             new LocaleElementsResourceLoader()); 
 
     register("(/.*\\.(css|cur|jpg|gif|png|jpeg|svg|js))",
              new CoreClassLoaderResourceLoader(parent));
   }
 
-  static private String _getCommonLibraryURI(boolean debug)
-  {
-    StringBuffer base = new StringBuffer(debug
-                                         ? "/adf/jsLibs/DebugCommon"
-                                         : "/adf/jsLibs/Common");
-    return base.append(_VERSION)
-               .append(".js")
-               .toString();
-  }
     
   static public String getLocaleElementsURI(String str, 
                                             Boolean incVersion)

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/LocaleElementsResourceLoader.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/LocaleElementsResourceLoader.java?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/LocaleElementsResourceLoader.java (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/java/org/apache/myfaces/trinidadinternal/resource/LocaleElementsResourceLoader.java Thu Aug 18 18:26:52 2011
@@ -36,23 +36,26 @@ import org.apache.myfaces.trinidad.resou
  */
 public class LocaleElementsResourceLoader extends AggregatingResourceLoader
 {
-  public LocaleElementsResourceLoader(
-    String  path)
+  public LocaleElementsResourceLoader()
   {
-    super(path,
+    super("",
           _INIT_LIBRARIES,
           new ClassLoaderResourceLoader());
           
     setSeparator(_NEWLINE_SEPARATOR);
   }
   
+  /**
+   * Since CoreRenderKitResourceLoader already does the matching, this method is overridden
+   * to just call getURL()
+   */
   @Override
-  protected URL findResource(
-    String path) throws IOException
+  protected URL findResource(String path) throws IOException
   {
     return getURL(path);
   }
   
+  
   /**
    * Returns a URL which is an aggregate of all the paths.
    *

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/ColorFormat.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/ColorFormat.js?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/ColorFormat.js (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/ColorFormat.js Thu Aug 18 18:26:52 2011
@@ -98,7 +98,7 @@ TrColorConverter.prototype.getAsObject  
   if (parseString == null)
     return null;
     
-  parseString = TrUIUtils.trim(parseString);
+  parseString = TrFormatUtils.trim(parseString);
   if (parseString.length == 0)
     return null
 

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Core.js Thu Aug 18 18:26:52 2011
@@ -2695,63 +2695,6 @@ function _multiValidate(
   return failureMap;
 }
 
-/**
- * Used for the converters and validators we provide which all have the form
- *
- * {0} - label
- * {1} - string value
- * {2} - extra param
- * {3} - extra param
- */
-function _createFacesMessage(
-  key,
-  label,
-  value,
-  param2,
-  param3
-)
-{
-  var summary = TrMessageFactory.getSummaryString(key);
-  var detail = TrMessageFactory.getDetailString(key);
-  // format the detail error string
-  if (detail != null)
-  {
-    detail = TrFastMessageFormatUtils.format(detail, label, value, param2, param3);
-  }
-  return new TrFacesMessage(summary,
-                          detail,
-                          TrFacesMessage.SEVERITY_ERROR);
-}
-
-/**
- * Used for the converters and validators we provide which all have the form
- *
- * {0} - label
- * {1} - string value
- * {2} - extra param
- * {3} - extra param
- */
-function _createCustomFacesMessage(
-  summary,
-  detail,
-  label,
-  value,
-  param2,
-  param3
-)
-{
-
-  // format the detail error string
-  if (detail != null)
-  {
-    detail = TrFastMessageFormatUtils.format(detail, label, value, param2, param3);
-  }
-
-  return new TrFacesMessage(summary,
-                          detail,
-                          TrFacesMessage.SEVERITY_ERROR);
-}
-
 
 function _getGlobalErrorString(
   input,
@@ -4745,21 +4688,6 @@ function _getEventObj()
  */
 var TrUIUtils = new Object();
 
-/**
- * Remove leading and trailing whitespace
- */
-TrUIUtils.trim = function(
-data)
-{
-  if (data != null && (typeof data) == 'string')
-    return data.replace(TrUIUtils._TRIM_ALL_RE, '');
-
-  return data;
-}
-
-// regular expression to gather whitespace at beginning and end of line
-TrUIUtils._TRIM_ALL_RE = /^\s*|\s*$/g;
-
 
 /**
  * Creates a function instance that will callback the passed in function

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/CoreFormat.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/CoreFormat.js?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/CoreFormat.js (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/CoreFormat.js Thu Aug 18 18:26:52 2011
@@ -111,6 +111,7 @@ TrLongConverter.prototype.getAsObject = 
                        label,
                        null);
 }
+
 function TrShortConverter(
   message,
   maxPrecision,
@@ -942,7 +943,7 @@ function _decimalParse(
   if (numberString == null)
     return null;
     
-  numberString = TrUIUtils.trim(numberString);
+  numberString = TrFormatUtils.trim(numberString);
   if (numberString.length == 0)
     return null
     

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/DateFormat.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/DateFormat.js?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/DateFormat.js (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/DateFormat.js Thu Aug 18 18:26:52 2011
@@ -1183,7 +1183,7 @@ TrDateTimeConverter.prototype.getAsObjec
   if (parseString == null)
     return null;
 
-  parseString = TrUIUtils.trim(parseString);
+  parseString = TrFormatUtils.trim(parseString);
   if (parseString.length == 0)
     return null;
 

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Locale.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Locale.js?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Locale.js (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/Locale.js Thu Aug 18 18:26:52 2011
@@ -17,6 +17,83 @@
  *  under the License.
  */
 
+/**
+ * Used for the converters and validators we provide which all have the form
+ *
+ * {0} - label
+ * {1} - string value
+ * {2} - extra param
+ * {3} - extra param
+ */
+function _createFacesMessage(
+  key,
+  label,
+  value,
+  param2,
+  param3
+)
+{
+  var summary = TrMessageFactory.getSummaryString(key);
+  var detail = TrMessageFactory.getDetailString(key);
+  // format the detail error string
+  if (detail != null)
+  {
+    detail = TrFastMessageFormatUtils.format(detail, label, value, param2, param3);
+  }
+  return new TrFacesMessage(summary,
+                          detail,
+                          TrFacesMessage.SEVERITY_ERROR);
+}
+
+
+/**
+ * Used for the converters and validators we provide which all have the form
+ *
+ * {0} - label
+ * {1} - string value
+ * {2} - extra param
+ * {3} - extra param
+ */
+function _createCustomFacesMessage(
+  summary,
+  detail,
+  label,
+  value,
+  param2,
+  param3
+)
+{
+
+  // format the detail error string
+  if (detail != null)
+  {
+    detail = TrFastMessageFormatUtils.format(detail, label, value, param2, param3);
+  }
+
+  return new TrFacesMessage(summary,
+                          detail,
+                          TrFacesMessage.SEVERITY_ERROR);
+}
+
+
+var TrFormatUtils = new Object();
+
+/**
+ * Remove leading and trailing whitespace
+ */
+TrFormatUtils.trim = function(
+data)
+{
+  if (data != null && (typeof data) == 'string')
+    return data.replace(TrFormatUtils._TRIM_ALL_RE, '');
+
+  return data;
+}
+
+// regular expression to gather whitespace at beginning and end of line
+TrFormatUtils._TRIM_ALL_RE = /^\s*|\s*$/g;
+
+
 var _digits;
 var _decimalSep;
 var _groupingSep;

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/javascript/META-INF/adf/jsLibs/NumberConverter.js Thu Aug 18 18:26:52 2011
@@ -211,6 +211,7 @@ TrNumberConverter.prototype.getAsString 
   }
 }
 
+
 /**
  * Returns the number value for the submitted string or undefined (see also _isConvertible).
  */
@@ -227,7 +228,7 @@ TrNumberConverter.prototype.getAsObject 
     if (numberString == null)
       return null;
     
-    numberString = TrUIUtils.trim(numberString);
+    numberString = TrFormatUtils.trim(numberString);
     if (numberString.length == 0)
       return null
 
@@ -327,7 +328,6 @@ TrNumberConverter.prototype.getAsObject 
  */
 TrNumberConverter.prototype._isConvertible = function()
 {
-  // The locale attribute is now supported on convertNumber.
   return (this._pattern == null);
 }
 

Modified: myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/resources/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/javascriptmap.properties
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/resources/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/javascriptmap.properties?rev=1159348&r1=1159347&r2=1159348&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/resources/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/javascriptmap.properties (original)
+++ myfaces/trinidad/branches/1.2.12.6.0-branch/trinidad-impl/src/main/resources/org/apache/myfaces/trinidadinternal/renderkit/core/xhtml/jsLibs/javascriptmap.properties Thu Aug 18 18:26:52 2011
@@ -16,11 +16,11 @@
 #    under the License.
 
 CharSets=Common
-CoreFormat=Common
+CoreFormat=CoreFmt
 DateField=Common
 DateFieldFormat=Common
-DateFormat=Common
-Locale=Common
+#DateFormat=Common
+#Locale=Common
 #LovInput=Common
 #
 #MarlinCore split into 3 javascript files
@@ -41,7 +41,7 @@ TableProxy=Common
 
 ColorField=Common
 ColorFieldFormat=Common
-ColorFormat=Common
+#ColorFormat=Common
 DiagrammerLib=Common
 Shuttle=Common
 RichTextEditor=Common