fixes #284

Author: wumpz

src/main/java/net/sf/jsqlparser/util/TablesNamesFinder.java

@@ -56,7 +56,7 @@
 public class TablesNamesFinder implements SelectVisitor, FromItemVisitor, ExpressionVisitor, ItemsListVisitor, SelectItemVisitor, StatementVisitor {
 
     private static final String NOT_SUPPORTED_YET = "Not supported yet.";
-	private List<String> tables;
+    private List<String> tables;
     /**
      * There are special names, that are not table names but are parsed as
      * tables. These names are collected here and are not included in the tables
@@ -228,6 +228,10 @@ public void visit(EqualsTo equalsTo) {
 
     @Override
     public void visit(Function function) {
+        ExpressionList exprList = function.getParameters();
+        if (exprList != null) {
+            visit(exprList);
+        }
     }
 
     @Override
@@ -326,7 +330,6 @@ public void visit(ExpressionList expressionList) {
         for (Expression expression : expressionList.getExpressions()) {
             expression.accept(this);
         }
-
     }
 
     @Override
@@ -576,17 +579,17 @@ public void visit(Replace replace) {
 
     @Override
     public void visit(Drop drop) {
-        throw new UnsupportedOperationException(NOT_SUPPORTED_YET); 
+        throw new UnsupportedOperationException(NOT_SUPPORTED_YET);
     }
 
     @Override
     public void visit(Truncate truncate) {
-        throw new UnsupportedOperationException(NOT_SUPPORTED_YET); 
+        throw new UnsupportedOperationException(NOT_SUPPORTED_YET);
     }
 
     @Override
     public void visit(CreateIndex createIndex) {
-        throw new UnsupportedOperationException(NOT_SUPPORTED_YET); 
+        throw new UnsupportedOperationException(NOT_SUPPORTED_YET);
     }
 
     @Override
@@ -599,12 +602,12 @@ public void visit(CreateTable create) {
 
     @Override
     public void visit(CreateView createView) {
-        throw new UnsupportedOperationException(NOT_SUPPORTED_YET); 
+        throw new UnsupportedOperationException(NOT_SUPPORTED_YET);
     }
 
     @Override
     public void visit(Alter alter) {
-        throw new UnsupportedOperationException(NOT_SUPPORTED_YET); 
+        throw new UnsupportedOperationException(NOT_SUPPORTED_YET);
     }
 
     @Override
@@ -619,7 +622,7 @@ public void visit(Execute execute) {
 
     @Override
     public void visit(SetStatement set) {
-        throw new UnsupportedOperationException(NOT_SUPPORTED_YET); 
+        throw new UnsupportedOperationException(NOT_SUPPORTED_YET);
     }
 
     @Override
@@ -649,7 +652,7 @@ public void visit(TableFunction valuesList) {
 
     @Override
     public void visit(AlterView alterView) {
-        throw new UnsupportedOperationException(NOT_SUPPORTED_YET); 
+        throw new UnsupportedOperationException(NOT_SUPPORTED_YET);
     }
 
     @Override

src/test/java/net/sf/jsqlparser/util/TablesNamesFinderTest.java

@@ -363,5 +363,15 @@ public void testGetTableListIssue194() throws Exception {
 		List<String> tableList = tablesNamesFinder.getTableList(selectStatement);
 		assertEquals(0, tableList.size());
 	}
-        
+    
+    @Test
+	public void testGetTableListIssue284() throws Exception {
+		String sql = "SELECT NVL( (SELECT 1 FROM DUAL), 1) AS A FROM TEST1";
+		Select selectStatement = (Select) CCJSqlParserUtil.parse(sql);
+		TablesNamesFinder tablesNamesFinder = new TablesNamesFinder();
+		List<String> tableList = tablesNamesFinder.getTableList(selectStatement);
+		assertEquals(2, tableList.size());
+        assertTrue(tableList.contains("DUAL"));
+        assertTrue(tableList.contains("TEST1"));
+	}
 }