Add an extra RV to the relationship resolver
authorPeter Rabbitson <ribasushi@cpan.org>
Wed, 10 Aug 2016 14:19:54 +0000 (16:19 +0200)
committerPeter Rabbitson <ribasushi@cpan.org>
Tue, 27 Sep 2016 08:21:29 +0000 (10:21 +0200)
commita3ae79ed1009ae4679909f4ec7dc0327c1adaae8
tree5fdb4401886ae4284375854fc0b69e37a74586d5
parentea3ee77d2d9e137b07ca4b2db14986e8310f4bec
Add an extra RV to the relationship resolver

A certain spot in the codebase check whether a relationship is "simple".
This additional flag allows to consider coderef conditions as well, instead
of simply punting with "not a HASH? - no can do"

See next commit for the actual switchover

While at it fix a subtle bug introduced in b5ce6748 - originally the helper
is_literal_value recognized -ident as a valid literal. Later on during the
migration into SQLA this logic was lost (I do not exactly recall the details),
yet the DBIC side was never adjusted. All callsites were audited to make sure
nothing else was missed.
lib/DBIx/Class/ResultSource.pm
lib/DBIx/Class/SQLMaker.pm
lib/DBIx/Class/_Util.pm