<div class="gmail_quote">On Wed, Jan 26, 2011 at 1:34 AM, Rob Kinyon <span dir="ltr">&lt;<a href="mailto:rob.kinyon@gmail.com">rob.kinyon@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im"><br>
</div>Although this advice wasn&#39;t because of avoiding this bug, it&#39;s<br>
generally considered to be a &quot;Bad Plan&quot;™ to make a relationship name<br>
the same as the foreign-key column you&#39;re using to fetch on. </blockquote><div><br></div><div>Take it from someone who made this mistake long ago and is still paying for it. If I try to switch the relationship names, I&#39;m dooming myself to endless bugfixes. :( I think newer D:C::Schema::Loader versions use different names, &quot;${column}_rel&quot; IIRC.</div>
<div><br></div><div>The repeated get_column() calls are ugly, ugly, ugly. Worse, forgetting to use get_column() potentially means I add more unneeded load on the database.</div><div><br></div><div>Drew</div></div>