Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.11.5

Bug #14501 typo in generator ingnore quote identifies option
Submitted: 2008-08-14 01:11 UTC
From: fieldtim Assigned:
Status: Bogus Package: DB_DataObject (version 1.8.8)
PHP Version: 5.2.1 OS: Linux
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 49 - 36 = ?

 
 [2008-08-14 01:11 UTC] fieldtim (Tim Field)
Description: ------------ --- lib/pear/php/DB/DataObject/Generator.php (revision 34279) +++ lib/pear/php/DB/DataObject/Generator.php (working copy) @@ -256,7 +256,7 @@ $table = $bits[1]; } } - $quotedTable = !empty($options['quote_identifiers_tableinfo']) ? + $quotedTable = !empty($options['quote_identifiers']) ? $__DB->quoteIdentifier($table) : $table; Test script: --------------- create a table called group and run the createTables script Expected result: ---------------- correct ini file Actual result: -------------- error

Comments

 [2008-08-14 03:47 UTC] alan_k (Alan Knowles)
We have to have a seperate configuration option for tableInfo() quoting as some databases do not work if you quote the table name and send it to tableInfo. There was another place where it was called and was not using quote_identifiers_tableinfo - that has now been fixed in CVS.