Fix RowCountOrGenericSubQ dropping group/having/order on the floor (RT#73244)

When realiasing carry all 'renames' in longest first order

Anything that isn't in the selector list needs a realias

Always reselect a bind-containing order by even if appearing in the select list

Do not alias plain column names to the inflator spec, do it only for funcs

This solves a problem with deliberate column renames in complex subqueries

Fix bind order in RNO limits when there are both select and order bindvals

Sanify handling of genericsubquery order determination, and order binds

Simplify 3-query skimming limits to omit the last redundant limit

Simplify skimming limits - simpler query when no offset is given

Refactor a lot of the limit dialects common code - (hopefully) no changes

fixed current_source_alias example code

Fix broken tests that would fail in future SQL::Abstract

Check more files for notabs/eol

No longer depend on MI::AuthorTests. Switch xt tests to run on make dist

Relax requirement of a config stanza introduced in 7b71391b3
A schema may very well define a configuration internally

Fix buggered skip, broken by reorg in d9bd5195

Removed PRIVATE METHODS pod section from Ordered.

Allow for tests to run in parallel (simultaneously from multiple checkouts)

This is an interim solution and is by no means the final thing. It simply
was possible to do in a short timeframe and cuts the test run time in half.

If you have DSN envvars set, use at least -s -j8 for best results (the
shuffling un-bunches similar tests, see discussion below)

Two things are at play:

First of all every SQLite database and every temp work directory is created
separately using the pid of the *main* test process (there can be children)
for disambiguation. Extra cleanup passes have been added to ensure t/var
remains clean between runs.

All other DSNs are reduced to their ->sqlt_type form and the result is used
for a global lockfile. Said lockfile is kept in /tmp so that multiple
testruns from multiple directories can be run against the same set of
databases with no conflicts.

Some of the tests are explicitly exempt from any locking and will run
regardless of environment, for example t/storage/dbi_env.t

The lockfiles are deliberately placed in File::Spec->tmpdir. This is done
so that multiple dbic checkouts can run against the same set of DSNs without
stepping on each other's toes.

Some notes on why this is not a great idea, even though it works flawlessly
under continuous test cycling: The problem is that our tests are not yet
ordered in a spwecific way. This means that multiple tests competing for
the same resource will inevitably lock all available test threads forming
several bottlenecks along the path of execution. This issue will be adressed
in a later patch, with the following considerations:
  - prove -l t/... must continue to work as is
  - test aggregation is something the test suite should try to avoid in
    general - after all DBIC is intended to be usable in CGI (yes, pure CGI)
    environments, so if the tests are getting heavy to run - this is an
    actual problem in need of fixing. Aggregation will instead sweep it under
    the rug
  - general reorganization of test groups / various path changes should only
    be attempted once we have a solid base for multi-db test runs

Reorganize CDBICompat tests - centralize prereq checks in one place

Fix complete breakage of ddl_filename on certain paths

The careless regex substitution was a poor choice, which became evident when
the db file switched to pid-postfixed names. The result was that tests
upgrading from v1.0 to 2.0 would completely obliterate a path ending with
/2\d0$/

Fix incorrect DSN in test (does not affect test condition)

Bump Moo deps

Enhance 19b088d1 to add SQLT dep regardless of Makefile mode

add a link to doc on MAS for MSSQL+ODBC

Add a link to
http://www.easysoft.com/developer/languages/perl/multiple-active-statements.html
to the POD for ::DBI::ODBC::Microsoft_SQL_Server in the Multiple Active
Statements section.

Better detection/synchronization of MSSQL dynamic cursor state

Revert 796acb6a - latest FreeTDS does not support MARs in any form

It looked like this was the case due to a botched test, which would allow
a silent reconnection to take place. What was thought to be a proper MARs
configuration turned out to be simply two connections working in sync

(view patch with diff -w)

added paragraph to update_or_create and find_or_create docs how to know if a row was updated/found or created

move schema state copy to _copy_state_from

Move the stuff needed to copy the state of a parent schema to another
including making a copy of the class_mappings and source_registrations,
registering sources and copying the storage to the private
_copy_state_from method which clone calls.

check DBD::ODBC version when disabling array ops

After 1.35 DBD::ODBC will disable array operations for execute_for_fetch
by default, and the $dbh attribute is changing from
'odbc_disable_array_operations' to 'odbc_array_operations', but we still
set the new attribute to 0 just in case.

Put the version checks and the setting of the appropriate attribute in
_disable_odbc_array_ops in ODBC.pm, and call it from the 3 drivers where
we currently need to disable them.

improvements for MSSQL driver via DBD::ADO

Various improvements and bug fixes:

 - Fix transaction support by changing CursorLocation on connection to
   adUseClient.

 - Support for VARCHAR(MAX)/VARBINARY(MAX)/NVARCHAR(MAX), add a catch
   for size => 'max' in _dbi_attrs_for_bind to set the ado_size workaround
   to LongReadLen, add a mapping to the data_type for literal
   'varchar(max)' entries as well. Tests for these types have been added
   as well.

 - Set $dbh->{LongReadLen} * 2 + 1 on connection, necessary for some LOB
   types.

 - Remove trailing null padding from BINARY/VARBINARY/IMAGE data via a
   custom Cursor class.

 - uniqueidentifier brace removal from data returned from ADO via Cursor
   class and re-addition via _dbi_attrs_for_bind and insert_bulk
   override for data sent to ADO.

 - Add type mappings for some missing MSSQL types and the types from other
   databases for cross-deployment.

 - Full set of tests, stolen from the MS Access tests, for savepoints,
   LOB support and GUIDs.

The trailing null padding and GUID brace removal is in utilities in
::DBI::ADO::CursorUtils which both ::DBI::ADO::MS_Jet (Access) and this
driver use, in their respective Cursor classes and in the drivers
themselves in select_single.

The cursor class is now autoloaded due to CAG component_class acc. group

Fix auto-pk for Sybase ASE, broken by refactoring in fabbd5cc

Was returning arrayref due to missing arrow, tests were passing because
arrayref numifies. Added a test that catches this.

Add all database connections via DBICTest::Schema to the leaktrace pool

Fix thinkos from bfec318f and 3edfebff

Disable ODBC batch ops for Access, they are broken

Some minor POD corrections

Apply very small doc correction to search_related
Fix incorrect last_modified / update comparison in ::Row/update
Add more direct documentation of grandparent methods in DBIC::Core

support FreeTDS 0.91 with Sybase ASE + DBD::Sybase

Requires disabling statement caching.

disable batch ops for firebird ODBC driver

make carp_once register messages not just callsite

Add a test as well in t/106dbic_carp.t .

Appease DBD::Sybase compiled with new freetds

As shitty as freetds is, it seems they at least got MARS right

Centralize (and privatize) handling of freetds driver/version detection

Multiple cleanups to accommodate broken FreeTDS driver

- Disable batch operations entirely (can get as bad as silent insert failures)
- Disable SELECT SCOPE_IDENTITY generation when used with dynamic cursors
- Clean some warnings while we're at it

Fix pessimization of offset-less Oracle limits, introduced in 6a6394f1

When there is only one RowNum operator, stability of the order is not relevant

Require bugfixed Module::Install

Remove all "magic number" DBI get_info calls from the codebase

Replace all use of wantarray tri-calls with explicit return or preserve_context

Passing test for mssql odbc bulk insert bug fixed by fabbd5cca9

Factor out IDENTITY_INSERT for Sybase ASE and MSSQL into a component

This not only greatly simplifies the code of multiple storages, but the
reduction of identity retrieval calls fixes multiple issues with some
odd driver combinations.

Run the sybase C subtest *after* the main test

Remove todoified tests reflecting a conceptual brainfart

Make sure do_query retries on potentially disconnected $dbh

Move Oracle blob tests into separate file (no functional changes!)

remove some Win32 workarounds in tests from cygwin

Cygwin can delete open files, so change some workarounds we had in tests
for closing open files before deleting them that were for both MSWin32
and cygwin to be for MSWin32 only.

base.pm now repeatedly require()s modules, exposing missing Exporter exemption

Add all storage instances to the test suite leaktracing pool

Fix leak of oracle storage objects in its _dbh_execute override

Replace with a much cleaner (and leak free) blockrunner instance

Fix the pure-perl in_global_destruction() emulation under threads

Also it seems that threads just don't work too well on < 5.8.5, ajust
the skip message to reflect this.

Make sure we load threads.pm before Test::More

removed unnecessary use of Test::Exception

ack -l Test::Exception t/ | xargs egrep -L 'lives_ok|dies_ok|throws_ok|lives_and'

Adjust tests to work correctly with the upcoming Test::Builder 1.005

Also deduplicate code in the persistence runners of t/52leaks.t

fix Win32 file use error in t/storage/reconnect.t

Even though the fake SQLite db file is chmod 0000, for some reason
SQLite still opens it on Win32 and holds it open; this prevents deleting
it and replacing it with the original db file so that the rest of the
tests can run.

TODO: followup with the DBD::SQLite folks.

Disconnect the $dbh on Win32 so the unlink can succeed, and remove the
following SKIP: block (for if db is "encrypted") which is no longer
necessary.

Update t/751msaccess.t broken by leak checker 65d35121

Can't have the cleanup sub close over $schema.

add SQLT dep if DBICTEST_SQLT_DEPLOY env is set

Add some code to Makefile.PL to check for $ENV{DBICTEST_SQLT_DEPLOY},
which if set requires SQL::Translator to be available or the tests
will fail. Add the dep as a test_requires directly, so that it is
scrubbed out of the META.yml in author mode due to not being in the
$test_requires hash.

making config flags actually recognize config files

Fix t/inflate/datetime.t warning on older perls

Fix failures with DBICTEST_SQLITE_USE_FILE=1 introduced by 50261284

Test suite wide leaktesting

Rewrite txn_do and dbh_do to use a (hidden for now) blockrunner

Fix more $schema leaks in the SQLT DBIC Parser (AUGHHHHH!!!!)

Add a TODO test for a very elaborate stack leak scenario

Fix fallout from a807d012 (folks please run your podtests when hacking POD)

- Stray =end in DBIC/DB.pm
- Inadvertently hidden documentation of txn_scope_guard
  (the =for comment never applied to the POD-block previously)

Minor messing with syntactically incorrect POD (no =headX blocks inside =begin, and =for is a one-paragraph block)

Simplify code of GenericSubQ limit dialect

Some cleanups and code dedup of Top and FetchFirst limit dialects

Relax requirement of having a primary key declared on table

Simplify RowNumber Over limit dialect implementation

Deduplicate code in rownum limit emulation

Use the RSRC unique constraint traversal to determine if an order_by is stable

Deduplicate code in RSC, use the RSRC unique constraint traversal instead

Add an internal unique colset finder and relax complex $rs update/delete code

As long as there is something to correlate the subquery by we are good

Remove the "row-by-row shift" Ordered.pm pessimization

The idea this is necessary was born when working with buggy old 1.1x
SQLite versions back in 2008. This no longer seems to be necessary,
but if I am wrong we can always bring it back with an actual
storage capability flag

Fix embarrassing leak triggered on calling new_related on an uninserted object

Reason is a typo-ed hash element in a weaken() call back in 68f3b0dd. Add a
bunch of extra tests to catch this, and also better validate the new_related
behavior.

Fix rogue child disconnecting a parent $dbh - no idea how I even missed that

Majorly cleanup $rs->update/delete (no $rs-aware code should be in ::Storages)

Overhaul of variable \[] handling in populate, and a lot of extra tests

Now things like multi-value \[ ' ? || ?', [...], [...] ] and populate
datasets where the values of the \[] change per row work correctly

Switch from using execute_array to execute_for_fetch directly

This saves us from pivoting our data into column slices, which execute_array
promptly turns back into our initial row-based format to feed to
execute_for_fetch. Apart from the obvious speed gain, this saves a lot of
memory since it avoids 2 copies of the (possibly rather large) dataset

Fix ASE bulk_insert for bind changes in 0e773352

When using the bulk API, update the binds to the structure expected by
_execute_array. Also temporarily disable $sth->finish in the bulk API
codepath because for some reason it rolls everything back. Modify the
test to always exercise both codepaths from now on.

Factor out bindattr resolver and tighten code a bit

Zero functional changes

Allow users to disable the "DT in search" warning introduced in e1038213

Shuffle things around, maint/ is now a proper toolchain tooldir

Normalize -'s and _'s in maint names and Makefile options

remove stale link and doc readonly pool queries

transactions cause an implicit force to master, this documents how you can
override this to force to pool

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

add a link to SQL::Abstract for WHERE clauses

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

Fix forgotten whitespace - this is what I get for no prove xt

Only invoke the author Makefile.PL includes if we are not part of a `make` run

Comment typo

Forgotten diag and some clarifications of fc4b0448

Factor out the Makefile.PL into easily digestable snippets

Also add explicit guard for attempts to `make dist` from within non-author mode.
The rationale is to disallow dist building on machines that do not have the optdeps
installed (this still does not giarantee `make test` will run, but is at least a
nudge in the right direction)

Assert that complex bindtytyping passage works for the search() bind attribute

use more correct subsection links in POD

remove ::Storage::DBI::sth from POD as it should never have been public