Release 0.08193

replaced Test::Deep::NoTest with Data::Compare to not break Test::Deep tests

Stop leaking cursors on oracle sequence retrieval

SQLMaker is a proper C3-enabled class, do not use SUPER::

fixed failing test when DBIC_TRACE_PROFILE is set

check database driver dependencies for tests

Open the logfile in append mode

Fix t/55namespaces_cleaned (RT#68814)

How ::Rekey was tested against the DBIC test suite

Yet another SQL munging atrocity (shadow needs it)

Fix all invocations of _resolve_condition to match the new signature

Do not consider non-sequence oracle triggers during seq detection

fix bigint handling in sqlite

Correct test-relationship name/type

More robust insert() logic to avoid undef warnings

Make Ordered play nice with other things hooking update()

explicit int cast, prevents the result rounding up to 1000 (4 digits)

don't mask missing deps with datetime_undef_if_invalid

Fixed a typo in the POD

don't add Sys::SigAction to author deps on Win32

Make Access inner joins 'INNER JOIN' to avoid JOIN syntax errors

fixed order of rows difference between first and subsequent pages for Oracle

Fix stupid oversight in update_all

Augment shorsighted code change in 5ef76b8b

The way _column_data_in_storage was unconditionally referenced, would
break both id() and ident_values(), which are supposed to construct
the identity based on current object state

The test is designed to work with blob-updates only, remove plain values

Rewrite handling of "storage-side old values" both for PK/ident and for
Ordered position/grouping

Massive code simplification, also opens the door to vastly simplifying
(and optimizing) DBIx::Class::Helper::Row::StorageValues, as only
differing values are kept by the core

Minor cleanup of ResultSource attributes

fix doubling of find_related condition

Fix newlines in skip messages

Extra cleanup of leftovers from 70171cd7

Also some commentary

Added tests for chained sqlt_deploy_hook()'s

We need to declare the sqlt_deploy_hook sub's in the code to make sure the
next::method knows they exist.

Signed-off-by: Pedro Melo <melo@simplicidade.org>

Revert "Fix incorrect default sqlt_deploy_hook signature" (fed15b916)

With this change, code that uses next::can or next::method inside
sql_deploy_hook(), like components, stops working.

Also fix documentation to match the actual code

Allow changing attributes on schema cloning

[docs] remove a stray "1;"

[doc] replace pseudocode with real code

fix typo

add DateTime parser for Sybase ASE

change FreeTDS related warnings to carp_once

make Oracle trigger queries safe for coderef connect_infos

take more care in mangling SELECT when applying subquery limits

Previously, if there was a SELECT sub-query in the WHERE
clause, the regular expression was being a bit too greedy
and stripping too much out.

Release 0.08192

turn 'use vars' into 'our'
move the $VERSION up to the top of DBIx/Class.pm to aid in version detection
(things like Module::Metadata find the $VERSION string and base it on the last package statement it has seen, not knowing that } 'ends' the package)

Fix incorrect > vs >= ver. check, causing 5.13.1 fails

remove meaningless mysql test that breaks on Win32

remove TODOs solved by helpers

Fix serious DBD::SQLite numeric datatype mismatch regression

remove done TODO entry (infer quote_char)

Release 0.08191

Added documentation for the -ident/-value operators in SQLMaker.

Skip blob like ? comparison on older oracle versions (doesn't work)

Release 0.08190_01

Fix assumption in Sybase::ASE blob updater: cond is not always a HASHREF

Throw away the error prone _strip_cond_qualifiers

Review of 61f031bf made it clear that in order to make this work properly
pretty much the entire sqla logic will need to be replicated. So instead
the hacky approach was taken to abuse a botched version of SQLA which
is guaranteed to parse its own command tree correctly :)

Revert the smart-select-ordering introduced in 36fd7f07

While on its surface an awesome idea, it proced to be problematic:
users of ->cursor and ->as_query expected that the result will have the same
column-order as what they put in the arguments. Also users abuse +select as
there's no tomorrow, which yielded interesting bugs. The current (and I hope
final) behavior is:

At the end of a chain, the resulting SQL will contain:

- a merger of all columns and then all +columns attributes in the
  declared order. If columns is declared as a multiselect hashref
  i.e. { as1 => sel1, as2 => sel2 }, then the selectors for this
  segment are ordered alphabetically. De-duplication is also always
  performed on columns arguments - only the first set of identical
  select/as pairs is preserved

- a merger of all select/as pairs, strict order, no deduplication

- a merger of +select/+as pairs, same rules as above

As a result of this prefetch on un-balanced resultsets now properly throws
an exception

Channel all DBIHacks methods to the replicated writer

Note this is a workaround to fix an actual reported bug. The whole system
reeks of misdesign, and needs to be revisited at some point.

Revert incorrect assumption about non-functional cached pagers 65245220

While on the surface it seems logical that a cached resultset is inherently
non-pageable, one has to remember that search chaining with different
arguments (i.e. different offset) fires a new query, not subject to the
existing resultset cache.

Use clearer varnames

Fix incorrect default sqlt_deploy_hook signature

Documentation regarding resultsets in list context

Cleanup Oracle's 00a28188 / add support for update/delete with blobs in WHERE

Make a separate path for as_query, so that the sql does not pass thorugh storage
multiple times. Also do not mangle the sql unless blob binds are detected.

fix MSSQL on_connect_call => 'use_dynamic_cursors' regression in 1db83fb9

prefetch docs: prefetch works on all rel types

Update the prefetch docs to state that prefetch works with all
relationship types, where previously it had said that it works with
only belongs_to and has_one.

Also update the subsequent example. The old example was confusing
because it specified two has_many relationships on the same level,
something that DBIC emits a warning about.  That example is replaced
with a CD-centric example that demonstrates how many different
relationship types can be used together in a prefetch, but omits the
multiple has_manys.

Add a warning, example, and explanation of the troubles with using
multiple has_many relations in a prefetch.

The old text incorrectly stated that prefetch will override join and
select attributes. Describe current behavior with examples.

savepoints for SQLite

Another overhaul of transaction/savepoint handling

- Move most of the transaction logic back to ::Storage, only leave
  DBI-specific overrides in ::Storage::DBI
- Fix bug where a nested rollback would destabilize the entire execution
  chain
- Better checks for connectivity/ordering of txn/svp operations
- Make DBIC::Storage::NESTED_ROLLBACK_EXCEPTION a proper exception subclass
- Standardize the txn/svp names - renamed the actual workers
  _dbh_begin_work               to _exec_txn_begin
  _dbh_commit                   to _exec_txn_commit
  _dbh_rollback                 to _exec_txn_rollback
  _svp_[begin|release|rollback] to _exec_svp_[begin|release|rollback]

Cleanup/improve the leaktest a bit

Tweaked example code for having non-db data accessors.

Fix custom-relationships on resultsources with multilevel monikers

Fix stupid optimizer bug with has_many joined over might_have

Drop DBI requirement down to 1.57

SQLite, MySQL, Pg, Oracle, ODBC and Sybase all tested to work

Fix $rs->next when using HRI with right-join-side data injected into main obj

handle BLOB and CLOB columns in WHERE for Oracle

Add retrieve_on_insert column_info flag, to autoretrieve RDBMS-side defaults

Also move the UniqueIdentifier code from insert() to _prefetch_autovalues()
where it makes more logical sense

make a proper storage base class for Firebird

fix some spelling mistakes

support sub-second precision for TIMESTAMPs for Firebird over ODBC

Fix complex limits with subqueries in selectors

(fix both incorrect splitting of the selector list and duplication of binds)

support for DBD::Firebird

Parameterize pagination

Use _is_binary_lob_type to determine datatypes in need of PG bytea binds

Remove use of _resolve_column_info call from _prep_for_execute in NoBindVars

Use the datatype as passed via the bind attribute resolution

Some test fixes and TODOs for older Oracle versions

Refactor/improve support of DBD::Sybase compiled against FreeTDS (mainly MSSQL)

Extend tests for MARs under MSSQL over DBD::Sybase

Consistently use MyApp::Schema throught the documentation

Test for an insane yet valid rel condition with subquery

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