switch from Class::MOP::class_of to Moose::Util::find_meta

Augment test suite with oddball relationship declarations

Among other things this tests for the issue described/solved in 9e7525a2

Reorder relationship sanity checks, run them only on known loaded classes

This avoids load-order issues in contrived situations where column definitions are
split across class inheritance (which gets rather fun doe to the prorotypical
inheritance of resultsource instances themselves)

While not an immediate problem in code in the wild, the hash randomization in 5.18
now leads to random load order, and as a result more load-order combinations are
tried and can eventually fail. This should reduce these situations to a minimum,
and 79061be1 should keep us appraised of any problems while DBICT:Schema evolves

Ideally the checks will be lifted up to schema-composition time, but we are not
there yet, one yak at a time.

Consolidate single-pk checks from relationship inferrence codepaths

More unreachable code - we've already thrown on !defined $pri

typo fix

MOAR workarounds - freeze CMM at a usable version

The dzil conversion introduced dependency on MBT which is extremely
hard to get going on older perls, left alone all dists made with it

Hopefully it will get fixed:
https://rt.cpan.org/Ticket/Display.html?id=88641

Disable perl introduced in ced296a8 until fixed Carp.pm is shipped

https://rt.cpan.org/Public/Bug/Display.html?id=88531

Run travis-prove under taint mode

Fix 79061be1 not working under -T

Excise unreachable code

Rename internal rsrc method to be more descriptive, stop proxying it

Make sure we can load each DBICTest::Schema result individually

This will ensure tests always fail if some sort of circular load order
issue is encountered, and we won't have to wait for a particular random
loading sequence to blow up the tests

Reorder travis tests by length, add 5.12.4

Only run our own tests concurrently, not stuff in after_success

Check that each preinstall on an old CPAN.pm completes

EU::MM on < 5.8.9 got fixed

Change CPAN mirror again - metacpan is a complete backpan-backed mirror

This way the deletion of the usable version of Tree::DAG_Node won't
affect testing

Lock down Tree::DAG_Node version until new Test::Warn appears

This will make travis green again

https://github.com/chorny/test-warn/pull/4
(also https://rt.cpan.org/Public/Bug/Display.html?id=88435)

Fix typoed variable (just remote the shorthand entirely) ^Simon^++

More typo fixes

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