Changes for 113e8d16 and 64c50e81

Prevent DBD::Firebird and DBD::InterBase from colliding on same DSN

Manpages are of no use on travis

Do one poisoned env testrun with a threaded perl

Only smoke blead-pan on allowed-to-fail runs

Move bleadperl to 'fail ok' submatrix - too much churn

Work around https://rt.cpan.org/Ticket/Display.html?id=88222

Temporary work around yet another cpanm issue

Use the same mirror for cpanm as we use for CPAN

Switch travis-perl tests to 5.18

I did manage to screw up locking in parallel testing after all

We try to grab a new engine lock within the same test before releasing
the one for the previous engine

Adjust DBD::Firebird travis init

Sync Example.pod with reality, though in fairness it just needs to die...

Automatically regenerate the example ddl/db from the example Schema

Generalize the SQLite DDL generation tool (see next commit)

Update example Result classes for proper DDL generation

Also given that this is one of the first things a newbie sees, it is prudent
to stick to canonical definition

Teach the example scripts how to find their DB from anywhere

These comments are woefully outdated

Setting DBIC_TRACE=1 won't help either - we use the fast-path insert which
mean the user won't see anything

Standardize examples/docs on Schema::Loader schema naming

A new user is very likely to encounter this naming convention these days

New podlators are allergic to leading link space

Extra paragraph dropped from 113e8d16 for some reason

Fixup weird wording introduced in 54e4be92

Apparently on_failure email flag affects the original author as well

Without this only the first committer who broke stuff will get an email
the rest of the breakage will remain silent

Fix name mangle in Replicated.pm broken by e8c31aec and then by 4a0eed52

OCD - undo some incorrect changes of non-typos from 4a0eed52f

typo fixes

Augment 84d8c2ad so that the custom join specification isn't so prominent

Also add a warning to (hopefully) deter folks from using these for simple
shit, where a schema redesign would suffice.

perlbrew exits with 0 if `perlbrew use` fails :(

Ignore stderr in extract_prereqs - only rely on $?

Install EU::CB in a separate step as to not tickle an old CPAN bug

Older CPAN.pm had a problem with the circular dependency resolution firing
erroneously *after* everything was successfully installed, e.g.:

https://s3.amazonaws.com/archive.travis-ci.org/jobs/9336470/log.txt

(scroll to the end)

Fix test error failing after "undef\n=>" was fixed to stringify

http://perl5.git.perl.org/perl.git/commit/21791330af

Temporary inslatt Pod::POM via cpan

Pod::POM installation tickles a toolchain deficiency
https://rt.cpan.org/Ticket/Display.html?id=87124
https://github.com/miyagawa/cpanminus/issues/291

Allow spaces in parallel installation module spec

Turns out Travis CPAN mirror is not local, add a fallback

See https://github.com/travis-ci/travis-cookbooks/commit/143a57d2ecee#diff-1

Sideffect of an empty PERL_CPANM_OPT was tickling this bug:
https://github.com/miyagawa/cpanminus/issues/288

Self-cleaning is not yet entirely stable - back it out for the time being

As a side-effect resolves RT#86267

Tests no longer fail if $ENV{DBI_DSN} is set

e.g. http://www.cpantesters.org/cpan/report/ef15dfbc-b80d-11e2-a6f8-e33723969de0

Propagate quote_names setting to SQLite SQLT producer

Fix POD typo Retur -> Return

Fix glossary link

point out where in the docs a user is most likely to spend reading time

quick-start documentation

* ready-to-use database file and schema classes
* introduce CD year column (already used in ::ResultSet docs)
* link from main synopsis

Don't try to run pperl --kill if we didn't find PPerl

There might be an unrelated pperl binary in the user's path, so avoid
running it unless we actually have PPerl.pm installed

Add header for custom coderef join conditions

The conditon docs are huge, making it hard to point people to the
coderef syntax when they need it, so add a header that can be linked
directly to.

Replace $row with $result in all docs

to be consistent and to clarify various return values
for example:
    $result->update() returns $result
before the example was
    $row->update() returns $result
which doesn't make clear that it's the same object the method was called on

fixed xt/pod.t failure in Row.pm

Release v0.08250

This stuff is done (either now or earlier)

Kill intra-devrel changelog entry

The log is large enough as it is, hence keeping only things that
are relevant when upgrading from 0.08210 to current

devrel 0.08249_05

Do not invoke the pk accessor in filter deflate

Stop adding empty caches to resultsets chained off prefetched objects

Prior to ce55688, the related_rs cache loading mechanism only conditionally
loaded if the base rs cache was defined. After this was removed an empty
arrayref (meaning "no related results") could potentially be set as the cache.
This commit ensures the cache is only set when there are actual cache results.

fix syntax error in pod documentation for find_or_new

Devrel 0.08249_04

Unbreak devrel version checks

One needs to supply the literal (string) value of a devrel to VERSION()
otherwise the checks fail. However in this case 5.8 emits annoying warnings.
Abstract the whole insanity in a neatish util function and use that.

Stop instantiating a cursor on single() use

This minor pessimization snuck in all the way back in 4e9fc3f3.
I am actually not entirely sure whether this isn't a bug - after
all this way Cursor::Cached and friends are bypassed. However
0.08210 has an identical behavior, thus sod it

Fix very subtle but deadly bug in single()

The _select_args result storage introduced in 975b573a did not take
into account that single() operates on the $rs itself, not a fresh
one. As such the first single() arg would be carried around and then
applied to subsequent next() etc iterations. Stop that, as we only
need the attrs in case of collapse => 1 and single() explicitly does
not support this.

While at it do not reuse the cached args at all - while it *appears*
safe, it may very well not be. Leave this for another day

Require bugfixed (and a tad faster) CAG

Bump minimum Test::Pod Dependency

In 47589465 we started feeding dirs to all_pod_files_ok - only Test::Pod 1.42
understands that

... and devrel 0.08249_03

Devrel 0.08249_02

Make some travis confs smoke latest -dev offerings from CPAN

Based on the Test::Builder dev-testing introduced in 992a24f6

It seems the upcoming DBD::SQLite fixes the txn sync issues

Short circuit the travesty introduced in 2aeb3c7f

Fix test failures with DBD::SQLite 1.38_02

This is what fb88ca2c was supposed to fix in the first place >.<

Silence pesky Devel::Cycle warning in leak test on 5.12+

And of course the test pragma is not supported everywhere

Similar issue but no solution: http://osdir.com/ml/sqlite-users/2010-01/msg00220.html
Experimentally determined cutoff at 3.7.9

Devrel 0.08249_01

Fix tests failing due to unspecified resultset retrieval order

An upcoming version of DBD::SQLite bundles an SQLite library
with different behavior wrt retrieval of unordered indexed columns.

Fix all tests that could be affected now or in the future, and
add travis smoking to detect the condition early

deployment_statements() and cursor_class() are storage-dependent

We need some sort of generic test for this, so we stop dropping stuff
on the floor. The logic is something like "if a public method is overriden
in a DBI:: subclass - we need to mark it for resolution". E.g.

rabbit@Thesaurus:~/devel/dbic/dbgit$ grep deployment_statements -r lib/DBIx/Class/Storage/DBI/
lib/DBIx/Class/Storage/DBI/Pg.pm:sub deployment_statements {
lib/DBIx/Class/Storage/DBI/Replicated.pm:    deployment_statements
lib/DBIx/Class/Storage/DBI/mysql.pm:sub deployment_statements {
lib/DBIx/Class/Storage/DBI/SQLite.pm:sub deployment_statements {
lib/DBIx/Class/Storage/DBI/Oracle/Generic.pm:sub deployment_statements {

Cleanup logic of _determine_driver - do not fire unless we have conninfo

Also switch some connection init warnings  s/carp/carp_unique/

More robust tests of dependency lazy-loading and delay of more req loads

Inspired by the lazy-loading chase in 723f25e0

Skip the test entirely on 5.8 - it is becoming too difficult to predict
extra dependency load order

Escape more frames when determining carp/throw_exception originator

Augments 5e0e5426b

Tweak timing of t/zzzzzzz_perl_perf_bug.t - we started getting false positives

http://www.cpantesters.org/cpan/report/356dd340-9de0-11e2-9265-e31b922ed931
http://www.cpantesters.org/cpan/report/2e2063f1-6bf4-1014-817e-b9974151da38

Not considering dropping the test because... well it is a good test ;)

Test live mssql server generated SQL

Clarify how to use SQL functions as part of a larger search

Removed usage of plain_value bind value workaround in the cookbook

Support for $val === [ {}, $val ] in literal SQL + bind specs

This wraps up the changes started in 0e773352. Now DBIC bind values can
be specified just like the ones for SQL::Abstract as long as no bind
metadata (e.g. datatypes) is needed

Also added an explicit check to catch when a non-scalar non-stringifiable
value is passed without a bind type metadata

Merge branch 'topic/constructor_rewrite' into master

Consolidate changelog from the 0.0824x-TRIAL cycle

Add (slowish) sanity check to detect incorrect non-nullable metadata

We can't get away with this - the alternative is a cryptic 'Use of uninitialized value'
exception - good luck debugging that

Explicitly disallow redirection of has_many columns to main object

This can never work in practice, but it is too obscure to have a user catch it

Store the final calculated select args in the original $rs

This is extremely ugly and ineffective, but we need it for some sanity checks
so that 0.08250 can ship. The whole thing needs to be reworked with a new
rsattrs instance, but that's another battle

Misc cleanup, no functional changes

Cut down amount of calls to _resolve_aliastypes_from_select_args

Make 'filter' rels work half-way sanely with partial prefetch

Another blast from the past - fix distinct/order behavior borked by d59eba65f

While the SQL we test for is valid syntax - the result it produces makes
zero sense for these resultset arguments. Reshuffle the distinct construction
and reuse the codepah for order_by rewrites.

This is a heavy change, but it is *extremely unlikely* anyone was relying on
the previous behavior - any aggregate value would have been wrong...

Fix "current rs is always first in from" brainfart introduced in 893403c81

Dear self from 4 years ago - what were you thinking? In any
case I can't recognize why this is needed so cleansing it with fire

Variation of dq's d35a4e048 - sanify multiplying join prune

Use a private more descriptive attr name

Factor out ORDER BY direction detection/stripping

The slightly unobvious regex is so that it's easier to add support for
more clauses (e.g. NULLS FIRST/LAST) later.

Overhaul GenericSubq limit - add support for multicol order

A lot of other cleanup as well - this should be the end of it ;)

Start running more limit dialect torture tests if possible

Recognize root source unqualified columns under limited prefetch

Consider where condition when determining leftmost order stability

Extract more eyebleed into DBIHacks in the process

Use the defined-or operator in generated code on newer perls

Even *less* expensive op ;)

space-compress tests making them readable - no changes

Another parser microoptimisation - lose nullability checks where possible

Stop needlesly invoking the $rs bool overload

Reorder prefetch logic for optimal execution (no functional changes)

Avoid yet another expensive try call

Normalize varnames (zero functional changes)

Fix test description (DrHyde++)

Merge branch 'master' into topic/constructor_rewrite

Add some extra code to enforce the assumption that any bind type constant
is accessible in _dbi_attrs_for_bind, or in other words that all necessary
DBDs are already loaded (concept originally introduced in ad7c50fc)

Without this the combination of 9930caaf7e (do not recalculate bind attrs
on dbh_do retry) and a2f228547 (do not wrap iterators in dbh_do) can result
in _dbi_attrs_for_bind being called before DBI/DBD::* has been loaded at all

Remove idiotic RowCountOrGenericSubQ - it will never work as part of as_query

I am not sure how I overlooked this, blame is on me :(

Replacing with a ROWCOUNT fallback at the appropriate place