Remove implemented items from TODO

Don't ship MYMETA.* files (RT#87713)

Demote diag about skip reason to note

Fix typos in POD and comments (RT#87644)

Avoid conditional assignment in variable declarations

any() already returns a boolean, just assign it directly

Release 0.07036

Ignore dist directory

Ignore Emacs droppings

Fix ordering issues in Pg loader

- Order foreign keys by name
- Order enum values by explicit order column or OID if unavailable

Fix tabs in Changes file

Fix "table" parameter in col_accessor_map callback (RT#84050)

Keep the old, erroneous "table_name" for backward compatibility.

Fix MySQL enums with empty strings and leading/trailing quotes (RT#86091)

Fix stray comma in Pg on_delete/on_update => CASCADE (RT#84706)

Release 0.07035

Release 0.07034_02

Fix fixtures to work with older DBD::SQLite

Release 0.07034_01

Fix typo in t/46relationships_multi_m2m.t

s/twoes/twos/

Remove implemented item from TODO

pass link table details to rel_name_map for many_to_many bridges (RT#81091)

If two tables have multiple many-to-many link tables between them, the
rel_name_map coderef needs to know which link table it's naming the rel
for.

Reorganize and freshen up deps

better schemas support for SQL Server > 2000

Use schema_name() instead of user_name() on > 2000 to detect the
current schema, and query [$db].sys.schemas instead of
[$db].dbo.sysusers for schema names on > 2000.

frew has reported that schema_name() has worked better for him for users
which aren't the owners of the schema.

introspect ON DELETE/UPDATE for SQL Anywhere

The default seems to be RESTRICT, but it's hard to tell, because the
actions are implemented as triggers and you have to query the catalog
triggers view to get at them. At least if you specify RESTRICT there is
no trigger generated.

SQL Anywhere does not support DEFERRABLE constraints. It has something
called CHECK ON COMMIT but that does not seem to have the same full
functionality, so for now we leave is_deferrable at 1.

Pg _table_fk_info to use catalog not info_schema

Rewrite Pg _table_fk_info query to use the catalog views rather than the
information_schema, as a user on IRC reported that the
information_schema query does not work for readonly Pg users, but the
catalog query does work.

update deps on ::Phrase and ::Tagger

Update the dep on Lingua::EN::Inflect::Phrase to 0.15 which is much
improved.

Update the dep on Lingua::EN::Tagger to the newest, 0.23 because 0.20
and earlier generated a warning which made Schema::Loader tests fail
(unescaped braces in regex) in perls version 5.17+.

add rel_type param for relationship_attrs coderef

Forgot to pass some sort of param for the rel type for the
relationship_attrs coderef, adding 'rel_type' with the value
'belongs_to', 'has_many' or 'might_have'.

Add tests to 45relationships.t and doc in ::Base POD.

release 0.07033

add a loud warning to Makefile.PL about rel attrs

Print a big boxed warning when running Makefile.PL (e.g. when installing
via CPAN) about the changes to belongs_to relationship attributes in the
last few revisions at the prompting of mst.

expand relationship_attrs POD

Explain the defaults for relationship attributes, how they are
overridden by the actual database values for foreign keys and can be
overridden by the user via this attribute.

Also explain how defaults are chosen for different databases for
belongs_to rels.

release 0.07032

detect inline deferrable FKs for SQLite

The previous attempt to detect DEFERRABLE on FKs only detected for
FOREIGN KEY clauses in DDL, not for inline constraints such as:

    int foo references bar(id) deferrable

add code and tests to detect inline FKs as well.

support coderef for relationship_attrs

Add support for passing a coderef as relationship_attrs and describe
thorougly in the ::Base POD (as well as improve the existing POD for
 it.)

This is the list of parameters:

 * rel_name

 * local_source

 * remote_source

 * local_table

 * local_cols

 * remote_table

 * remote_cols

 * attrs

The hash is passed as a list, not as a hashref.

The coderef must return a hashref of attributes or nothing.

This is thoroughly tested in t/45relationships.t.

I had to add a fallback => 1 to the DBObject.pm stringification
overload, so that eq would work, and this makes the stringification much
more functional.

author dep on POD test modules, check in 02pod.t

Use ::Optional::Dependencies for POD testing modules, so that they are
author deps and the minimum required version of Pod::Simple (3.22, which
checks that the correct =encoding is set) is also used.

change =encoding utf8 to =encoding UTF-8 in Loader

I didn't know why 02pod.t was erroneously passing for me, ilmari pointed
out that my Pod::Simple was out of date, and I needed at least 3.21.

Upgrading Pod::Simple caused 02pod.t to produce the fail when the
=encoding directive was missing.

I like using the UTF-8 encoding rather than the more lax utf8 encoding,
so change it now that I can properly test it out.

release 0.07031

fix 02pod.t fail for lack of =encoding (RT#79481)

Users report 02pod.t failing due to a UTF-8 character without an
=encoding utf8 statement (arcanez and Marcel Gruenauer.)

For some reason this test passes on my machine, even though I have the
latest Test::Pod.

Marcel Gruenauer sent me a patch to RT which adds the =encoding utf8
statement at the top of the POD, which I added.

I want to use =encoding UTF-8 instead, since that's more correct, but
since I can't test it will leave utf8.

release 0.07030

allow user to set qualify_objects to false

The qualify_objects attribute in the DBIx::Class::Schema::Loader::Base
was automatically set to true when there were multiple db_schema's, even
if the user explicitly set it to false. This changes that behaviour,
allowing more flexibility on prefixing the names.

release 0.07029

introspect ON DELETE and DEFERRABLE for Oracle

Oracle has no ON UPDATE rules, so change the default to NO ACTION so
that users have no surprises when deploying an Oracle schema to SQLite
etc.

Add support to the generic ::Loader::DBI::_table_fk_info for
introspecting ON DELETE/UPDATE rules and DEFERRABLE clauses, as much as
the DBD supports it.

For Oracle I had to get the DEFERRABLE value manually in an override, as
well as making sure the on_update was set, as the Oracle driver uses the
::Loader::DBI method.

introspect ON DELETE/UPDATE FK clauses for DB2

is_deferrable still defaults to 1 because DB2 does not have deferrable
FKs. At some point we will try to implement with_deferred_fk_checks for
DB2 in DBIC.

introspect ON/DEFERRABLE FK clauses for SQLite

The ON clauses are simple to get, pragma foreign_key_list returns them,
but for the DEFERRABLE clause we have to parse the table DDL from
sqlite_master, a bit hairy but should be correct.

release 0.07028

bump Lingua::EN::Tagger dep to 0.20

Lingua::EN::Tagger before 0.20 had some unescaped curly braces in
regexes, this produces a warning on perl 5.17+ which breaks our tests.

FK ON clause introspection for MSSQL

Augment the _table_fk_info code which uses information_schema to get the
delete_rule and update_rule, similarly to how it was done for
PostgreSQL. MSSQL does not have DEFERRABLE, but it has a way to disable
FK constraints on a table temporarily, which we will implement for
$storage->with_deferred_fk_checks for MSSQL in DBIC. For this reason,
is_deferrable still defaults to 1 for MSSQL.

minor optimization for Pg FK attrs, better test

Set the FK attributes in Pg _table_fk_info only once instead of once for
each column.

Also test DEFERRABLE being on instead of off, since it defaults to off.

release 0.07027

introspect ON DELETE/UPDATE and DEFERRABLE for pg

Apparently Pg has an information_schema, so I copied the _table_fk_info
from the MSSQL driver and modified it to introspect the ON and
DEFERRABLE clauses.

I will add this introspection for MSSQL shortly as well, after which I
will remove the duplication from these two drivers and add an
InformationSchema component.

release 0.07026

introspect ON DELETE/UPDATE clauses for MySQL FKs

The default for these has been CASCADE for a while now, the idea was
that this is the most useful default for deployed schemas and it would
be replaced with the actual introspected value when this capability
would be added.

A user wrote in to RT that this capability was missing for MySQL so I
decided to start on adding it, with MySQL first.

This means that for MySQL the default for on_delete/on_update is now
RESTRICT rather than CASCADE, and this will probably soon be true for
other backends that support these clauses.

Also there are some whitespace changes in this commit due to some
cleanup autocommands I put into my .vimrc, read it with git diff -w.

update rbuels' email address in CONTRIBUTORS

release 0.07025

support MSSQL 2000 again, optimization for MSSQL

multischema support broke compatibility with SQL Server 2000 in 0.07011,
because system tables were used to find identity columns, and system
tables changed between 2000 and 2005.

We now check the SQL Server version and use a different query for 2000
and later versions.

Also optimize columns_info_for to use one query for the whole table,
instead of doing two separate queries for each column.

release 0.07024

work around broken kseq in Pg fk info (RT#77062)

The keyseqs from $dbh->foreign_key_info from DBD::Pg were erroneously
high, and since they are used as an array index, they were causing the
fk_info to return undefs and break everything.

Add a grep defined, to the column arrays in fk_info which fixes
everything.

release 0.07023

fix base ::DBI loader to properly order FK columns

bump up Class::Inspector dep for blead (RT#74236)

release 0.07022

separate queries for default on ASE (RT#74170)

Some Sybase ASE servers apparently cannot LEFT JOIN to the syscomments
table to get the default_value, so do the query separately.

Also combine the data_type/len/prec. and is_identity queries.

set correct size for n(var)?char columns for ASE

Based on @@ncharsize.

make common tests use fewer active statements

Newer versions of the Sybase ASE developer's edition have a limit of 25
user connections and do not allow setting it higher, this did not
support the common tests.

Change the ->first calls on ResultSets to ->single to not keep active
statements around.

The common tests now run against the latest version of Sybase ASE
developer's edition.

support Informix common tests over shared mem

Support the default Informix connections over shared memory rather than
TCP for running the common tests, including the extra tests.

Shared memory connections have the limitation that a process may only
open one connection to the DB server at a time.

Also fix the t/lib/dbixcsl_common_tests.pm to work with only one
database connection at a time.

release 0.07021

register sources on schema class, never instance

Change _invoke_loader in Loader.pm to always invoke the loader on the
Schema class, never the instance, even if an instance is passed in, and
then merge the state of the class into the instance if necessary.

Fix up a couple of tests to work with this new logic.

Also change make_schema_at to use ->connect instead of ->connection
because of people relying on instance state in ->connection (RT#74175),
being careful to copy the storage back to the Schema class.

release 0.07020

fix mro issues under 5.8

Add a Class::C3::reinitialize() under 5.8 after the ::DBI ->_rebless.

Also make sure MRO::Compat is loaded everywhere it needs to be and
everything does a use mro 'c3';.

release 0.07019

Also remove a leftover ->_lc call in ::DBI::_columns_info_for that was
screwing things up.

audit drivers for case issues (RT#75805)

Add missing ->_lc calls to MSSQL driver in _table_pk_info and
_table_fk_info, and rework use of column name in queries in
_columns_info_for based on the value of _preserve_case. This fixes the
RT in question, as tested with the DDL provided by the reporter.

Fix collation detection in MSSQL driver when in a database other than
master by changing 'sys.databases' to '[$db].sys.databases' and doing a
"use [$db]" beforehand (with a warning silencing fixup for ADO.)

Check for collisions of columns like 'Foo' and 'fOO' in preserve_case=0
mode in ::DBI::_columns_info_for and throw an exception if detected.

UNRELATED CLEANUP: Remove warning fixups for _table_comment and
_column_comment from Access and MSSQL ADO drivers as we now check for
the existance of the comment tables.

Add missing ->_lc call in Pg _table_uniq_info, which was only for very
old versions of DBD::Pg anyway.

In _columns_info_for in the SQL Anywhere driver, fold the column names in
queries to lower case, as SQL Anywhere is case preserving, but not case
sensitive.

In _columns_info_for in the SQLite driver, map the column names from the
table_info pragma to the ->_lc versions.

Fold column names to lowercase in the mysql driver's _columns_info_for
as MySQL is case preserving but not case sensitive for column names.

release 0.07018

skip dbicdump tests on Win32 due to test fails

t/23dumpmore.t and t/26dump_use_moose.t fail on Win32 with some types of
refs and characters  in -o params to dbicdump because of quoting issues.

There is currently no module like String::ShellQuote for Win32, haarg is
releasing Win32::ShellQuote soon, and then I will reenable these tests.

Also the fucking pigs completely fucked up my windows computer and when
I was trying to run some tests with dbicdump I got constant perl hangs,
so I cannot debug this right now and I disabled ALL dbicdump tests on
Win32.

I removed the reference to the FUCKING_PIGS environment variable
because, at least, the behavior of the tests is now at least consistent
with that of other machines, or was, except for the perl hangs I'm
getting now, and because I'm making this skip for all dbicdump tests,
not just those with refs so it is in the same part of the code, so might
as well.

Changes entry for "fix undef warn in fk info"

fix undef warn in fk info for DBDs without schemas

SineSwiper reported an undef warning from ::DBI in _table_fk_info
because of an s/// substitution on the schema values extracted from
$dbh->foreign_key_info, which can be undef for DBDs which do not have
the concept of schemas.

Change the substitution to only run if the values are defined.

work around ORA-24345 from $dbh->column_info

jhannah reported getting ORA-24345: A Truncation or null fetch error
occurred (DBD ERROR: ORA-01406 error on field 13 of 18, ora_type 8,
 LongReadLen too small and/or LongTruncOk not set)

on running the loader on a table with a very long DEFAULT of
"to_number(decode(substrb(userenv('CLIENT_INFO'),1,1),' ',
null,substrb(userenv('CLIENT_INFO'),1,10)))"

The fix is simply setting $dbh->{LongReadLen} in the _dbh_column_info
override for ::DBI::Oracle, but also make it work without the setting by
making the code involved much more robust.

Change Loader/DBI to fall back to the non-column_info code (that does a
 SELECT ... WHERE 1=0 and examines the $sth) when $dbh->column_info
fails in _columns_info_for, and wrap the fetchrow_hashref in a try {}
catch { +{} }; then skip columns where fetchrow_hashref fails and fill
them in from the WHERE 1=0 type_info code.

Also use this method instead of $dbh->column_info in Loader/DBI/Oracle
_table_columns by removing the overloaded method and using the
Loader/DBI implementation.

While there, clean up the _table_columns implementation in Loader/DBI to
use $self->_lc insetad of $sth->{NAME_lc} for consistency and
overridability.

Make the ::DBI::Oracle _columns_info_for pass all tests with both the
$dbh->column_info output and the type_info output from the parent ::DBI.

fix spelling mistake in Base POD (RT#74796)

release 0.07017

experimental support for Pg loader run inside txn

To get this to work, there must not be any database errors thrown. So we
cannot use our usual technique of selecting from nonexistant tables and
throwing away the errors.

To that end, _table_comment and _column_comment in ::Loader::DBI now
check for the tables in $self->_tables before doing a query on them.

jnap added a test to the common tests in a previous commit which I
augment here to be $TODO for DBs other than Pg and to throw away the
usual rescan warnings.

There is also a warning for 'commit ineffective with AutoCommit enabled'
which we also throw away and is more worrisome, and will need further
attention, and is one of the reason this support is marked experimental.
That and this test failing for SQLite as well as some other DBs.

failing test for dumping schema inside a txn

add files created by Mac OS X to .gitignore

add SineSwiper to CONTRIBUTORS

Ignore metas

Fix minor $dbh var adjustments

Add support for REMARKS in *_info to use for *_comments

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.