Missing commas in reference builder

[GoogleCodeExporter]

Parse and recreate the following query:
SELECT * FROM TableA JOIN TableB USING(Col1, Col2)

The creator returns
SELECT * FROM TableA INNER JOIN TableB USING (Col1 Col2)
, which is not valid SQL.

I fixed the problem by changing RefClauseBuilder.php.
line 87 is now: $sql .= ", ";
line 89 is now: return "(" . substr($sql, 0, -2) . ")";

Original issue reported on code.google.com by kitts...@gmail.com on 2 Jul 2014 at 10:28

[GoogleCodeExporter]

Upon further exploration, this is not a robust fix, as it can break some other 
type of reference clauses (specifically ON clauses).

I am using the version added 2014-01-08.

Original comment by kitts...@gmail.com on 2 Jul 2014 at 11:24

[GoogleCodeExporter]

It is better to enhance the parser first. After USING follows a column list and 
not only a bracket_expression. This will give the builder a chance to create 
different things.

Original comment by pho...@gmx.de on 11 Sep 2014 at 8:00

• Changed state: Accepted

[GoogleCodeExporter]

Please try r1380, it contains some changes to fix the issue.

Original comment by pho...@gmx.de on 11 Sep 2014 at 9:30

• Changed state: Started

[GoogleCodeExporter]

Original comment by pho...@gmx.de on 15 Sep 2014 at 7:01

• Changed state: Fixed