10 years ago(travis) Attempt to grab more info on parallel_installdeps_notest failures
Peter Rabbitson [Wed, 24 Sep 2014 04:28:06 +0000]
(travis) Attempt to grab more info on parallel_installdeps_notest failures

10 years agoNew Pod::Inherit does not need Pod::Tree
Peter Rabbitson [Fri, 14 Dec 2012 08:28:44 +0000]
New Pod::Inherit does not need Pod::Tree

10 years agoClarify licensing, ensure footers are consistent throughout the project
Peter Rabbitson [Tue, 23 Sep 2014 07:04:32 +0000]
Clarify licensing, ensure footers are consistent throughout the project

Note - this *DOES NOT CHANGE* the licensing of the project in any way, it
is still dual-licensed like perl5, just as it was before this commit. All
additions are for clarifying purposes only.

The way licensing is presented is an amalgamation of information and
prior art found at the following sources:
 - http://www.perlfoundation.org/cpan_licensing_guidelines
 - https://metacpan.org/source/ETHER/Moose-2.1212/LICENSE
 - `git log --color -w -p v5.20.0 ^perl-5.000^ -- Artistic`

10 years agoDeal with authorship properly, in a future-sustainable fashion
Peter Rabbitson [Mon, 22 Sep 2014 10:35:38 +0000]
Deal with authorship properly, in a future-sustainable fashion

This module is arguably one of the most successful collaborative projects on
CPAN. Make it known to the world by introducing a clear canonical source of
authorship metadata.

I have been holding off jumping on the "contributors from git" bandwagon
because of multiple issues

- Not every contribution results in a direct, attributtable commit
- We already had an extensive contributor list, much larger than the shortlog
- Loss of ircnicks (a fine tradition started in 266bdcc3)

So this commit does the proverbial bullet-biting and introduces *proper*
handling of all the ins and outs, including author-mode (CI as well) tests
for proper attribution. As a side-effect we break a lot of the footer links,
to be fixed in next commit.

Btw as I write this `195 contributors made this library what it is today`...
Mind. Blown.

10 years agoRe-attribute a substantial chunk of docs, due to lost original author
Peter Rabbitson [Sun, 21 Sep 2014 03:59:38 +0000]
Re-attribute a substantial chunk of docs, due to lost original author

The mysterious <g> is seen on the channel between Aug 24 and Sep 3 of 2008
talking to castaway and various other hackers, supplying a number of doc
patches and then vanishes. Due to the need of correct authorship imposed
by the next commit, reattribute the docs to castaway instead

This is the only such instance - all other authors have been accounted for.
Deemed acceptable as the name is entirely and utterly lost in the ether :(

10 years agoInitial AUTHORS stub (the last one where `git blame -C -C AUTHORS` worked)
Peter Rabbitson [Mon, 22 Sep 2014 18:35:15 +0000]
Initial AUTHORS stub (the last one where `git blame -C -C AUTHORS` worked)

Due to how git currently works one will need to `git log -S"line"` in the
future in order to find when a contributor entry was really made. Preserving
this particular arrangement historically, before more reordering and additions
come in the next commit.

10 years agoThe revert 8494142c incorrectly killed 4d93345c, reinstating
Peter Rabbitson [Wed, 24 Sep 2014 03:15:54 +0000]
The revert 8494142c incorrectly killed 4d93345c, reinstating

The loading of IO::Handle is still needed as demonstrated by failing
SQLA optional tests. Bleh!

10 years agoViciously deal with more strictures fallout
Peter Rabbitson [Wed, 24 Sep 2014 00:22:48 +0000]
Viciously deal with more strictures fallout

The 'what can possibly go wrong' blew up with RT#99083, there is definitely
more where this came from. Try to maintain safety by going to even greater
lengths ensuring we do not load any of it ourselves

10 years ago(travis) Bump tested perls a bit
Peter Rabbitson [Tue, 23 Sep 2014 23:29:46 +0000]
(travis) Bump tested perls a bit

10 years ago(travis) reload the already running RDBMS, as a precaution
Peter Rabbitson [Tue, 23 Sep 2014 23:25:53 +0000]
(travis) reload the already running RDBMS, as a precaution

Also get a newer libmysqlclient, just in case...

10 years agoReorganize dist-time meta handling (no semantic changes)
Peter Rabbitson [Mon, 22 Sep 2014 08:49:38 +0000]
Reorganize dist-time meta handling (no semantic changes)

10 years agoA massive amount of link fixes (just links, almost no rewording)
Peter Rabbitson [Sun, 21 Sep 2014 18:00:27 +0000]
A massive amount of link fixes (just links, almost no rewording)

kryde++ for the insanely useful tool App::PodLinkCheck. It is a bit heavy
and way too noisy for automated use, but with some handholding it is
scarily effective (as can be seen by the sheer size of this commit)

10 years agoProper end-of-file for DBIx/Class.pm
Peter Rabbitson [Sun, 21 Sep 2014 01:23:38 +0000]
Proper end-of-file for DBIx/Class.pm

10 years agoAdd forgotten bit preventing scan of where-binds (originally part of 1b8f2dd9)
Matt S Trout [Mon, 7 Oct 2013 11:40:53 +0000]
Add forgotten bit preventing scan of where-binds (originally part of 1b8f2dd9)

10 years agoAdd basic .dir-locals.el
Dagfinn Ilmari Mannsåker [Sat, 20 Sep 2014 12:24:00 +0000]
Add basic .dir-locals.el

10 years ago(travis) Attempt to revert 86370cc74, SC is too slow on uptake. Grumble
Peter Rabbitson [Fri, 19 Sep 2014 17:43:57 +0000]
(travis) Attempt to revert 86370cc74, SC is too slow on uptake. Grumble

10 years agoProperly handle empty group_by/order_by
Peter Rabbitson [Fri, 19 Sep 2014 08:32:10 +0000]
Properly handle empty group_by/order_by

10 years agoImprove SQLT::Parser warning messages (build further upon e089c417)
Peter Rabbitson [Thu, 18 Sep 2014 13:43:30 +0000]
Improve SQLT::Parser warning messages (build further upon e089c417)

10 years agoProperly implement very sloppy and lazy hack from bad0b73b
Peter Rabbitson [Wed, 17 Sep 2014 17:36:45 +0000]
Properly implement very sloppy and lazy hack from bad0b73b

We are abusing local() here - might as well abuse it correctly.
This way we do not add a permanent shitty name to the relinfo structure and
nobody needs to fix their tests. Winning!

10 years agoRevert e9f71ab2 - it ends up breaking a declared API
Peter Rabbitson [Wed, 17 Sep 2014 16:58:29 +0000]
Revert e9f71ab2 - it ends up breaking a declared API

I should have thought of this earlier - a debug object is not necessarily
isa(IO::Handle) (the documentation never mentioned this). And indeed: the
tests of DBIx::Class::QueryLog do exactly this - they use an object with
just a print() method and absolutely nothing else.

So instead of disabling the sticky autoflush (which really is only a visible
change on STDERR dup) document this behavior and move on.

10 years agoFix broken temporary backcompat shim for naughty _resolve_condition callers
Peter Rabbitson [Tue, 16 Sep 2014 16:24:25 +0000]
Fix broken temporary backcompat shim for naughty _resolve_condition callers

The 9b96f3d41 refactor undid all the backcompat shims that were introduced
in c0f445097. Reinstate the insanity until after 28...

10 years agoMultilevel find_or_(multi)create got inadvertently broken
Peter Rabbitson [Tue, 16 Sep 2014 08:04:59 +0000]
Multilevel find_or_(multi)create got inadvertently broken

Relax the check of what we feed to the relcond resolver

10 years agoYet another loose end on the cond collapser
Peter Rabbitson [Tue, 16 Sep 2014 06:01:11 +0000]
Yet another loose end on the cond collapser

Fingercross this is the last one... this transform is turning out ot be a
bad bad bad idea :(((

10 years agoEnsure the tempextlib is available at distbuild time
Peter Rabbitson [Mon, 15 Sep 2014 13:32:28 +0000]
Ensure the tempextlib is available at distbuild time

Missed one spot to load ::_Util during bbcc1fe8

10 years agoPut in place deprecation forgotten for several years
Peter Rabbitson [Mon, 15 Sep 2014 06:22:24 +0000]
Put in place deprecation forgotten for several years

More info and rationale in the commit msg of c200d949

10 years agoSplit DBIC from SQLMaker test (deprecated in next commit)
Peter Rabbitson [Mon, 15 Sep 2014 06:11:55 +0000]
Split DBIC from SQLMaker test (deprecated in next commit)

10 years agoEnsure the custom rel cond resolver does not trigger forgotten compat shim
Peter Rabbitson [Sun, 14 Sep 2014 18:59:58 +0000]
Ensure the custom rel cond resolver does not trigger forgotten compat shim

During the rush to get custom rels out the door (this is why rushing
fucking sucks), a697fa31 introduced a shortsighted workaround into
::SQLMaker::_from_chunk_to_sql(). This code slipped consequent review
and made its way into the codebase... 4 FUCKING YEARS AGO!!! >:(

Since it is not known how much stuff relies on the insanity being there
(moreover we have tests that rely on it) leave things as is for the time
being. The only change is  making the cond resolver *completely* oblivious to
the "single-element hash" workaround (albeit via a silly hack).

In the process exposed that ora-joins module is entirely incapable
of understanding non-equality conds... fml

See next commits for added warnings, etc.

10 years agoA more robust column equality extractor
Peter Rabbitson [Sun, 14 Sep 2014 18:41:55 +0000]
A more robust column equality extractor

It is a mistake to reach into $ret->{condition} - it may very well be a
non-hash structure. Also replace the silly regexping with the (already
regexed out) results of the info resolver

10 years agoEnsure undef_on_null_fk does not affect non-introspectable custom conds
Peter Rabbitson [Mon, 15 Sep 2014 09:39:12 +0000]
Ensure undef_on_null_fk does not affect non-introspectable custom conds

10 years agoYet another missed sanity check in the relcond resolver
Peter Rabbitson [Mon, 15 Sep 2014 08:40:39 +0000]
Yet another missed sanity check in the relcond resolver

rel/col duality is just hateful

10 years agoMake sure the me-relname-flip does not affect any error text
Peter Rabbitson [Sun, 14 Sep 2014 18:55:55 +0000]
Make sure the me-relname-flip does not affect any error text

10 years agoFix copy() assuming all columns are native
Peter Rabbitson [Mon, 15 Sep 2014 09:55:53 +0000]
Fix copy() assuming all columns are native

10 years agoAdjust for SQLA's API change 966200cc8
Peter Rabbitson [Sun, 14 Sep 2014 18:00:40 +0000]
Adjust for SQLA's API change 966200cc8

10 years ago(travis) Accommodate a slow mirror grrrrr
Peter Rabbitson [Mon, 15 Sep 2014 11:42:00 +0000]
(travis) Accommodate a slow mirror grrrrr

10 years ago(travis) Stop auto-upgrading everything on non-clean installs
Peter Rabbitson [Mon, 15 Sep 2014 11:58:48 +0000]
(travis) Stop auto-upgrading everything on non-clean installs

It adds little value currently and makes version-specific installs difficult
(see next commit)

10 years agoFix syntax error in ::Manual::Joining example
Tommy Butler [Wed, 10 Sep 2014 08:51:27 +0000]
Fix syntax error in ::Manual::Joining example

fixed syntax error on line 245 (missing closing curly bracket)

Closes: #60

10 years agoFix incorrect cond construction in _minimal_valueset_satisfying_constraint
Peter Rabbitson [Tue, 9 Sep 2014 23:38:43 +0000]
Fix incorrect cond construction in _minimal_valueset_satisfying_constraint

The function in question introduced in d681f1bb (which builds upon work in
8e40a627) correctly uses _extract_fixed_condition_columns, but then fails to
account for all literals having their leading { '=' => ... } being stripped,
nor does it consider UNRESOLVABLE_CONDITION as a valid return value.

Tests and fixes to get this rolling, thanks go to Lianna Eeftinck for testing
and reporting \o/

10 years agoEnsure ::Schema::Versioned connects only once by reusing the main connection
Peter Rabbitson [Tue, 9 Sep 2014 23:48:35 +0000]
Ensure ::Schema::Versioned connects only once by reusing the main connection

10 years agoStop using Sys::SigAction in pg tests
Peter Rabbitson [Mon, 8 Sep 2014 11:06:59 +0000]
Stop using Sys::SigAction in pg tests

10 years agoFix for gut-reaching travesty as reported by gbjk
Peter Rabbitson [Mon, 8 Sep 2014 09:14:29 +0000]
Fix for gut-reaching travesty as reported by gbjk

10 years agoNo longer use rel_info($rel)->{class} in the cond resolver
Peter Rabbitson [Sun, 7 Sep 2014 11:20:18 +0000]
No longer use rel_info($rel)->{class} in the cond resolver

It turns out there are a lot of codebases there containing garbage in the
rel definition. Punt for after 0.082800 to lean up that mess.

Also be less strict on checking the foreign_values contents - downgrade
mismatches to a warning (but still hard-require ::Row ancestry)

10 years agoSimplify self_result_object argument check in cond resolver
Peter Rabbitson [Sun, 7 Sep 2014 11:11:30 +0000]
Simplify self_result_object argument check in cond resolver

10 years agoMinor relcond resolver refactor, no func. changes
Peter Rabbitson [Sun, 7 Sep 2014 11:04:04 +0000]
Minor relcond resolver refactor, no func. changes

10 years agoSkip malformed relationships in SQLT parser instead of vague warnings
Peter Rabbitson [Fri, 5 Sep 2014 09:14:36 +0000]
Skip malformed relationships in SQLT parser instead of vague warnings

10 years agoMove and simplify a chunk of parser code
Peter Rabbitson [Fri, 5 Sep 2014 09:05:46 +0000]
Move and simplify a chunk of parser code

No functional changes

10 years agoMake sure IO::Handle is loaded - missing stubs on older perls
Peter Rabbitson [Fri, 5 Sep 2014 08:33:40 +0000]
Make sure IO::Handle is loaded - missing stubs on older perls

Also adjust the documentation in the wake of the e9f71ab2a change

10 years agoMake sure the resolve_cond shim attempts to lint the right thing
Peter Rabbitson [Thu, 4 Sep 2014 22:59:01 +0000]
Make sure the resolve_cond shim attempts to lint the right thing

This only matters in compat mode, so sod testing

10 years ago(travis) Sorry metacpan, your mirror is simply shit >:(
Peter Rabbitson [Fri, 5 Sep 2014 09:16:09 +0000]
(travis) Sorry metacpan, your mirror is simply shit >:(

10 years agoChangelog for RT#98161
Peter Rabbitson [Thu, 4 Sep 2014 15:53:32 +0000]
Changelog for RT#98161

Breakage introduced in 8d005ad9, fixed for good by a combination of 5268b1da
135ac69d and 95da0f23

10 years agoBetter describe the _resolve_relationship_condition API
Peter Rabbitson [Thu, 4 Sep 2014 12:28:25 +0000]
Better describe the _resolve_relationship_condition API

10 years agoAdd one more spot of unroll
Peter Rabbitson [Thu, 4 Sep 2014 10:50:11 +0000]
Add one more spot of unroll

10 years agoOne more fail-case missed in 135ac69dd
Peter Rabbitson [Thu, 4 Sep 2014 10:19:48 +0000]
One more fail-case missed in 135ac69dd

Now the test attached to RT#98161 actually passes... le sigh

At this point I am very wary of the entire codepath: yes, it is clearly the
right thing to do, and the logic is sound, but odd edge cases keep popping
up like this... OTOH there is no way to properly do equality inferrence
without this entire dance, in other words: rock&hardplace. Hopefully the
test suites will shake this out, apologoies to all affected :(

10 years agoBetter optional diag in t/sqlmaker/dbihacks_internals.t
Peter Rabbitson [Thu, 4 Sep 2014 10:17:17 +0000]
Better optional diag in t/sqlmaker/dbihacks_internals.t

10 years agoFix condition collapser corrupting -X operators
Peter Rabbitson [Wed, 3 Sep 2014 10:44:46 +0000]
Fix condition collapser corrupting -X operators

This is (fingercross) the complete fix for RT#98161

10 years agoMake sure empty cond collapser works on all positions
Peter Rabbitson [Tue, 2 Sep 2014 02:20:26 +0000]
Make sure empty cond collapser works on all positions

Adds another round of sql stabilization (akin to 5268b1da6)

10 years agoMake sure cond collapser works case insensitively
Peter Rabbitson [Wed, 27 Aug 2014 06:32:52 +0000]
Make sure cond collapser works case insensitively

10 years agoDeduplicate (and stabilize) the result of _collapse_cond
Peter Rabbitson [Fri, 22 Aug 2014 11:40:14 +0000]
Deduplicate (and stabilize) the result of _collapse_cond

Among other things set the stage to a fix of RT#98161 (later commit)

10 years agoSwitch to a sane deduplication system
Peter Rabbitson [Fri, 22 Aug 2014 08:53:46 +0000]
Switch to a sane deduplication system

The original naive approach from 8e40a627f was utterly braindead and prone
to false positives. Fix this for good

10 years agoRemove a superfluous pair of []s in cond collapser
Peter Rabbitson [Wed, 27 Aug 2014 06:01:31 +0000]
Remove a superfluous pair of []s in cond collapser

10 years agoFix "muse" typo in Relationship::Base's POD.
Stefan Hornburg (Racke) [Thu, 14 Aug 2014 11:56:55 +0000]
Fix "muse" typo in Relationship::Base's POD.

10 years agoUpdate 'as' vs '-as' discussion in ResultSet.pm
Ben Hutton [Fri, 8 Aug 2014 12:55:03 +0000]
Update 'as' vs '-as' discussion in ResultSet.pm

Call me pedantic, but I think moving the statment to a new line makes it
more prominant. I may have seen it before heading to irc to ask why it's
broken. Yes I should have read the section fully, but I didn't.

Also made the two statements use the same terminiolgy to be consistent
and add context.

10 years agoReword the having attribute reference
Olaf Alders [Tue, 5 Aug 2014 16:25:23 +0000]
Reword the having attribute reference

10 years agoAdd extra doc line expanding on the role of the prefetch attribute
Ben Hutton [Thu, 7 Aug 2014 10:17:42 +0000]
Add extra doc line expanding on the role of the prefetch attribute

10 years agoEnsure the TempExtlib actuall exists
Peter Rabbitson [Thu, 7 Aug 2014 03:05:39 +0000]
Ensure the TempExtlib actuall exists

10 years agoS::L tests rely on preserving the sqlmaker between reconnections
Peter Rabbitson [Tue, 5 Aug 2014 12:56:18 +0000]
S::L tests rely on preserving the sqlmaker between reconnections

While this is utterly retarded, punt and backout the b6a469f7 change for now.
Will revisit again during the connect_info consolidation

10 years agoTighten the 'txn-op on disconnected handle' checks
Peter Rabbitson [Tue, 5 Aug 2014 12:37:28 +0000]
Tighten the 'txn-op on disconnected handle' checks

10 years agoSimplify some ::Storage::DBI methods (no tempvars)
Peter Rabbitson [Tue, 5 Aug 2014 12:35:49 +0000]
Simplify some ::Storage::DBI methods (no tempvars)

No functional changes at all

10 years agoFix $rs->populate with column name array but no row data
Dagfinn Ilmari Mannsåker [Fri, 1 Aug 2014 14:17:21 +0000]
Fix $rs->populate with column name array but no row data

Two regressions for both context types snuck in during the the big populate()
rewrite d0cefd99

- In scalar context the return value incorrectly went from () to []
- In void context an empty $data was handed to ::Storage::_insert_bulk

10 years agoDeprecate insert_bulk - we will be changing its signature down the road
Peter Rabbitson [Tue, 5 Aug 2014 11:13:10 +0000]
Deprecate insert_bulk - we will be changing its signature down the road

Besides there is really no reason for users to call it directly

10 years agoFix both a dubious test and a regression in populate args immutability
Peter Rabbitson [Tue, 5 Aug 2014 10:13:16 +0000]
Fix both a dubious test and a regression in populate args immutability

The rewrite in d0cefd99 optimized populate enough that the same data structure
would now make it all the way to the stringifier in ::Storage. This was not
caught due to a deficient test.

Read diff under -w for maximum sense

10 years agoCosmetic rearranging/renaming of some parts of the populate codepath
Peter Rabbitson [Sun, 3 Aug 2014 16:01:56 +0000]
Cosmetic rearranging/renaming of some parts of the populate codepath

Zero functional changes

10 years agostrip schema name from ADD CONSTRAINT / CREATE INDEX
Fitz Elliott [Thu, 26 Jun 2014 19:12:06 +0000]
strip schema name from ADD CONSTRAINT / CREATE INDEX

 * Pg tablenames may have a schema prefix.  This leads to invalid ADD
   CONSTRAINT / CREATE INDEX statments being generated by ->deploy(),
   since constraint and index names may not have a period in them. This
   patch strips the schema part from the table name when constructing
   unique index and constraint names.  The fix was taken from
   ribasushi's email to the mailing list:

   http://lists.scsys.co.uk/pipermail/dbix-class/2013-February/011141.html

10 years ago"Fix" symptoms described in ba7892a8c4 (I still do not entirely get it)
Peter Rabbitson [Wed, 30 Jul 2014 12:00:38 +0000]
"Fix" symptoms described in ba7892a8c4 (I still do not entirely get it)

Extra fail reports are:
http://www.cpantesters.org/cpan/report/f74def2b-6bfb-1014-b341-c67b58fe2ee5
http://www.cpantesters.org/cpan/report/3123ab19-6bf8-1014-b488-656aa945bbef

10 years agoEnsure proper behavior when quoting is *disabled* (wraps up 08ac7648)
Peter Rabbitson [Wed, 30 Jul 2014 11:48:22 +0000]
Ensure proper behavior when quoting is *disabled* (wraps up 08ac7648)

10 years agoThe extra reltype flag is breaking some suites - make the full change
Peter Rabbitson [Tue, 29 Jul 2014 09:17:36 +0000]
The extra reltype flag is breaking some suites - make the full change

Might as well get this over with, still consistent with Swindon

10 years agoRemove unused leftover from c19ca6e80 - there was much flailing :(
Peter Rabbitson [Tue, 29 Jul 2014 09:01:11 +0000]
Remove unused leftover from c19ca6e80 - there was much flailing :(

10 years agoKill rel-exists sanity checks for 28, reinstate them later
Peter Rabbitson [Tue, 29 Jul 2014 08:44:41 +0000]
Kill rel-exists sanity checks for 28, reinstate them later

Something that simple allows RapidApp to pass its tests without modifications
in other words - worth the pain

10 years agoThe fix in f9b5239ac was both shortsighted and insufficient
Peter Rabbitson [Tue, 29 Jul 2014 05:12:16 +0000]
The fix in f9b5239ac was both shortsighted and insufficient

Move the quote_names => quote_identifiers inferrence into ::Storage::DBI
proper, and ensure tests cover it this time

Also bump SQLT dep, as the previous version throws on diffing DBICTest::Schema
under SQLite

10 years agoCheck relationship declarations more consistently
Peter Rabbitson [Tue, 29 Jul 2014 03:45:11 +0000]
Check relationship declarations more consistently

10 years agoRevert improved test added in b6b8f72f, replace with original from dabe173a
Peter Rabbitson [Tue, 29 Jul 2014 03:20:16 +0000]
Revert improved test added in b6b8f72f, replace with original from dabe173a

The new test contains a case which it *seems* to me needs to be an exception.
Since we are already pretty late with 28, I am just reverting the test alone
(while keeping the changes). This way I am not backing myself into a corner
with a published test in 29.

10 years agoMore diag from t/53lean_startup.t when under smoker env
Peter Rabbitson [Tue, 29 Jul 2014 03:14:00 +0000]
More diag from t/53lean_startup.t when under smoker env

Trying to pinpoint cause for the following two:
http://www.cpantesters.org/cpan/report/c13d6ffc-6bf4-1014-8850-81d5948d76d3
http://www.cpantesters.org/cpan/report/51d7cbe3-6bf5-1014-88de-68e8687ecb02

10 years agoA little cleanup of ->id callsites (incomplete)
Peter Rabbitson [Tue, 29 Jul 2014 02:17:04 +0000]
A little cleanup of ->id callsites (incomplete)

This is just to stave off an unlikely but possible 'id' colname masking a
multicol PK

The real workaround is to do something akin to Moose's ->meta, but that's
another battle.

10 years agoProperly throw on FC with find (it can never work anyway)
Peter Rabbitson [Tue, 29 Jul 2014 01:06:09 +0000]
Properly throw on FC with find (it can never work anyway)

This is the proper solution for RT#95054

10 years agoDeprecate _build_unique_cond, move its guts to ::ResultSource where it belongs
Peter Rabbitson [Tue, 29 Jul 2014 00:12:33 +0000]
Deprecate _build_unique_cond, move its guts to ::ResultSource where it belongs

10 years agoThe merger failing is no longer a possibility after 8e40a627
Peter Rabbitson [Mon, 28 Jul 2014 08:26:46 +0000]
The merger failing is no longer a possibility after 8e40a627

Remove the useless try{} block

10 years agoShuffle logic in find() a bit, no functional changes
Peter Rabbitson [Mon, 28 Jul 2014 07:29:13 +0000]
Shuffle logic in find() a bit, no functional changes

10 years agoFix set_column on non-native (+columns) selections (RT#86685)
Peter Rabbitson [Sun, 27 Jul 2014 22:39:19 +0000]
Fix set_column on non-native (+columns) selections (RT#86685)

10 years agoTighten even more inflate_result() invocations, for crazy overloads
Peter Rabbitson [Sun, 27 Jul 2014 21:50:25 +0000]
Tighten even more inflate_result() invocations, for crazy overloads

10 years agoEnsure collapse is respected regardless of selection type
Peter Rabbitson [Sun, 27 Jul 2014 12:13:45 +0000]
Ensure collapse is respected regardless of selection type

Switch the attribute name/logic (passthrough on explicit presence only)

10 years agoUpgrade the "too many args on customcond" warning from 1adbd3f to an exception
Peter Rabbitson [Sun, 27 Jul 2014 16:39:31 +0000]
Upgrade the "too many args on customcond" warning from 1adbd3f to an exception

10 years agoAdd reference to RT for the fix from d0cefd99
Peter Rabbitson [Sun, 27 Jul 2014 13:45:38 +0000]
Add reference to RT for the fix from d0cefd99

10 years agoBring back resolve_condition deleted in 03f6d1f7
Peter Rabbitson [Fri, 25 Jul 2014 14:17:44 +0000]
Bring back resolve_condition deleted in 03f6d1f7

It doesn't cost anything, and the compat shim turned out to be good enough
to make this worthwhile. Also apparently CPAN is using it still FML:
http://grep.cpan.me/?q=->_*resolve_condition

10 years agoThis was an embarrassing close call - entirely redo custom set_from_related
Peter Rabbitson [Fri, 25 Jul 2014 12:35:19 +0000]
This was an embarrassing close call - entirely redo custom set_from_related

I am flailing at this point - unacceptable. TL;DR: 03f6d1f7 is wrong, 1adbd3fc
is wrong, so is 5592d63 and most importantly 350e8d57 is unacceptably wrong.

The entire "I will inflate a bag of data into a synthetic object" was wrong
from the very beginning (03f6d1f7) but I was too busy keeping the tets passing
to realize what crack-mountain I was standing on.

The crux of the issue is: it didn't occur to me that forward-resolution
(related_resultset) *always* starts from an object by definition, while
reverse-resolution (set_from_related) takes either an object or a bag of
values or undef. Moreover: in the custom coderef the user does not care at
all about the type - all they want is to get their values. In light of this
self_result_object is actually a mistake - we should have simply provided
self_values from the start, but that ship has already sailed. On the other
hand the reverse-resolution bag is a new feature, and we can simply (and
correctly) settle on passing in a hashref of values: foreign_values

This fully avoids the "how do I get an object out of plain data" problem
and nicely settles all the outsanding (untested until now) problems.

I just wish I would have seen this a week earlier... sigh. At least not
shipping this live is a consolation...

10 years agoChange version style/handling (doc coming up shortly)
Peter Rabbitson [Thu, 24 Jul 2014 02:17:09 +0000]
Change version style/handling (doc coming up shortly)

10 years agoMinor limit dialects doc fix
Jens Gassmann [Thu, 24 Jul 2014 02:06:38 +0000]
Minor limit dialects doc fix

10 years agoAdded support for handling Class::DBI::Column in CDBICompat
Mike Francis [Mon, 21 Jul 2014 15:13:46 +0000]
Added support for handling Class::DBI::Column in CDBICompat

10 years agoAdded handling for implicit inflate/deflate of CDBI has_a relationships
Mike Francis [Fri, 18 Jul 2014 13:45:51 +0000]
Added handling for implicit inflate/deflate of CDBI has_a relationships

10 years agoAdd 3rd example of col info in ResultSource->add_column
Mintywalker [Sat, 12 Apr 2014 17:13:27 +0000]
Add 3rd example of col info in ResultSource->add_column

Rather than just say "it's a hashref", provide a simple example of what that
hashref might look like.

10 years agoThe current_source_alias is prepended, not a literal "me."
Karen Etheridge [Wed, 23 Jul 2014 19:25:55 +0000]
The current_source_alias is prepended, not a literal "me."

10 years agoBundle trial SQLA for the trial of DBIC (reuse dq technique f4a8b21e)
Peter Rabbitson [Wed, 23 Jul 2014 18:02:52 +0000]
Bundle trial SQLA for the trial of DBIC (reuse dq technique f4a8b21e)

Supersedes 609fa215, to be reverted before 0.082800