Use undef instead of '%' fore the table and type arguments to $dbh->tables
authorDagfinn Ilmari Mannsåker <ilmari@ilmari.org>
Tue, 19 Aug 2014 17:31:03 +0000 (18:31 +0100)
committerDagfinn Ilmari Mannsåker <ilmari@ilmari.org>
Tue, 19 Aug 2014 17:32:50 +0000 (18:32 +0100)
commit36c8c37c10fbb9a3f6578a8df14cd6cde1bb774f
tree8c0ea11e54965718a4418377e02d902a881e8834
parent7c304e590cd0eac21aed4c62266e43129d6ff9ce
Use undef instead of '%' fore the table and type arguments to $dbh->tables

Neither the DBI documentation nor the ODBC specification allow '%' for
the type, but some DBDs do (or did, DBD::Pg 3.4.0 no longer does)
allow it. For the ones that didn't, the loader subclass used override
->_tables_list to pass undef explicitly.

Instead, use undef in the default implementation, but allow subclasses
to override as needed (none currently do).

Also change the table argument to default to undef, since that's
equivalent according to the spec and all the DBDs I've tested with or
looked at (SQLite, MySQL, Pg, Oracle, FireBird, ODBC, InterBase).
Changes
lib/DBIx/Class/Schema/Loader/DBI.pm
lib/DBIx/Class/Schema/Loader/DBI/ADO.pm
lib/DBIx/Class/Schema/Loader/DBI/ODBC.pm
lib/DBIx/Class/Schema/Loader/DBI/mysql.pm