Crazy looping in render partial

And to possibly save you your next message,
    @picks.sort_by{|x| x.id}.reverse
doesn't change @picks.

Perhaps you want:

def index
   @picks =Product.find_person(session[:user_id]).sort_by {|p| - p.id }
   render :partial => "pick", :collection => @picks
end

Note the use of '- p.id' to reverse the order in one step. It also seems odd (to me at least) that "find_person" returns more than one object since it is singular; "find_people" might be clearer. Of course, that just makes me wonder why the Product model is finding a Person at all.

-Rob

Rob Biedenharn http://agileconsultingllc.com
Rob@AgileConsultingLLC.com
Skype: rob.biedenharn