merge comments

TO_MERGE notes file added

warn when redefining relationships (RT#73876)

Add a carp when a relationship is being redefined and DBIC_TRACE is on.

Fix filename interpolation in DBIC::Optional::Dependecies

Ignore Emacs droppings and generated README

release 0.08200

put off deprecation and clarify test output

rewrite test to use existing schema

release 0.08199

Dev release changes

Back out constructor/prefetch rewrite introduced mainly by 43245ada4a

It was shipped against the authors advice, while containing multiple known
bugs. After the expected bugreports went warnocked for over two weeks by the
new DBIC release team, it seems that the only way to partially restore the
release quality DBIC users have come to expect, is to currently throw this
code away until better times.

Should resolve RT#78456 and the issues reported in these threads:
http://lists.scsys.co.uk/pipermail/dbix-class/2012-July/010681.html
http://lists.scsys.co.uk/pipermail/dbix-class/2012-July/010682.html

failing test for prefetch bug where prefetched values of 0 might break inflation

remove ->deploy from castaway's test

New test for recent prefetch/undef bug

 Fix Class::AccessorGroup to allow instances in get_component_class.
 Fixes result_class($obj) exploding instead of just using the object.

release v0.08198

fix silent Oracle connection failures

::Storage::DBI::_server_info was eating connection exceptions by
ignoring exceptions from $dbh->get_info, but this information is
important for _rebless when e.g. determining which class to use based on
the DB server version, so rethrow the exceptions when
$self->{_in_determine_driver} is true.

better type check for mssql+ado binary null strip

The data_type in the t/lib result class for the blob for the tests
changed to 'blob', while the binary type null stripper only expected
'binary' or 'image' data types, which is wrong for possibly
cross-deployed schemas.

Change _strip_trailing_binary_nulls in ::ADO::CursorUtils to use the
newly added and more generic ::Storage::DBI::_is_binary_type instead of
just checking for 'binary' or 'image'.

mark some tests in admin/10script.t on Win32 TODO

The test_dbicadmin() tests are failing on Win32 due to some sort of
quoting issue with the temp directory. I'm not sure how the temp
directory is being used here, so it will have to wait for someone else,
in the meanwhile I marked the tests TODO.

Also changed the " backwhacking to use Win32::ShellQuote instead and
added it to optdeps, unfortunately that didn't fix the problem, since
the problem seems to be some sort of DBICTest::RunMode interaction with
dbicadmin.

preload threads.pm in storage/txn.t for fork error

SineSwiper reported this error in t/storage/txn.t on Strawberry Perl
5.12 32bit:

You must load the 'threads' module before Moo::_Utils at t/storage/txn.t
line 130.

Preload threads in a BEGIN block at the beginning of the .t file for
Win32. This fixes the error.

skip 51threadnodb.t on Win32 on < 5.14

SineSwiper reported a crash with a dialog from "make test" in
t/51threadnodb.t on Strawberry Perl 5.12 32bit.

The test passes on my ActivePerl 64bit 5.14.2 and there is a passing
test report for 5.14 here:
http://www.cpantesters.org/cpan/report/239d2ae4-87c3-1015-ad08-a09405085451

Disable test on Win32 for perls below 5.14 unless the environment
variable TEST_AUTHOR is set to be safe.

release 0.08197

no longer dep on dev rel since this is not a dev rel

fix pod for unicode name

Add Travis YML for Travis-CI support
   (See http://travis-ci.org/#!/SineSwiper/dbix-class/jobs/1665376)

fix broken syntax

Add authority for new moules

fix tests on solaris

fix debian mssql/odbc setup docs for multiarch

release 0.08197_01

fix for RT#77241

fix typo

add more docs on software_limit resultset attr

Add a paragraph to Storage/DBI/Sybase/ASE and a section under SEARCHING
to Cookbook.

update proxy/cascade_update doc in Relationship::Base

Remove mention of obsolete DBIx::Class::WebForm.

Mention HTML::FormHandler::Model::DBIC instead, which is actively
maintained.

Remove/fix bogus defaults values in example

is_auto_increment already provides a default, and for the other columns
we either don't want a default value or it needs to be an integer.
Explicit is_auto_increment => 0 is also pointless and distracting.

Adjust tests for new Moo

Simplify untainting - Module::Find can't return invalid names

Use the now pure-perlized Devel::GlobalDestroy module

Fix more fallout from 9b871b00, centralize the ugly in a base Schema class

Remove realiasing overengineering introduced in 86bb5a27da

mst is right - there is no viable use case for this, cleanse with fire
Also I managed to misspell subUery... twice

More limit torture

Fix lapse on limited incomplete has_many prefetches

Do not artificially order the internals of a has_many prefetch subquery

Thre is no point using the artificial order which is only needed for efficient
collapse. The user didn't request an order - this means any rows will do

Saner naming for helper test method

Fix test failure on MSWin32

Make sure order realiasing remains in proper sequence on sorting

We now support 999 realiased order criteria, this is beyond sufficient

Fix test borkage after 9b871b00c - ALWAYS RUN TESTS RETARDO!

Better handling of bogus tmpdir returned by File::Spec on MSWin32

add missing catfile so tests can run on win32

Version/dist Makefile.PL safety checks

Add strictures to the Makefile.PL inc-snippets

This is stupid - no need to build the SQL manually, the local flag is enough

No join pruning on a SINGLE TABLE FROM CLAUSE

Merge branch 'topic/constructor_rewrite'

Minor cleanups in ResultSet.pm (no functional changes)

Cleanup of stale constructor codepath comments

I think we are done here

Move scary stuff to its own class

Initial full test pass - all fetches are eager for now

Merge branch 'master' into topic/constructor_rewrite

Add torture of limiting subselect with non-root table selection renamer

me.minyear is not a valid alias, minyear is

Entire test suite now executable under tainted perl (prove -lT)

Merge massive rework of limit dialects

Document software_limit and properly throw on related as_query calls

Extra tests for all the improvements

Take into account resultset conditions when determining rs order stability

This takes care of the overly-aggressive exception scenario introduced by
86bb5a27

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.