Description: ------------ It's really easy to correct, that's just that the compiler call an implode function which doesn't work "Warning: implode() [function.implode]: Invalid arguments passed in /usr/share/php/SQL/Compiler.php on line 271" and put empty parenthesis when it receive an insert without column_names like "INSERT INTO table1 VALUES (1,"test 1");" It should check if the column_names array exist in the tree/array. Test script: --------------- <?php require_once 'SQL/Parser.php'; require_once 'SQL/Compiler.php'; $parser = new SQL_Parser(); $compiler = new SQL_Compiler(); $req = $parser->parse("INSERT INTO table1 VALUES (1,'test 1')"); echo $compiler->compile($req); ?> Expected result: ---------------- insert into table1 values (1, 'test 1') Actual result: -------------- Warning: implode() [function.implode]: Invalid arguments passed in /usr/share/php/SQL/Compiler.php on line 271 insert into table1 () values (1, 'test 1')

The following patch has been added/updated: Patch Name: CompilerInsertPatch Revision: 1240405006 URL: http://pear.php.net/bugs/patch-display.php?bug=16139&patch=CompilerInsertPatch&revision=1240405006&display=1

The following patch has been added/updated: Patch Name: CorrectPatch Revision: 1242035321 URL: http://pear.php.net/bugs/patch-display.php?bug=16139&patch=CorrectPatch&revision=1242035321&display=1

This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.