<div class="gmail_quote">On Wed, Jan 26, 2011 at 1:34 AM, Rob Kinyon <span dir="ltr"><<a href="mailto:rob.kinyon@gmail.com">rob.kinyon@gmail.com</a>></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't because of avoiding this bug, it's<br>
generally considered to be a "Bad Plan"™ to make a relationship name<br>
the same as the foreign-key column you'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'm dooming myself to endless bugfixes. :( I think newer D:C::Schema::Loader versions use different names, "${column}_rel" 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>