[Bast-commits] r8565 - DBIx-Class/0.08/branches/chaining_fixes/lib/DBIx/Class

[ribasushi at dev.catalyst.perl.org]

Author: ribasushi
Date: 2010-02-05 18:11:09 +0000 (Fri, 05 Feb 2010)
New Revision: 8565

Modified:
   DBIx-Class/0.08/branches/chaining_fixes/lib/DBIx/Class/ResultSetColumn.pm
Log:
Should not be needed

Modified: DBIx-Class/0.08/branches/chaining_fixes/lib/DBIx/Class/ResultSetColumn.pm
===================================================================
--- DBIx-Class/0.08/branches/chaining_fixes/lib/DBIx/Class/ResultSetColumn.pm	2010-02-05 18:10:06 UTC (rev 8564)
+++ DBIx-Class/0.08/branches/chaining_fixes/lib/DBIx/Class/ResultSetColumn.pm	2010-02-05 18:11:09 UTC (rev 8565)
@@ -83,11 +83,6 @@
   $new_parent_rs ||= $rs->search_rs;
   my $new_attrs = $new_parent_rs->{attrs} ||= {};
 
-  # FIXME - this should go away when the chaining branch is merged
-  # since what we do is actually chain to the original resultset, we need to throw
-  # away all selectors (otherwise they'll chain)
-  delete $new_attrs->{$_} for (qw/columns +columns select +select as +as cols include_columns/);
-
   # prefetch causes additional columns to be fetched, but we can not just make a new
   # rs via the _resolved_attrs trick - we need to retain the separation between
   # +select/+as and select/as. At the same time we want to preserve any joins that the