fixes #84

Author: wumpz

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

@@ -40,7 +40,7 @@
 /**
  * Find all used tables within an select statement.
  */
-public class TablesNamesFinder implements SelectVisitor, FromItemVisitor, ExpressionVisitor, ItemsListVisitor {
+public class TablesNamesFinder implements SelectVisitor, FromItemVisitor, ExpressionVisitor, ItemsListVisitor, SelectItemVisitor {
 
     private List<String> tables;
     /**
@@ -163,6 +163,12 @@ public void visit(WithItem withItem) {
 
     @Override
     public void visit(PlainSelect plainSelect) {
+        if (plainSelect.getSelectItems() != null) {
+            for (SelectItem item : plainSelect.getSelectItems()) {
+                item.accept(this);
+            }
+        }
+
         plainSelect.getFromItem().accept(this);
 
         if (plainSelect.getJoins() != null) {
@@ -173,7 +179,9 @@ public void visit(PlainSelect plainSelect) {
         if (plainSelect.getWhere() != null) {
             plainSelect.getWhere().accept(this);
         }
-
+        if (plainSelect.getOracleHierarchical() != null) {
+            plainSelect.getOracleHierarchical().accept(this);
+        }
     }
 
     @Override
@@ -455,6 +463,13 @@ public void visit(JdbcNamedParameter jdbcNamedParameter) {
 
     @Override
     public void visit(OracleHierarchicalExpression oexpr) {
+        if (oexpr.getStartExpression() != null) {
+            oexpr.getStartExpression().accept(this);
+        }
+
+        if (oexpr.getConnectExpression() != null) {
+            oexpr.getConnectExpression().accept(this);
+        }
     }
 
     @Override
@@ -463,13 +478,24 @@ public void visit(RegExpMatchOperator rexpr) {
     }
 
     @Override
-	public void visit(RegExpMySQLOperator rexpr) {
-    	visitBinaryExpression(rexpr);
-	}
-    
+    public void visit(RegExpMySQLOperator rexpr) {
+        visitBinaryExpression(rexpr);
+    }
+
     @Override
     public void visit(JsonExpression jsonExpr) {
     }
 
-	
+    @Override
+    public void visit(AllColumns allColumns) {
+    }
+
+    @Override
+    public void visit(AllTableColumns allTableColumns) {
+    }
+
+    @Override
+    public void visit(SelectExpressionItem item) {
+        item.getExpression().accept(this);
+    }
 }

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

@@ -264,6 +264,20 @@ public void testGetTableListFromUpdate3() throws Exception {
 		assertTrue(tableList.contains("MY_TABLE2"));
 		assertTrue(tableList.contains("MY_TABLE3"));
 	}
+    
+    @Test
+	public void testCmplxSelectProblem() throws Exception {
+		String sql = "SELECT cid, (SELECT name FROM tbl0 WHERE tbl0.id = cid) AS name, original_id AS bc_id FROM tbl WHERE crid = ? AND user_id is null START WITH ID = (SELECT original_id FROM tbl2 WHERE USER_ID = ?) CONNECT BY prior parent_id = id AND rownum = 1";
+		net.sf.jsqlparser.statement.Statement statement = pm.parse(new StringReader(sql));
+
+		Select selectStatement = (Select) statement;
+		TablesNamesFinder tablesNamesFinder = new TablesNamesFinder();
+		List<String> tableList = tablesNamesFinder.getTableList(selectStatement);
+		assertEquals(3, tableList.size());
+		assertTrue(tableList.contains("tbl0"));
+        assertTrue(tableList.contains("tbl"));
+        assertTrue(tableList.contains("tbl2"));
+	}
 
 	private String getLine(BufferedReader in) throws Exception {
 		return CCJSqlParserManagerTest.getLine(in);