From 280fe8eb13a658f80d31f135a39fa207d9ea8b34 Mon Sep 17 00:00:00 2001
From: "Brian S. O'Neill" <bronee@gmail.com>
Date: Sat, 6 Jan 2024 09:15:44 -0800
Subject: [PATCH] fix: ExpressionVisitor.visit(AllTableColumns) method isn't
 being called.

This fixes a regression introduced in change #4b4ae04f44ff18b669d0f30d637e5b7c64b085e4: feat: BigQuery Except(..) Replace(..) syntax
---
 .../statement/select/AllTableColumns.java      |  6 ++++++
 .../ExpressionVisitorAdapterTest.java          | 18 ++++++++++++++++++
 2 files changed, 24 insertions(+)

diff --git a/src/main/java/net/sf/jsqlparser/statement/select/AllTableColumns.java b/src/main/java/net/sf/jsqlparser/statement/select/AllTableColumns.java
index 429ebcd35..cda2e8136 100644
--- a/src/main/java/net/sf/jsqlparser/statement/select/AllTableColumns.java
+++ b/src/main/java/net/sf/jsqlparser/statement/select/AllTableColumns.java
@@ -9,6 +9,7 @@
  */
 package net.sf.jsqlparser.statement.select;
 
+import net.sf.jsqlparser.expression.ExpressionVisitor;
 import net.sf.jsqlparser.expression.operators.relational.ExpressionList;
 import net.sf.jsqlparser.schema.Column;
 import net.sf.jsqlparser.schema.Table;
@@ -50,4 +51,9 @@ public AllTableColumns withTable(Table table) {
     public StringBuilder appendTo(StringBuilder builder) {
         return super.appendTo(table.appendTo(builder).append("."));
     }
+
+    @Override
+    public void accept(ExpressionVisitor expressionVisitor) {
+        expressionVisitor.visit(this);
+    }
 }
diff --git a/src/test/java/net/sf/jsqlparser/expression/ExpressionVisitorAdapterTest.java b/src/test/java/net/sf/jsqlparser/expression/ExpressionVisitorAdapterTest.java
index fa301142a..52e71b8bf 100644
--- a/src/test/java/net/sf/jsqlparser/expression/ExpressionVisitorAdapterTest.java
+++ b/src/test/java/net/sf/jsqlparser/expression/ExpressionVisitorAdapterTest.java
@@ -15,6 +15,7 @@
 import net.sf.jsqlparser.expression.operators.relational.InExpression;
 import net.sf.jsqlparser.parser.CCJSqlParserUtil;
 import net.sf.jsqlparser.schema.Column;
+import net.sf.jsqlparser.statement.select.AllTableColumns;
 import net.sf.jsqlparser.statement.select.PlainSelect;
 import net.sf.jsqlparser.statement.select.Select;
 import net.sf.jsqlparser.statement.select.SelectVisitorAdapter;
@@ -241,4 +242,21 @@ public void testRowConstructor() throws JSQLParserException {
                 "CAST(ROW(dataid, value, calcMark) AS ROW(datapointid CHAR, value CHAR, calcMark CHAR))")
                 .accept(adapter);
     }
+
+    @Test
+    public void testAllTableColumns() throws JSQLParserException {
+        PlainSelect plainSelect = (PlainSelect) CCJSqlParserUtil.parse("select a.* from foo a");
+        final AllTableColumns[] holder = new AllTableColumns[1];
+        Expression from = plainSelect.getSelectItems().get(0).getExpression();
+        from.accept(new ExpressionVisitorAdapter() {
+
+            @Override
+            public void visit(AllTableColumns all) {
+                holder[0] = all;
+            }
+        });
+
+        assertNotNull(holder[0]);
+        assertEquals("a.*", holder[0].toString());
+    }
 }