Fix func_rs() and as_subselect_rs() to start behaving as advertised
authorPeter Rabbitson <ribasushi@cpan.org>
Tue, 27 Sep 2016 17:35:15 +0000 (19:35 +0200)
committerPeter Rabbitson <ribasushi@cpan.org>
Fri, 30 Sep 2016 15:45:56 +0000 (17:45 +0200)
commit6c5aa1fbffdc9e5679d2f68780b11a9569ec1993
tree529a34b225c5f43c0b9703aa1f71d5bee094baea
parent9ab0364d36a4357b766f6dfccfb1df5ef69b079b
Fix func_rs() and as_subselect_rs() to start behaving as advertised

No idea how it never got noticed, but both have been broken since the very
first commits that introduced the methods ( 4fa7bc22 / e4bb6727 ). While
changing them 7 years later is a rather serious modification of behavior,
the old way never worked without users having to force-scalar each call site.

If someone has been relying on e.g. [ func_rs(...) ] to return actual result
objects instead of the resultset instance - things will blow up rather quickly
and loudly (aside from the carp()-ed warning encouraging users to switch to
scalar ctx explicitly)

[ func( ... ) ] of course continues to behave like before (directly returning
raw values off the cursor... sigh)
Changes
lib/DBIx/Class/ResultSet.pm
lib/DBIx/Class/ResultSetColumn.pm
t/88result_set_column.t