Change all for@ loops to safer foreach loops (Base.pm)

Change all for@ loops to safer foreach loops that don't clobber $_
(and thus @tables) because non-DCSL::DBI::* DBDs fail on
_setup_src_meta.

release 0.07015

many_to_many rel bridge generation for link tables

For tables that have 2 FKs whose only columns are members of the FKs,
and having all columns in the PK, many_to_many relationship bridges are
now generated for the referred to classes.

release 0.07014

make copy of moniker_parts in disambiguation code

The local $self->{moniker_parts} code did not make a copy of
$self->moniker_parts before unshifting to it, so modifications were made
to the original reference, breaking ->rescan.

release 0.07013

add notes about table object stringify to name

Add notes to the POD for callbacks that the table object parameter
stringifies to the unqualified table name, also add a note to the
::Table POD.

auto-disambiguate monikers from different schemas

Automatically prefix schema and/or database to clashing monikers for the
same table name from different schemas/databases.

Update docs to reflect this behavior.

Also rewrite all the multischema tests to use this new feature instead
of explicit moniker_parts.

release 0.07012

update docs for callbacks receiving table objects

Update docs for callbacks like custom_column_info to say that they now
receive a table object (e.g. ::Loader::Table) rather than the table
name (RT##72260).

update SEE ALSO section of Loader.pm

Add links to Intro, Tutorial and Loader::Base.

better cleanup of t/var

Add an END action in t/lib/dbixcsl_test_dir.pm to delete t/var, first
closing any open DBI handles because if there is an open SQLite handle
to a .db file in that dir, the .db file cannot be removed on Win32. Also
copy the file to t/backcompat/0.04006/lib/dbixcsl_test_dir.pm.

Also add t/var to MANIFEST.SKIP just in case.

fix code running 2x in dynamic schema_base_class

Fix a bug that ran the connection method in
schema_base_class/schema_components twice on the second connect, with
tests.

Move the RelBuilder _array_eq method to an array_eq sub in Utils.

use temp file instead of IPC::Open2 for Win32

Change implementation of filter_generated_code with a string for program
name from IPC::Open2 to File::Temp with redirection, because this code
was hanging on Win32 (RT#72226).

add correction to Changes for wrong RT#

uniq_to_primary option fixes RT#51696, not RT#25944.

release 0.07011

fix some warnings for MSSQL over ADO

Fix invalid object warnings for nonexistant table/comment tables, and
"Changed context" warnings for the extra database tests.

add -I option to dbicdump

Add an -I option to dbicdump to add things to @INC during the dump, like
perl's -I option (see perldoc perlrun.)

For the config file dbicdump format, add the 'lib' key that is the
equivalent of the -I option.

Add error checking to ::Utils slurp_file and write_file subs in the open
calls.

Add alnewkirk to list of CONTRIBUTORS for dbicdump.

remove duplicate from Changes

stop editing custom content from renamed files

Remove an unnecessary bit of cleverness that was removing the default
custom content comment (# You can replace...) from files that are being
renamed for an upgrade (either to singularization or use_namespaces.)

This fixes RT#70507.

add attribution for alnewkirk, mention force_ascii

Add attribution for the dbicdump config file support for alnewkirk in
Changes and add his handle in ::Loader CONTRIBUTORS section.

Add naming => { force_ascii => 1 } to Changes as well.

update Changes with MooseX::MarkAsMethods

update Changes for naming=v8

common tests for schema_base_class and components

Test calling schema_components methods and test schema_base_class in the
common tests.

better CamelCASE support for naming=v8

The previous method of splitting CamelCase identifiers did not work
identifiers containing words in all caps, such as 'VLANValidID'.

Use 'wordsplit' from String::CamelCase which handles these types of
identifiers correctly for naming=v8 mode, while preserving the old
behavior for v7 mode for monikers, column accessors and relationship
names.

New features related to multischema support also use the v8 mode
implicitly.

Add t/80split_name.t to test the 'split_name' utility sub from ::Utils,
at the heart of this change; for both v8 and v7 modes.

update use_moose docs and opt deps

Add namespace::autoclean as an optional dep for use_moose. Even though
it is a dependency of MooseX::MarkAsMethods, we want to list it
explicitly.

Update use_moose and only_autoclean POD to say that the overloads being
protected are operator overloads.

namespace::autoclean to MooseX::MarkAsMethods

Use MooseX::MarkAsMethods with autoclean => 1 instead of
namespace::autoclean for the use_moose option, to protect operator
overloads.

Add an only_autoclean option to use namespace::autoclean as before.

fix some naming version checks

Previous code didn't take into account that naming->{key} can be set to
something other than vX.

Fix a call to ->_to_identifier in the monikerization code.

t/23dumpmore.t passes again.

fix double backcompat warning

Set naming and use_namespaces in the temp loader used in ::Loader to get
the schema_base_class and schema_components so that no backcompat check
is done and warning isn't issued.

fix uninitialized warning

Change the temporary loader used to extract the schema_base_class and
schema_components to be created with the schema => $self arg, so the
constructor (and _check_back_compat) can read the Schema.pm.

update Changes with schema_base_class fix

better dynamic schema_base_class implementation

Remove the restriction that dynamic schemas can't define a connection
method by finding the next method to call in mro::get_linear_isa.

use connection hack only for schema_base_class

Use the horrible mro hack in ::Loader::connection only if
schema_base_class and/or schema_components were defined.

Add warnings to the POD for these options that dynamic schemas cannot
define a connection method.

bump String::ToIdentifier::EN dep to 0.05

0.04 has some horrible bugs, this is the first version that works
reasonably well.

implement schema_base_class for dynamic schemas

Implement schema_base_class and schema_components support for dynamic
schemas at 'connection' time.

Augment btilly's test to check that schema_components are also applied
at 'connection' time.

This requires a rather horrible hack in ::Loader::connection, because
mro cannot normally deal with a changing @ISA inside of a call chain,
but it works.

test for connecting through schema_base_class

Add test for applying schema_base_class to dynamic schemas at
'connection' time.

update DBIx::Class::Schema::Loader POD

Update list of supported DBDs and minimum necessary DBIx::Class version.
Remove the admonition that this module is for simple situations only.
Add link to ::Loader::Base for options.

add 'ALL' key for naming hash

Because I earlier added the 'force_ascii' parameter for the naming hash,
I now add the 'ALL' key to set 'relationships', 'monikers' and
'column_accessors' in one shot. This lets you do:

    naming => { ALL => 'v8', force_ascii => 1 }

fully implement naming=v8 and force_ascii

Use String::ToIdentifier::EN or ::Unicode on column accessors and
relationship accessors as well as monikers, for the experimental v8
naming mode.

update comment table support for multi-db_schema

Update the POD related to the table_comments and column_comments tables
to note that these tables must be in the same database and schema as the
tables they are describing.

Add a clone method to ::DBObject and use it to find the comment tables,
changing just the table name to table_comments_table or
column_comments_table.

better rel postfix stripping for v8 mode

Change the _id stripping in ::RelBuilder to strip other things as well,
like Id, ref, cd, code, _num.

This is only activated for the experimental { naming => 'v8' } mode.

fix no table warnings for MS Access over DBD::ADO

The _table_comment and _column_comment methods in ::DBI do queries on
the 'table_comments' and 'column_comments' tables for getting
table/column comments in databases which do not support them natively
(or well.) These tables usually do not exist.

DBD::ADO throws a warning from Win32::OLE when it encounters a query on
a table that does not exist. Here we trap and ignore those warnings for
the comments tables.

fix some issues with multi-db_schema support

Fix unique constraint detection for Oracle in multischema setups
(RT#70851) previously identically named tables in different schemas with
an identically named unique constraint would cause the columns in the
constraint definition to be repeated. Modifies the multischema extra
tests for all databases to test for the correctness of the dumped unique
constraint.

Update Oracle multischema extra tests to include the complete set of
test cases used in multischema test sets for other databases.

Update Oracle is_auto_increment detection code to work in multischema
setups. The new logic is by necessity fuzzier, please report false
positives to RT. At some point this will get rewritten to use a PL/SQL
parser on the trigger code.

Add a test for the miscaching of table keys when there are identically
named tables in different schemas for all databases. This was fixed for
MySQL in master.

Completely rewrite the table monikerization code to work better with
moniker_parts, preserving the functionality of all naming versions.
Schema/database names are transformed to parts of the moniker name using
String::ToIdentifier::EN::Unicode (or String::ToIdentifier::EN if

  naming => { force_ascii => 1 }

is set.) to translate non-identifier characters.

Add the next naming version, v8 as an experimental mode. In this mode,
the table names go through String::ToIdentifier::EN::Unicode as
well, instead of non-identifier characters being stripped out.

The clashing monikers error message is updated to mention moniker_parts
in multischema configurations.

The ::RelBuilder duplicate relationship name disambiguator has been
updated to handle the case of relationships in the same class pointing
to identically named tables in different schemas/databases.

Improve the multischema extra tests to include an identically named
table in both schemas and to use moniker_parts. This also reduces the
chances of moniker clashes on a tester's database.

Change the multidatabase tests to test both

    db_schema => { db1 => '%', db2 => '%' }

and

    db_schema => { '%' => '%' }

as well as to include an identically named table.

fix uninitialized warnings for Firebird

Check if the character_set_id for a column is NULL before comparing it
to a value to fix uninitialized warnings in the tests.

ignore type_info errors from DBD::Sybase+MSSQL

When testing multi-database schemas on MSSQL, a weird error comes up
from type_info_all in DBD::Sybase. This needs to be investigated
further, but for now the call to type_info in ::DBI is wrapped in a try.

remove placeholders from tab/col comments queries

Changes the use placeholders in ::DBI _table_comment and _column_comment
methods to inline $dbh->quote protected values.

This is for DBD::Sybase with ASE versions that do not support
placeholders and MSSQL connections with a low tds version.

At some point we'll have methods for doing queries that do the right
thing depending on what the driver/database supports.

update SQLAnywhere ODBC driver for default \"null"

Previously a fix was committed to support DEFAULT NULL in Pg, this
broke the SQLAnywhere ODBC driver tests, because ::DBI::SQLAnywhere had
a fixup for default_value => \"NULL", and it is now lowercased.

Only the ODBC driver sets nullable columns DEFAULT NULL, so this fixup
has been moved to ::DBI::ODBC::SQL_Anywhere, with the correct case.

All SQL Anywhere tests pass again.

add "use utf8;" at the top of all generated files

In preparation for supporting Unicode identifiers in the future, this
commit adds "use utf8;" to the top of all generated files, before the
"package ...;" statement.

add POD for using non-English databases

Adds POD describing the loader options to use to turn off inflection for
English.

use full table name for MySQL key cache

Primary/unique key cache now caches by $table->sql_name instead of
$table.

Failure to do so returned the wrong keys for the same table name in
multi-db_schema dumps.

match case of FK remote database in MySQL

Matches the remote database of a cross-database foreign key in MySQL to
the case of the database in SHOW DATABASES, if it's there and the user
has permissions to run SHOW DATABASES.

make MySQL FK handling more robust

Now handles FKs to schemas not listed in db_schema, and to tables that
don't exist at all.

minor fix for 5.8.1

sub {} returns the args it is passed on 5.8.1, so have to use
sub { return () } .

remove dependency on File::Slurp

The current version of File::Slurp has a "use 5.6.2;" statement. This
throws a warning on perl 5.10.0 which breaks some of our tests. The
statement should be "use 5.006002;".

My attempts to contact the author, Uri Guttman were not successful.

As we have our own convenience method 'slurp_file' in ::Utils now, there
is little need to depend on File::Slurp anymore anyway.

For these reasons the dependency has been removed.

allow constraint/exclude options together

fix t/backcompat/0.04006 tests

The Oracle driver no longer lowercases table names from the catalog, and
these names are normalized to upper case. The main common tests alias
the lowercase table names for the monikers and classes hashes for this
case; this commit adds the same code to the t/backcompat/0.04006 common
tests (in lib/dbixcsl_common_tests.pm .)

fix preserve_case tests in common tests

Rescans with preserve_case option reset to the original value after
doing a rescan with preserve_case set to 1 and running the preserve_case
tests.

Failure to do so was breaking the Oracle extra tests.

Fix the failing pg tests (unicode wasn't slurped)

fix backcompat

Fixes generating the old moniker when upgrading from a previous code
version (v4 unsingularized monikers -> singularized.)

t/25backcompat.t passes again.

fix uninit warn for no pod and result_base_class

fix multiple DEFAULT NULLs

    $info->{default_value} = \'null'

is unsafe because Data::Dump optimizes
refs to constants, so we must make sure to use:

    my $null = 'null';
    $info->{default_value} = \$null;

for all scalar refs in the dump.

fix defaults broken by last Pg defaults fix commit

update Changes with Pg defaults fixes

improve Pg default handling

Correctly handles these cases:
 - <type> DEFAULT NULL
 - boolean DEFAULT 0::boolean

sort unique constraints

Relationships and unique constraints are now sorted by name in the
Result files to cut down on pointless diff spam in version control.

This commit also adds checking for the SCHEMA_LOADER_TESTS_NOCLEANUP
environment variable when removing the test sqlite DBs used by various
tests, this allows for inspecting them with the sqlite3 shell.

multi db_schema support

Allows putting multiple schemas or databases and schemas into the
db_schema option. Supports cross-schema foreign keys where available. On
Sybase ASE, MSSQL and Informix multiple databases as well as schemas are
supported (but only Sybase ASE allows cross-database foreign keys.)

Adds the moniker_parts option for resolving name clashes when using
multiple schemas that have tables with the same name.

Table names are now represented by the ::Table class (all the code for
which is in the ::DBObject superclass.) There is also ::Table::Sybase
for Sybase ASE and MSSQL as well as ::Table::Informix for Informix. In
the future other database objects such as sequences may use the
::DBObject class.

Currently there are extra tests for objects in separate schemas and
cross-schema relationships for each database separately, these may later
be merged into the common tests in some fashion.

All database backends are up to date with respect to their capabilities
for fully qualified table names, with some exceptions.

 - no support for dblinks in Oracle
 - no support for serverlinks in MSSQL
 - no support for databases attached as namespaces in SQLite

For SQLite, Firebird and MS Access it is possible to pass a single
db_schema and have DBIC names qualified with it, but this capability is
primarily for testing and deployment.

silence comment tests

Adds quiet => 1 to loader_options for the comment tests to disable the
"Dumping manual schema..." warnings.

update Changes with dbicdump config file support

opt deps for dbicdump config and test

Adds Config::Any to dbicdump_config group in ::Optional::Dependencies
and Config::Any with Config::General to the test_dbicdump_config group.

Adds t/60dbicdump_config.t test for using a config file with dbicdump.

config file support for dbicdump script

Changes dbicdump so that if only one parameter is given it is used as
the config file loaded with Config::Any. The config file must have the
keys schema_class and connect_info, and optionally loader_options.

pigs don't fly, they just roll around in shit

Revert commit f1740f1b11e070f4e794fa43c606323d3ef528ab.

some changes to filter code

Renames filter_generated_text => filter_generated_code.

Changes the filter coderef signature to take the type of file being
filtered, 'schema' or 'result'.

Adds support for setting the option to a string which is used as an
external filter program.

Updates Changes.

adds the ability to filter generated code

Adds the filter_generated_text option which takes a coderef for
filtering the generated code. The coderef takes the class and code as
parameters.

flying pigs. almost as likely as win32 doing what you wanted it to.

minor changes to table/column comment code

Updates Changes.

Mentions Oracle support in generate_pod POD in ::Loader::Base.

Adds checking for generic comments via ->next::method in ::DBI::Oracle
as well.

Wraps table/column comment queries for MySQL on information_schema in
try blocks, as not all versions of MySQL have information_schema (5.0+
 do.)

Changes MySQL comment tests to use the new slurp_file util.

Makes the test messages in t/30_*.t more clear.

generic comments mechanism, MySQL specific support

Adds support for the table_comments and column_comments tables, which
override the db-specific metadata comment support in ::Loader::DBI.

Documents these tables in ::Loader::Base .

Also adds support for MySQL table and column comments from
information_schema.

fucking pigs broke my Win32 perl

The fucking pigs did something to my Win32 perl so that command line
args with special characters like [] and {} are not parsed correctly.
This problem does not show up for CPAN testers running these tests,
t/23dumpmore.t and t/26dump_use_moose.t .

This commit adds a check for refs in loader options or connect_info in
the t/lib/dbixcsl_dumper_tests.pm test lib, and skips tests on Win32 if
the FUCKING_PIGS environment variable is set.

support alternate line endings

Adds a utility method slurp_file to ::Utils for reading files in UTF-8
and converting the CRLFs or LFs inside to the platform \n. Changes all
uses of File::Slurp::read_file to slurp_file and changes some regexes to
match \r? at line ends.

fix t/10_11msaccess_common.t on non-Win32

Changes a "use Win32::OLE" to a require where needed.

fix backcompat tests

Updates the RelBuilder classes to take into account that
->_inflect_singular and ->_inflect_plural return a list instead of a
scalar now.

add quiet option

Also adds some improvements to the t/lib/dbixcsl_dumper_tests.pm test
library:

- list warnings if warning count doesn't match

- fixes undefined warning if there are more expected warnings than
  received warnings

make $schema->loader a public method

Also adds some TODO entries, adds Task::Weaken to dependencies and
removes a duplicate dependency on List::MoreUtils.

Removing these files from master because they are not for the next release.

I will recreate them on a branch, as I originally intended to.

Add failing test for feature I want.

add schema_components option

sort relationships

better option validation for result_component_map, result_role_map and rel_collision_map

add handle for Ben Tilly in CONTRIBUTORS

add rel_name_map option

fix syntax error in common tests

fix the decimal data type for MS Access over ODBC

handle MySQL enums/sets containing elements with single quotes

another enum test for MySQL

fix enum/set detection for MySQL (RT#68717)

document dumping a remote host

Signed-off-by: Caleb Cushing <xenoterracide@gmail.com>

fix is_nullable detection on MS Access

fix POD for scalarref tables

fix backcompat tests

fix uniq_to_primary test on DB2 and SQLAnywhere

improve uniq_to_primary test