#416 fixed.

cflint · Aug 12, 2017 · 8dfe18a · 8dfe18a
1 parent e5b7926
commit 8dfe18a
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/main/java/com/cflint/CFLint.java b/src/main/java/com/cflint/CFLint.java
@@ -525,6 +525,11 @@ private void process(final Element elem, final String space, final Context conte
                 }
             } else if (elem.getName().equalsIgnoreCase("cfqueryparam")) {
                 scanElement(elem, context);
+                for(final Entry<String, CFExpression> expression : unpackTagExpressions(elem).entrySet()){
+                    if (expression != null) {
+                        process(expression.getValue(), elem, context);
+                    }
+                }
             } else if (elem.getName().equalsIgnoreCase("cfinclude")) {
                 scanElement(elem, context);
                 final String path = elem.getAttributeValue("template");

diff --git a/src/test/java/com/cflint/TestUnusedLocalVarChecker.java b/src/test/java/com/cflint/TestUnusedLocalVarChecker.java
@@ -1,6 +1,7 @@
 package com.cflint;
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertTrue;
 
 import java.io.IOException;
 import java.util.List;
@@ -253,7 +254,6 @@ public void testMultipleUnusedVarFunctionInTag() throws ParseException, IOExcept
 
 
     @Test
-    @Ignore
   	public void testUsedVarInQuaryParam() throws ParseException, IOException {
   		final String tagSrc = "<cfcomponent>\r\n"
   			+ "<cffunction name=\"changePassword\">\r\n"
@@ -269,9 +269,9 @@ public void testUsedVarInQuaryParam() throws ParseException, IOException {
   			+ "</cffunction>\r\n"
   			+ "</cfcomponent>";
 
-        cfBugs.process(tagSrc, "test");
+  		cfBugs.process(tagSrc, "test");
       		final List<BugInfo> result = cfBugs.getBugs().getBugList().get("UNUSED_LOCAL_VARIABLE");
-  		assertEquals(0, result.size());
+  		assertTrue(result == null);
   	}
 
 }