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