<p>This has now been merged at <a href="https://github.com/dbsrgits/dbix-class/commit/39e2a212" class="commit-link"><tt>39e2a21</tt></a> with the seemingly more correct change:</p>
<pre><code> # Class::DBI allows Non database has_a with implicit deflate and inflate
# Hopefully the following will catch Non-database tables.
- if (!$f_class->isa('DBIx::Class') && !$f_class->isa('Class::DBI')){
- $args{'inflate'} ||= sub { $f_class->new(shift); }; # implicite inflate by calling new
- $args{'deflate'} ||= sub { my $value = shift; "$value"; }; # implicite deflate by stringification
+ if( !$f_class->isa('DBIx::Class::Row') and !$f_class->isa('Class::DBI::Row') ) {
+ $args{'inflate'} ||= sub { $f_class->new(shift) }; # implicit inflate by calling new
+ $args{'deflate'} ||= sub { shift . '' }; # implicit deflate by stringification
}
</code></pre>
<p>Please let me know if this is in fact wrong (perhaps you <em>do</em> want to exempt ::ResultSet instances just like ::Row... though it doesn't make much sense)</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br>Reply to this email directly or <a href="https://github.com/dbsrgits/dbix-class/pull/51#issuecomment-49957786">view it on GitHub</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/302594__eyJzY29wZSI6Ik5ld3NpZXM6QmVhY29uIiwiZXhwaXJlcyI6MTcyMTc4NDcwOCwiZGF0YSI6eyJpZCI6MzczNzU1Nzd9fQ==--f5042ac0876b17a769082999422da63230b32176.gif" width="1" /></p>