[engine] Invalid method calls should definitely be logged

git-svn-id: https://svn.apache.org/repos/asf/velocity/engine/trunk@1854695 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 14 insertions, 0 deletions

diff --git a/velocity-engine-core/src/main/java/org/apache/velocity/runtime/parser/node/ASTMethod.java b/velocity-engine-core/src/main/java/org/apache/velocity/runtime/parser/node/ASTMethod.java
index 403cd28a..c046b618 100644
--- a/velocity-engine-core/src/main/java/org/apache/velocity/runtime/parser/node/ASTMethod.java
+++ b/velocity-engine-core/src/main/java/org/apache/velocity/runtime/parser/node/ASTMethod.java
@@ -165,6 +165,20 @@ public class ASTMethod extends SimpleNode
         VelMethod method = ClassUtils.getMethod(methodName, params, paramClasses,
             o, context, this, strictRef);
 
+        // warn if method wasn't found (if strictRef is true, then ClassUtils did thorw an exception)
+        if (o != null && method == null)
+        {
+            StringBuilder plist = new StringBuilder();
+            for (int i = 0; i < params.length; i++)
+            {
+                Class param = paramClasses[i];
+                plist.append(param == null ? "null" : param.getName());
+                if (i < params.length - 1)
+                    plist.append(", ");
+            }
+            log.warn("Object '{}' does not contain method {}({}) at {}[line {}, column {}]", o.getClass().getName(), methodName, plist, getTemplateName(), getLine(), getColumn());
+        }
+
         /*
          * The parent class (typically ASTReference) uses the icache entry
          * under 'this' key to distinguish a valid null result from a non-existent method.