Merge branch 0.08200_track into master

Consolidate all constants under DBIC::_ENV_, bump n::c breakage to < 5.8.5

Overhaul populate code - fix \[] support and exotic values (arrays, etc.)

In addition cleanup populate() error messages a bit

Comprehensive MSAccess support over both DBD::ODBC and DBD::ADO

Fix/clarify Oracle decision whether to use WhereJoins

Remove the suggestion that 8i supports ANSI but is slow - leave it
as a test only (the docs are confusing enough already)

Fix (and test) quoting with the older style of WhereJoins used by Oracle

Also simplify the join overload process a bit

fix from => $rs->as_query

make the DB2/AS400 storage a subclass of DB2, do RNO detection, fix FetchFirst

Several 5.8.old fixes

* Stop running threading tests on < 5.8.5 - DBD::Pg is *really* unhappy there
* Switch t/55namespaces_cleaned.t to Package::Stash - the hand-written syntax
  breaks 5.8.1 and I'm lazy (and it's an implicit dep anyway)
* Stop auto-cleaning the imports of DBIC::Carp - it segfaults all over the
  place on 5.8.1 (will revisit when I rewrite n::c in pure-perl)

Massive rewrite of bind handling, and overall simplification of ::Storage::DBI

There's no practical way to split this into smaller pieces so here it goes:

Bind attribute handling was badly integrated into dbic almost from the
start. Until now the only information about a value was encoded as the
column name contained as the first element of the bind arrayref. The
column name was then resolved to the proper colinfo (deep in ::Storage::DBI)
and then a match was ran on the datatype to try to find an appropriate
set of bind attributes. Besides being fragile and inefficient, this
method also broke down completely when:
  * No column name could be associated with a bind (arguments to complex
    literal functions)
  * as_query results would encode the column names that can no longer
    be resolved since the inner result sources are no longer visible

To fix this all up and provide more flexibility the standard [ $col => $val ]
was replaced with [ \%args => $val ]. The format of \%args is currently:

  {
    dbd_attrs => '
      If present (in any form) this is what is being passed directly to
      bind_param. Note that different DBD's expect different bind args,
      e.g. DBD::SQLite takes a single numerical type, while DBD::Pg takes
      a hashref if bind options. If this is specified all other bind
      options described below are ignored
    ',
    sqlt_datatype => '
      If present it is used to infer the actual bind attribute by passing
      to $resolved_storage->bind_attribute_by_data_type(). Note that the
      data type is somewhat freeform (hence the sqlt_ prefix) - currently
      drivers are expected to dtrt when given a common datatype name (not
      ideal, but that's what we got at this point). Defaults to the
      "data_type" from the add_columns colinfo.
    ',
    sqlt_size => '
      Currently used to correctly allocate buffers for bind_param_inout().
      Defaults to "size" from the add_columns colinfo, or to a sensible value
      based on the "data_type"
    ',
    dbic_colname => '
      Used to fill in missing sqlt_datatype and sqlt_size attributes (if
      they are explicitly specified they are never overriden). Also used
      by some weird DBDs where the column name should be available at
      bind_param time (hello Oracle).
    ',
  }

For backcompat/convenience the following shortcuts are supported:

  [ $name => $val ] === [ { dbic_colname => $name }, $val ]

  [ \$dt => $val ] === [ { sqlt_datatype => $dt }, $val ]

  [ undef => $val ] === [ {}, $val ]

  ( pending in the next patch: [ $val ] === [ {}, $val ] )

On each passage through the storages (either for execute or for as_query
formatting) the information is filled in whenever available, so that by
the time the final binds_param takes place ::Storage::DBI::_dbi_attrs_for_bind
has all the available information about a particular bind value (no matter
where it came from).

A side efect of this is that as_query now always returns resolved
[ \%args => $val ] forms of bind values (hence the huge amount of test changes
in this patchset). While it should not be a major concern, it could
potentially throw off tests that expect a specific output of as_query. If
this becomes a problem a "compat mode as_query" flag will be introduced asap.

Additional changes in this patchset are:

* The signatures of pretty much the entire execution chain changed. Luckily
  everything that required changing was private. All drivers were adjusted
  appropriately (though something could have been missed). Affected methods
  on ::Storage::DBI are:

  _prep_for_execute
  _dbh_execute
  _execute
  _select_args_to_query
  _max_column_bytesize

  additionally the invocation of _prep_for_execute moved from _dbh_execute
  to _execute, and the return of _select_args also changed

* source_bind_attributes was deprecated. Luckily it was never documented in
  the main documentation. Sadly it was documented in individual storage
  drivers. As such it was necessary to provide a compat shim that would invoke
  the thing if it is detected (with the approproate warning)

* _fix_bind_params was renamed to _format_for_trace

Privatize _sth

Shuffle tests around (no changes)

Correctly select the root source of the inner part of a complex prefetch

Also better diagnostics to aid source mis-application during colinfo
resolution

Stop flooding the console with garbage on pg blob errors

Ditch Carp::Clan for our own thing

Fix syntax error that slipped into 9c1700e3

use relationship bridge consistently w/ many_to_many speak

Delete duplicated test (somehow 1:1 with t/search/subquery.t)

Fix long relationship/column names in oracle for good

Do not pass in the relname as "keyword" anymore - it will interfere
with the aliasing since ::Storage::Oracle and SQLMaker::Oracle will
produce different results

Fixup SQLA monkeypatch in anticipation of new SQLA version

Cleanup that namespacing mess

Remove significance of some test envvars (and add a test to trip up downstream packagers)

Remove test prototype that was never completed further than a loop calling ok(1)

Makes DBICTEST_STORAGE_STRESS redundant

Add startup sanity check of the ::DBI::Replicated method dispatch tables

In order to do this during testing only introduce the ::_ENV_::DBICTEST macro,
and also make sure DBICTest::RunMode is loaded before the macro is set
(therefore the multiple test changes)

Lazily load DBD::Pg in ::Storage::DBI::Pg, only when typing is required

Fixed bug in _strip_cond_qualifiers, patch generously by Grant Street Group.

Fixed: RS example code was broken

cascading delete on a nonexistent relation should warn instead of
dieing uninformatively.

reorganize docs for load_namespaces method

Fix 'exit via next' warning from DBIx::Class::ResultSource::sequence

Make sure the component_class autoloader works across module reloads

(works around corner cases from db29433c)

fixup Firebird ODBC driver for DBD::ODBC 1.29

Switch sql_maker_class and datetime_parser_type to component_class accessors

Switch cursor accessor to CAG's component_class type for autoloading

create file_columns table when DBICTEST_SQLT_DEPLOY env var is not set

Breakage introduced in 123df52

Minor POD spelling and link fixes

* Use proper L<Pod::Name/Section> style, rather than the incorrect
  L<Pod::Name#Section> style (thanks mst)
* Some minor spelling errors picked up by Lintian

Modified: small doc patch explaining a gotcha

Extra test for mysql_auto_reconnect and fork

Lazy-load as many of the non-essential modules as possible

Cleanup C3 handling, require updated Class::C3::Componentised

Fix quote_names assumption of DBD::* availability

Move the test bits related to deprecated IC::File out of the main testschema

Make pg tests cleanup better after themselves

Fix rev_rel_info on prototype sources

Simplify implementation of reverse_relationship_info (no func. changes)
The only difference is getting rid of an arrayref support condition, but
it is not supported at the top level anyway

Temporarily monkeypatch DBD::ADO to fix warnings during global destruction

rewrite SQLAnywhere GUID normalizing as a cursor_class (formerly a _select_args hack)

Refactor UUID generation logic in ::Storage::DBI::UniqueIdentifier

Patch to schema->deploy against a file

Fixes skipped lines when a comment is followed by a statement
Explicitly split on single line comments and not just ;\n
Add some FIXME comments on the state of deploy

use -in SQL::Abstract operator instead of literal 'IN' in cookbook

Display a warning when an ODBC or ADO subclass is not found

Multiple code/test/doc improvements for MSSQL over DBD::ADO

Add extra test blob type

Cleanup/extend blob/clob detection/checks

Use a safer $dbh->get_info

Fix replicated fails introduced with 2b8cc2f2

fixed description to cookbook example

fixed cookbook example

add mailmap for consistency

Add proper DateTime handling to t/52leaks.t

Better exception when things get really really bad with prepare()

add quote_names connect_info option

Fix wrong author email from f92a9d79

Fix broken sections links in POD

Fix for SpeedyCGI and an extensive test of persistent environments (RT#65131)

Ignore to-be-leaktested classdata, instead of forcibly freeing it

Correct Makefile.PL authortests borkage

Fix last-resort error message and do an explicit require (though
it should *not* be necessary people broke it somehow)

Stop eating syntax errors in ODBC drivers

Fix broken select/group bind in Oracle (after 0542ec57 & 4c2b30d6)

Disable default mysql_auto_reconnect for MySQL

improve MSSQL+ODBC initialization, clarify docs on SCOPE_IDENTITY() and dynamic cursors

Release 0.08190

remove mention of name_sep from Schema::Loader doc

update Schema::Loader docs

Merge branch 'people/riba/extrels' into 0.08200_track_do_not_rebase

Rewrite the check for nontrivial joinfree conditions to throw on both new_related and set_from_related

Add some extra tests written while debugging, remove design draft

Fix $object->search_related aliasing, change semantics of _resolve_condition

Change the RV of _resolve_condition one last time. Now it checks the RV of the
resolved coderef (if any), and chooses (preferrably) the join-free condition
or the cross-table one. In list context returns a flag signifying if the
resulting condition is available for standalone use (false) or requires the
joins to remain (true)

Pass relationship name to _resolve_condition explicitly

The API is ambuguous at best, will need a cleanup some day, for now just do by
tackling even more arguments to the method, yielding insanity like:
_resolve_condition($cond, $obj, $f_rel, $f_rel) BLEH!

Sanity check args passed to a coderef conditionmaker

use $self->{from_bind} for extra binds transport

The __overriden_select method was used to build extra binds
that should be transported in the $self->{from_bind} array
instead.

Untodoify tests - these are not 'nice to have', they must work

Back out "support for prefetch from resultsource using extended_rels"

Blindly taking the condition retuned by an incomplete (not
all pieces of the hashref are supplied) call to the condition
maker, and attempting to order by its keys assuming it is a
hash is... naive

do not use "me" on the related_resultset pessimization

This was requested by ribasushi, but it is still incomplete.
Added a TODO test with the expected final query.

Problem is: the alias is defined in $source->_resolve_join,
which is completely disconnected from the search context
so it cannot receive a different alias

refuse to invoke create_related as class method

t/cdbi/09-has-many.t has a specific test for it.

fix the alias in the identity column

When building the identity where clause, discover the current_source_alias
to build the hash correctly

eager check if the create_related is at all possible

When calling create_related, it should stop trying if the extended rel
does not provide the complete relationship definition.

When getting a related_resultset in a row with custom rels without the extended definition, we step back, get a rs equivalent to the current row and do a search_related there.

adds a test with many to many and extended rels that is currently failing.

support for prefetch from resultsource using extended_rels

checks if the complex conditions are overriden in set_from_related

Switch code/documentation/examples/tests to the new single-arg syntax

Relationship documentation on extended (custom) relationship conditions

kills test that wrongfully expects sqla to die with hash in joins

cascades default to 0 when using extended rels

makes search_related on extended rels without the optimized version work. involves hacking the from attribute

Most of the code reviewed... Missing ResultSet->populate, ResultSet->find, Row->copy and Relationship::Accessor->add_relationship_accessor

A spec of what needs to happen to get complete custom relationships support

First attempt to make extended_rels work.

Reshape initial tests