Description: ------------ Currently DataGrid_DataSource_SQLQuery does not handle sorting on a field where the table name is passed with the field. Example 'table.field' will be quoted incorrectly as `table.field` when it should be `table`.`field` This is necessary for queries where a field name alone would be ambiguous.

IMHO the only reasonable solution (without using any magic like detecting special cases like "table.field") is to provide an option that allows to disable the call of _quoteIdentifier(). If set to false, people would have to take care of quoting themselves on the sort() call. Matt, Olivier, any opinions on this?

This bug has been fixed in CVS. 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. --- There is now a new option 'quote_identifiers'.

Mark, I think this option is dangerous, it is a breach allowing sql injection. I think it is better to transparently take the dot into account, escaping with a regex or something magic. AFAIK, the dot is standard SQL, it should be portable.

The regex idea is good, I hadn't considered such a solution. We don't need the new option anymore then. Changed code is in CVS.