Ensure the custom rel cond resolver does not trigger forgotten compat shim
authorPeter Rabbitson <ribasushi@cpan.org>
Sun, 14 Sep 2014 18:59:58 +0000 (20:59 +0200)
committerPeter Rabbitson <ribasushi@cpan.org>
Mon, 15 Sep 2014 12:41:06 +0000 (14:41 +0200)
commitc200d94979bde5ac74070d3e898927433b0e667c
treef0c4ecf04e0beacb78182cbd13c7236837b446fe
parent21621fe4565697ead298e6829425dc0e9e5ba816
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.
lib/DBIx/Class/ResultSource.pm
lib/DBIx/Class/SQLMaker/OracleJoins.pm
t/lib/DBICTest/Schema/Track.pm
t/relationship/custom.t