Unhide ::SQLMaker::OracleJoins to properly appear on s.c.o.

Pass tests with DBICTEST_SQLITE_USE_FILE set

Changes for 135e8ce

Mark ->with_deferred_fk_checks as RDBMS-specific

Fixxor pause indexing

Ensure t/53lean_startup.t remains lean

Cleanup compose_namespace(), clarify leaktests wrt classdata

Fix segfault in t/storage/quote_names.t

Throw a sensible error when a malformed 'as'-spec is supplied

Cleanup warnings in t/55namespaces_cleaned, sophisticate require overload

Remove the transparrent hook lazy-pager-count experiment

It has proven a very stable and reliable implementation, but in the quest
for fatpacked DBIC should now go to the archives.

This atrocity will be removed from sqla, do not use here either

Fix find() pessimization when invoked with bare values

Add Changes entry for DBD::SQLite fixes.

Relax expected error message syntax.

SQLite 1.34 includes more detail in the error, allow any message
starting with "datatype mismatch".

Don't rely on GROUP BY and DISTINCT() sorting identically.

Release 0.08195

Fix oracle (I am so stupid)

Release 0.08194

fix link in replication introduction

Check DBD::Pg >= 2.17.2 with Pg >= 9.0 for BYTEA

Checks for DBD::Pg < 2.17.2 when using PostgreSQL version 9.0 or greater
when BYTEA column binds are encountered and prints a warning that a
newer version of DBD::Pg is needed for these types of columns to work in
bind_attribute_by_data_type in the Pg storage.

Skips the t/72pg_bytea.t tests for this combination of versions, as well
as for versions of DBD::Pg less than 2.9.2 regardless of Pg version.

fix popups during tests on Win32 (hopefully)

A tester reports that t/73oracle_hq.t causes an error popup on Win32.
This test was checking for DBD::Oracle (and thereby trying to load it)
before checking for the DBICTEST_ORA_DSN environment variables.

This commit moves the check for necessary environment variables for the
t/73oracle.t and t/73oracle_hq.t test files to before the module
requirements tests, thereby not trying to load DBD::Oracle, and causing
an error popup.

fix a couple test failures on Win32

Adds a skip_all to t/storage/global_destruction.t due to it causing a
segfault triggered by some sort of perl bug.

Adds a \Q...\E to t/94versioning.t in a warning regex as well as
changing a `perl -pi -e` to `perl -pi.bak -e` and removing an extraneous
semicolon for the test to pass.

Oracle: fix LOB conversions for non-LOBs (RT#69548)

Recalculates the LOB binds index for UPDATE queries, as it was
erroneosly not recalculated before. This resulted in non-LOB columns
being treated as LOBs by the SQL mangling code.

ribasushi also optimizes the code for the predominant case of queries
without LOBs in the WHERE clause.

Fix last test susceptible to spurious warnings, cleanup a bit

Reshuffle Makefile.PL a bit and add parallel-testing exception

Stop Data::Compare from loading plugins at will

Moar gitignore

Rip out forgotten dep

Really fix t/53lean_startup.t

fix $rs->populate([]) to be a no-op rather than an exception

Fix spurious test failures caused by use of Data::Compare

fix spurious PAUSE message

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