Use explicit variable on loop

Fix and guard against erroneous use of list context in internal DBIC code

This situation is dangerous in case the end-user employs something like
DBIx::Class::Helper::ResultSet::IgnoreWantarray

Besides the trivial fix in ::Row::copy, this commit introduces the
DBIC_ASSERT_NO_INTERNAL_WANTARRAY=1 mechanism (which will be
automatically picked up and engaged by our CI due to eed5492f). As a
bonus the check is usable on DBIC dependencies as well.

Fail early properly even in the presence of -jN

Without the reordering of prints and without the mysterious sleep() call it
seems that prove just loses the STDERR text entirely

fix pod formatting of this code snippet

Stop using the deprecated Class::MOP::load_class()

Use MooseX::Types::LoadableClass instead, requiring the version that
fixes the constraints to not throw an exception on syntactically
invalid names, so that we can coerce a prefix onto ::Foo.

Further tradeoffs on relationship sanity checking (augments 9e7525a2)

It turns out what we did earlier is not enough - what is truly needed is
shutting off *all* checks on non-belongs_to rels. Otherwise industrious devs
will inevitable tie themselves into a circual loading order knot.

The problem (as in 9e7525a2) is not so much that we need to bend over
backwards to support circular loads, but that these problems could very
well be masked by stable hash ordering of perl < 5.18. In other words this
commit serves to prevent perfectly working code from suffering intermittent
and hard-to-debug issues when someone upgrades their perl in 2018.

Fail early if full test is run in a stale checkout

Proper fix for t/storage/replicated.t (augments 617474ab7e)

Instead of blindly changing the use_ok to use figured out and moved together
the pieces responsible for the dance in the 1st place

Fix t/storage/replicated.t class loading problem

Do all the skip_all in BEGIN {}, and just 'use' the modules instead of
use_ok().

Throw clearer exception on ->new_related() with a non-existent relationship

Smoker-wide parallel testing of installed deps

Better diagnostics of skipped non-essential tests

Engage nuclear option of travis testing

For the record - I am not happy about having to make this commit. However the
reality is that for every basic toolchain module I fix, another two are broken
by dzil-addicted (though well meaning) folks. Since this essentially paralyzes
any useful work, resort to simply ignoring some of the meaningless tests that
are autopopulated by ill-designed dzil author bundles.

This allows to revert 655d80dd, 72cd4007, and generally streamline the set of
workarounds

A preemptive note for anyone who is concerned that I am wasting precious tuits
on a dubious goal. Please realize that one of the corner stones of my personal
desire to participate in OSS comes from disdain of forced obsolescence (i.e.
one shall upgrade each and every year because the author feels like it). If I
can not guarantee my software performs correctly on old but usable versions of
our beloved VM - a massive portion of the very reason I "keep playing" goes
away entirely. With that gone the world beyond OSS suddenly becomes
exceedingly more appealing, at least much more appealing as opposed to pushing
features into a library unusable by most real-world production environments.
Additionally whether I or anyone else has encountered users benefitting from
my efforts is not relevant - the DarkPan universe is vast.

In other words instead of seeing these activities as a mindless fetish,  see
them for what they are: one of the main reasons I am a CPAN contributor to
begin with.

--ribasushi

Delist Example and add QuickStart to the DocMap

Pending my review of PR#35

Update DocMap - include missing local Manual pages (PR#22)

some of the local Manual pages are not mentioned in the DocMap which should be,
as people will use the DocMap as a stepping point, and may thus overlook them

A fixed DBD::Firebird went to CPAN as well

These workarounds are thankfully no longer needed

Loading strictures will only complain about missing optdeps if used
within a t/, and dbic is (and will remain, sorry Matt) structures-free
deliberately, so there you have it

This was fixed in March 2013

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