Changeset 1346 for trunk/universe/kauri-template/src/main/java/org/kauriproject/template/el/TemplateFunctionMapper.java

Timestamp:

2009-07-02 15:06:52 (3 years ago)

Author:

bruno

Message:

Introduce a TemplateException? class, and use that in the template code instead of a generic RuntimeException?.

File:

• trunk/universe/kauri-template/src/main/java/org/kauriproject/template/el/TemplateFunctionMapper.java (modified) (3 diffs)

trunk/universe/kauri-template/src/main/java/org/kauriproject/template/el/TemplateFunctionMapper.java

@@ -16 +16 @@
 package org.kauriproject.template.el;
 
-import org.apache.commons.logging.Log;
-import org.apache.commons.logging.LogFactory;
+import org.kauriproject.template.TemplateException;
 
 import javax.el.FunctionMapper;
@@ -26 +25 @@
 public class TemplateFunctionMapper extends FunctionMapper {
     private Map<String, Method> functions = new HashMap<String, Method>();
-    private Log log = LogFactory.getLog(getClass());
 
     public TemplateFunctionMapper(FunctionRegistry functionRegistry) {
@@ -52 +50 @@
             addFunction(prefix, "max", Math.class.getMethod("max", double.class, double.class));
             addFunction(prefix, "min", Math.class.getMethod("min", double.class, double.class));
-            // add additional math functions
-            Method[] methods = MathFunctions.class.getDeclaredMethods();
-            for (Method method : methods) {
-                addFunction(prefix, method.getName(), method);
-            }
         } catch (NoSuchMethodException nex) {
-            log.error("Error adding math functions to the EL: " + nex);
+            throw new TemplateException("Error registering math functions with EL", nex);
+        }
+
+        // add additional math functions
+        Method[] methods = MathFunctions.class.getDeclaredMethods();
+        for (Method method : methods) {
+            addFunction(prefix, method.getName(), method);
         }
     }