Crazy looping in render partial

And to possibly save you your next message,
doesn't change @picks.

Perhaps you want:

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

Note the use of '-' 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 Biedenharn
Skype: rob.biedenharn