#553 implemented. catch variables are no longer false flags

src/main/java/com/cflint/CFLint.java

@@ -609,6 +609,14 @@ private void process(final Element elem, final String space, final Context conte
                 scanElement(elem, context);
                 processStack(elem.getChildElements(), space + " ", context);
                 handler.pop();
+            } else if (elem.getName().equalsIgnoreCase(CF.CFCATCH)) {
+                scanElement(elem, context);
+                handler.push(CF.CFCATCH);
+                if(elem.getAttributeValue("name")!=null){
+                    handler.addVariable(elem.getAttributeValue("name"));
+                }
+                processStack(elem.getChildElements(), space + " ", context);
+                handler.pop();
             } else {
                 scanElement(elem, context);
                 for (final Entry<String, CFExpression> expression : unpackTagExpressions(elem).entrySet()) {
@@ -875,7 +883,12 @@ private void process(final CFScriptStatement expression, final Context context)
                 final CFTryCatchStatement cftry = (CFTryCatchStatement) expression;
                 process(cftry.getBody(), context);
                 for (final CFCatchStatement stmt : cftry.getCatchStatements()) {
+                    handler.push(CF.CFCATCH);
+                    if(stmt.getVariable()!=null){
+                        handler.addVariable(stmt.getVariable().getName());
+                    }
                     process(stmt.getCatchBody(), context);
+                    handler.pop();
                 }
                 process(cftry.getFinallyStatement(), context);
             } else if (expression instanceof CFReturnStatement) {

src/test/resources/com/cflint/tests/VarScoper/553_catch.cfc

@@ -0,0 +1,14 @@
+component {
+    public string function renderPage( required any page, required any docTree, required boolean edit ){
+        try {
+            var renderedPage = renderTemplate(
+                  template = "templates/#_getPageLayoutFile( arguments.page )#.cfm"
+                , args     = { page = arguments.page, docTree=arguments.docTree, edit=edit }
+                , helpers  = "/builders/html/helpers"
+            );
+        } catch( any e ) {
+            e.additional.luceeDocsPageId = arguments.page.getid();
+            rethrow;
+        }
+    }
+}

src/test/resources/com/cflint/tests/VarScoper/553_catch.expected.txt

@@ -0,0 +1,11 @@
+{
+  "version" : "",
+  "timestamp" : 1513437208,
+  "issues" : [ ],
+  "counts" : {
+    "totalFiles" : 0,
+    "totalLines" : 0,
+    "countByCode" : [ ],
+    "countBySeverity" : [ ]
+  }
+}

src/test/resources/com/cflint/tests/VarScoper/553_cfml_catch.cfc

@@ -0,0 +1,9 @@
+<cfcomponent>
+  <cffunction name="foo">
+    <cftry>
+    <cfcatch name="e">
+        <cfset e.additional.luceeDocsPageId = 1/>
+    </cfcatch>
+    </cftry>
+  </cffunction>
+</cfcomponent>

src/test/resources/com/cflint/tests/VarScoper/553_cfml_catch.expected.txt

@@ -0,0 +1,11 @@
+{
+  "version" : "",
+  "timestamp" : 1525483036,
+  "issues" : [ ],
+  "counts" : {
+    "totalFiles" : 0,
+    "totalLines" : 0,
+    "countByCode" : [ ],
+    "countBySeverity" : [ ]
+  }
+}